Review Request 126341: app notifier is crashing right now. build a test around it

Harald Sitter sitter at kde.org
Mon Dec 14 09:36:14 UTC 2015


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/126341/
-----------------------------------------------------------

Review request for Kubuntu, Muon Package Management Suite and Aleix Pol Gonzalez.


Repository: discover


Description
-------

prevent notifier from crashing when process was not initialized yet


Diffs
-----

  libdiscover/backends/ApplicationBackend/ApplicationNotifier.cpp 7611d046b5761b75f9535415b9b8903fef7ed49b 
  libdiscover/backends/ApplicationBackend/CMakeLists.txt e2bb7fbeecde6983aa48b6bdb79a6c6fd41c83e3 
  libdiscover/backends/ApplicationBackend/tests/CMakeLists.txt 274adf3a1f82ee8cd2f0f1bd6392d005b09195c6 
  libdiscover/backends/ApplicationBackend/tests/NotifierTest.h PRE-CREATION 
  libdiscover/backends/ApplicationBackend/tests/NotifierTest.cpp PRE-CREATION 

Diff: https://git.reviewboard.kde.org/r/126341/diff/


Testing
-------

ran test without fix -> exception fail
ran test with fix -> pass


Thanks,

Harald Sitter

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.ubuntu.com/archives/kubuntu-devel/attachments/20151214/b3686ea3/attachment.html>


More information about the kubuntu-devel mailing list