Review Request 127398: Add unit tests for KNotification and fix a whole bunch of issues uncovered thanks to them
David Edmundson
david at davidedmundson.co.uk
Fri Mar 18 01:56:36 UTC 2016
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/127398/#review93674
-----------------------------------------------------------
src/knotification.cpp (line 396)
<https://git.reviewboard.kde.org/r/127398/#comment63865>
what if id == 0 ?
- David Edmundson
On March 16, 2016, 8:23 p.m., Martin Klapetek wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/127398/
> -----------------------------------------------------------
>
> (Updated March 16, 2016, 8:23 p.m.)
>
>
> Review request for KDE Frameworks.
>
>
> Repository: knotifications
>
>
> Description
> -------
>
> Adds basic set of unit tests including fake notifications server.
>
> This helped uncover and fix these issues:
>
> * Calling close() on KNotification that was not "sent" would not emit closed() and would not delete it
> * Closing a notification can delete the KNotification object prematurely
> * Invoking an action leads to unnecessary dbus roundtrip
> * Invoking an action would fail to properly close and delete the KNotification object
>
>
> Diffs
> -----
>
> CMakeLists.txt 6d09051
> autotests/CMakeLists.txt PRE-CREATION
> autotests/fake_notifications_server.h PRE-CREATION
> autotests/fake_notifications_server.cpp PRE-CREATION
> autotests/knotification_test.cpp PRE-CREATION
> autotests/knotifications5/qttest.notifyrc PRE-CREATION
> autotests/qtest_dbus.h PRE-CREATION
> src/knotification.cpp 17b0be2
> src/knotificationmanager.cpp e80d48c
> src/knotificationplugin.cpp acf964c
> src/notifybypopup.cpp b9489c1
>
> Diff: https://git.reviewboard.kde.org/r/127398/diff/
>
>
> Testing
> -------
>
> Unit tests all pass.
>
>
> Thanks,
>
> Martin Klapetek
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20160318/4b7bd686/attachment.html>
More information about the Kde-frameworks-devel
mailing list