Review Request 108560: Fix kconfigcompiler_test

Alexander Richardson arichardson.kde at googlemail.com
Tue Jan 22 21:45:14 GMT 2013


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

Review request for kdelibs.


Summary (updated)
-----------------

Fix kconfigcompiler_test


Description (updated)
-------

Fix kconfigcompiler_test

It previously did not find the required files since CMake set TESTSRCDIR
wrongly. It was surrounded by quotes and lacking a trailing /

This did not cause test failures since QCOMPARE/QVERIFY was not used
directly in the test function, so it simply printed messages instead.


Diffs (updated)
-----

  tier2/kconfig/autotests/kconfig_compiler/CMakeLists.txt 846476a6370ad41b1fbc808e2c738c85042e59a3 
  tier2/kconfig/autotests/kconfig_compiler/kconfigcompiler_test.h 1be213f5036c03a3ef9347005d5d53e6b440a4d4 
  tier2/kconfig/autotests/kconfig_compiler/kconfigcompiler_test.cpp f900aaf53de1996213c27357fee5b436ab3c4c8f 

Diff: http://git.reviewboard.kde.org/r/108560/diff/


Testing (updated)
-------

Unit test now passes instead of printing messages and failing silently


Thanks,

Alexander Richardson

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20130122/2f47e7d9/attachment.htm>


More information about the kde-core-devel mailing list