feedc0de/qt-creator
1
0
Fork 0
forked from qt-creator/qt-creator
Code Pull Requests Activity

CamelHump: Extend tests

Browse Source 
The else part within "if (!c.isLetter())"
(camelhumpmatcher.cpp:79) was not covered
by the tests at all, so far.

Change-Id: I35eb5908110e4692cda41b58f419867da00b6501
Reviewed-by: Orgad Shaneh <orgads@gmail.com>
This commit is contained in:
Andre Hartmann
2017-10-04 04:50:15 +02:00
committed by André Hartmann
parent 7d35e08afe
commit 0fb6a1479f
1 changed files with 3 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

3
tests/auto/utils/camelhumpmatcher/tst_camelhumpmatcher.cpp
View File

@@ -68,6 +68,7 @@ void tst_CamelHumpMatcher::camelHumpMatcher_data()
QTest::newRow("incorrect-hump") << "lyn" << "VeryLongCamelHump" << -1;
QTest::newRow("humps") << "VL" << "VeryLongCamelHump" << 0;
QTest::newRow("skipped-humps-upper") << "VH" << "VeryLongCamelHump" << -1;
QTest::newRow("numbers") << "4" << "Test4Fun" << 4;
QTest::newRow("question-wildcard") << "Lon?Ca" << "VeryLongCamelHump" << 4;
QTest::newRow("unmatched-question-wildcard") << "Long?Ca" << "VeryLongCamelHump" << -1;
QTest::newRow("asterisk-wildcard") << "Long*Ca" << "VeryLongCamelHump" << 4;
@@ -133,6 +134,8 @@ void tst_CamelHumpMatcher::highlighting_data()
<< MatchStart{13} << MatchLength{1};
QTest::newRow("humps-continued") << "LoCa" << "VeryLongCamelHump"
<< MatchStart{4, 8} << MatchLength{2, 2};
QTest::newRow("numbers") << "4" << "TestJust4Fun"
<< MatchStart{8} << MatchLength{1};
QTest::newRow("wildcard-asterisk") << "Lo*Hu" << "VeryLongCamelHump"
<< MatchStart{4, 13} << MatchLength{2, 2};
QTest::newRow("wildcard-question") << "Lo?g" << "VeryLongCamelHump"