Skip to content

Commit

Permalink
BUG: Fix ctkCoreSettingsTest testMultiplePaths on Windows
Browse files Browse the repository at this point in the history
Follow-up on commit f0bb5d7 (ENH: Allow saving relative paths to settings)
where the test case was incorrectly specified.

It fixes the following error:

10: FAIL!  : ctkCoreSettingsTester::testMultiplePaths(windows-style absolute paths) Compared lists differ at index 0.
10:    Actual   (settings.value(key).toStringList()): "d:/windows/path/internal/dir"
10:    Expected (expectedStoredValues): "<APPLICATION_HOME_DIR>/internal/dir"
10: C:\path\to\CTK\Libs\Core\Testing\Cpp\ctkCoreSettingsTest.cpp(115) : failure location
  • Loading branch information
jcfr committed Jan 13, 2024
1 parent 2cc640a commit 2772723
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion Libs/Core/Testing/Cpp/ctkCoreSettingsTest.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -139,7 +139,7 @@ void ctkCoreSettingsTester::testMultiplePaths_data()
<< (QStringList() << "<APPLICATION_HOME_DIR>/internal/dir");

QTest::newRow("windows-style absolute paths") << "c:/windows/path"
<< (QStringList() << "d:/windows/path/internal/dir" << "c:/windows/external/subdir/file.txt")
<< (QStringList() << "c:/windows/path/internal/dir" << "c:/windows/external/subdir/file.txt")
<< (QStringList() << "<APPLICATION_HOME_DIR>/internal/dir" << "c:/windows/external/subdir/file.txt");
#endif

Expand Down

0 comments on commit 2772723

Please sign in to comment.