Review Request 117329: Split presence handling into a new class and add easier way to test.

Commit Hook null at kde.org
Thu Apr 3 11:35:46 UTC 2014


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


This review has been submitted with commit 139e22761c2511906435b0e8149354ff8750d6e1 by David Edmundson to branch master.

- Commit Hook


On April 2, 2014, 11:08 a.m., David Edmundson wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/117329/
> -----------------------------------------------------------
> 
> (Updated April 2, 2014, 11:08 a.m.)
> 
> 
> Review request for Telepathy.
> 
> 
> Repository: ktp-kded-module
> 
> 
> Description
> -------
> 
> Split presence handling into a new class. Code is a direct copy/paste from telepathy-module (changes to that should happen in later commits)
> 
> Add small app for running the status handling of the kded
> Rather than restarting kded all the time developers can run the small test app to check the status handling.
> 
> 
> Should be possible to turn this into a proper unit test at some point in the near future later.
> 
> 
> Diffs
> -----
> 
>   CMakeLists.txt d08606b 
>   status-handler.h PRE-CREATION 
>   status-handler.cpp PRE-CREATION 
>   telepathy-module.h 2213cdf 
>   telepathy-module.cpp 030a0d9 
>   tests/CMakeLists.txt 7696369 
>   tests/status-handling-main.cpp PRE-CREATION 
> 
> Diff: https://git.reviewboard.kde.org/r/117329/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> David Edmundson
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-telepathy/attachments/20140403/00a94077/attachment.html>


More information about the KDE-Telepathy mailing list