diff options
| author | Thiago Macieira <thiago.macieira@intel.com> | 2022-09-06 10:23:36 -0700 |
|---|---|---|
| committer | Thiago Macieira <thiago.macieira@intel.com> | 2022-09-16 08:37:49 -0700 |
| commit | 4b997d1851bc62b9a1eb761f08d3b22c85bb0b51 (patch) | |
| tree | 015a3e7518f1189d1d9487e767ce51c5364a2ee7 /tests/auto/corelib/io/qfile | |
| parent | 68f641095c5403925e49954aea0e65c4574121e0 (diff) | |
QFileDevice: clear cached permissions on setPermission()
In theory, if we succeed, the permissions should be what we set, but
let's not make that assumption. And if we failed, it might be because
the file disappeared or something else, so re-stat()ing the file is a
good idea.
Pick-to: 6.4
Fixes: QTBUG-7211
Change-Id: If5d5ef6220874ae8858efffd171255506b7bbee0
Reviewed-by: Mårten Nordheim <marten.nordheim@qt.io>
Diffstat (limited to 'tests/auto/corelib/io/qfile')
| -rw-r--r-- | tests/auto/corelib/io/qfile/tst_qfile.cpp | 6 |
1 files changed, 6 insertions, 0 deletions
diff --git a/tests/auto/corelib/io/qfile/tst_qfile.cpp b/tests/auto/corelib/io/qfile/tst_qfile.cpp index e8fcb7d9ce..3c738abae4 100644 --- a/tests/auto/corelib/io/qfile/tst_qfile.cpp +++ b/tests/auto/corelib/io/qfile/tst_qfile.cpp @@ -1423,6 +1423,9 @@ void tst_QFile::permissionsNtfs() void tst_QFile::setPermissions() { +#ifdef Q_OS_QNX + QSKIP("This test doesn't pass on QNX and no one has cared to investigate."); +#endif if ( QFile::exists( "createme.txt" ) ) QFile::remove( "createme.txt" ); QVERIFY( !QFile::exists( "createme.txt" ) ); @@ -1433,9 +1436,12 @@ void tst_QFile::setPermissions() f.close(); QFile::Permissions perms(QFile::WriteUser | QFile::ReadUser); + QVERIFY(f.setPermissions(QFile::ReadUser)); + QVERIFY((f.permissions() & perms) == QFile::ReadUser); QVERIFY(f.setPermissions(perms)); QVERIFY((f.permissions() & perms) == perms); + // we should end the test with the file in writeable state } void tst_QFile::copy() |