----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/128916/#review99210 -----------------------------------------------------------
autotests/kconfigtest.cpp <https://git.reviewboard.kde.org/r/128916/#comment66800> I'd put both `cgLocal.writeEntry("someLocalString", "whatever2");` and `QVERIFY(!cgLocal.sync());` inside else clause. - Gleb Popov On Sept. 15, 2016, 2:59 p.m., Evgeniy Sadovnik wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/128916/ > ----------------------------------------------------------- > > (Updated Sept. 15, 2016, 2:59 p.m.) > > > Review request for KDE Frameworks and Gleb Popov. > > > Repository: kconfig > > > Description > ------- > > The test checks that saving a read-only config file fails. But because root > can write into read-only files, the test is failing when running by root. > Check for uid when running the test and make it XFAIL if we are running as > root. > > > Diffs > ----- > > autotests/kconfigtest.cpp 2b905b5 > > Diff: https://git.reviewboard.kde.org/r/128916/diff/ > > > Testing > ------- > > > Thanks, > > Evgeniy Sadovnik > >