D15923: Fix system language dependent unit test
Peter Braun
noreply at phabricator.kde.org
Wed Oct 3 17:18:02 BST 2018
pbraun created this revision.
pbraun added a reviewer: Konsole.
pbraun added a project: Konsole.
Herald added a subscriber: konsole-devel.
pbraun requested review of this revision.
REVISION SUMMARY
When running the konsole unit tests on a system where the language setting is not english the test`ProfileTest::testFallbackProfile` fails because it compares the translated value of the `Name` property with the string literal `"Default"`. To fix the test we can use `UntranslatedName` instead.
TEST PLAN
1. Run the konsole unit tests on a system where the language is not english
Output of `ctest --verbose` before the fix:
8: FAIL! : Konsole::ProfileTest::testFallbackProfile() Compared values are not the same
8: Actual (fallback->property<QString>(Profile::Name)): "Standard"
8: Expected (QStringLiteral("Default")) : "Default"
8: Loc: [/var/run/media/peter/MEDIA/Projects/kde/konsole/src/autotests/ProfileTest.cpp(237)]
After the fix all tests should pass.
REPOSITORY
R319 Konsole
REVISION DETAIL
https://phabricator.kde.org/D15923
AFFECTED FILES
src/autotests/ProfileTest.cpp
To: pbraun, #konsole
Cc: konsole-devel, herrold, ngraham, maximilianocuria, hindenburg
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/konsole-devel/attachments/20181003/95de1a82/attachment-0001.html>
More information about the konsole-devel
mailing list