Review Request 108560: Fix kconfigcompiler_test by porting to QFINDTESTDATA
Commit Hook
null at kde.org
Thu Jan 24 09:38:34 UTC 2013
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/108560/#review26116
-----------------------------------------------------------
This review has been submitted with commit 78aac03412c1d8191349e8099f4d8af2f1d35884 by Alex Richardson to branch frameworks.
- Commit Hook
On Jan. 23, 2013, 6:14 p.m., Alexander Richardson wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/108560/
> -----------------------------------------------------------
>
> (Updated Jan. 23, 2013, 6:14 p.m.)
>
>
> Review request for KDE Frameworks.
>
>
> Description
> -------
>
> Fix kconfigcompiler_test by porting to QFINDTESTDATA
>
> 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
> -----
>
> 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
> -------
>
> 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-frameworks-devel/attachments/20130124/c49b37f9/attachment-0001.html>
More information about the Kde-frameworks-devel
mailing list