Details
-
Bug
-
Resolution: Fixed
-
P2: Important
-
None
-
4.6.0, 6.8
-
None
-
25
-
713926eb6 (dev), 24424fa93 (dev)
-
Foundation Sprint 115
Description
This test was accidentally disabled in Qt 4.6 commit 31e358f2290c145b839fc5b7b277922c1ab6e19b ("fix tests for QT_NO_PROCESS and when running tests as root"), which introduced an #else that disabled the entire meat of the test (that is, the test was for everything that is neither Unix nor Windows, which is an empty set).
Attempting to re-enable it shows it's broken:
FAIL! : tst_QSettings::testErrorHandling(0600 0100) Compared values are not the same Actual ((int)settings.status()): 1 Expected (statusAfterSetAndSync) : 0 Loc: [tests/auto/corelib/io/qsettings/tst_qsettings.cpp(941)] FAIL! : tst_QSettings::testErrorHandling(0600 0100) 'settingsDir.removeRecursively()' returned FALSE. () Loc: [tests/auto/corelib/io/qsettings/tst_qsettings.cpp(314)] FAIL! : tst_QSettings::testErrorHandling( -1 0300) 'settingsDir.removeRecursively()' returned FALSE. () Loc: [tests/auto/corelib/io/qsettings/tst_qsettings.cpp(314)] FAIL! : tst_QSettings::testErrorHandling(0600 0300) 'settingsDir.removeRecursively()' returned FALSE. () Loc: [tests/auto/corelib/io/qsettings/tst_qsettings.cpp(314)] FAIL! : tst_QSettings::testErrorHandling(0600 0500) Compared values are not the same Actual ((int)settings.status()): 1 Expected (statusAfterSetAndSync) : 0 Loc: [tests/auto/corelib/io/qsettings/tst_qsettings.cpp(941)] FAIL! : tst_QSettings::testErrorHandling(0600 0500) 'settingsDir.removeRecursively()' returned FALSE. () Loc: [tests/auto/corelib/io/qsettings/tst_qsettings.cpp(314)]
It appears the majority of the failures are in the cleanup code, after the test has executed, so it's possible it's actually passing, but the clean up fails because some files or directories are left without reading permissions.
Attachments
| For Gerrit Dashboard: QTBUG-126008 | ||||||
|---|---|---|---|---|---|---|
| # | Subject | Branch | Project | Status | CR | V |
| 563519,4 | tst_QSettings: make it clear testErrorHandling isn't run | dev | qt/qtbase | Status: MERGED | +2 | 0 |
| 587410,10 | cleanupTestFiles: make directories available for removing on UNIX | dev | qt/qtbase | Status: MERGED | +2 | 0 |