dfaure created this revision.
dfaure added reviewers: mdawson, apol, mpyne.
Restricted Application added a project: Frameworks.
REVISION SUMMARY
The CI has $HOME=/home/jenkins/ with a trailing slash, which leads to
FAIL! : KConfigTest::testPath() Compared values are not the same
Actual (sc3.readPathEntry("homepath", QString())): "/home/jenkins//foo"
Expected (HOMEPATH): "/home/jenkins/foo"
The fix is to also replace the '/' in "$HOME/", when the env var ends
with '/' and there is indeed a '/' after the env var in the source
string.
TEST PLAN
kconfigtest now passes with HOME=/home/dfaure/, and the newly
added unittest passes.
REPOSITORY
R237 KConfig
BRANCH
master
REVISION DETAIL
https://phabricator.kde.org/D9097
AFFECTED FILES
autotests/kconfigtest.cpp
src/core/kconfig.cpp
To: dfaure, mdawson, apol, mpyne
Cc: #frameworks
