Review Request 116030: Extend tests to cover getConf... calls

Kevin Krammer krammer at kde.org
Tue Feb 25 16:17:08 UTC 2014


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

(Updated Feb. 25, 2014, 4:17 p.m.)


Review request for KDE Frameworks and Chusslove Illich.


Changes
-------

Added function to remove test config after test and call it from cleanupTestCase.
Renamed new config file to ktranscript.ini, should fit better in cross-platform world


Repository: ki18n


Description
-------

Write a test config to a test location using QStandardPath's test feature.
Test getConf... calls in success and fallback mode.
Actually found a missing bool -> script bool conversion. fixed

Chusslove: how about using ktranscript.ini for the file to look up using QStandardPaths? Maybe a more obvious on other platforms?


Diffs (updated)
-----

  autotests/CMakeLists.txt 6e926ba 
  autotests/ktranscripttest.h 7ea7818 
  autotests/ktranscripttest.cpp e3a27ff 
  autotests/test.js ad53b1b 
  autotests/testhelpers.h PRE-CREATION 
  autotests/testhelpers.cpp PRE-CREATION 
  src/ktranscript.cpp 44c8b63 

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


Testing
-------

All previously existing tests continue to run :)


Thanks,

Kevin Krammer

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20140225/669a6b25/attachment.html>


More information about the Kde-frameworks-devel mailing list