----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/128916/#review99231 -----------------------------------------------------------
autotests/kconfigtest.cpp (line 1376) <https://git.reviewboard.kde.org/r/128916/#comment66819> Isn't a QSKIP if user is root in the beginning of the test a better approach ? - Sune Vuorela On Sept. 16, 2016, 7:17 a.m., Evgeniy Sadovnik wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/128916/ > ----------------------------------------------------------- > > (Updated Sept. 16, 2016, 7:17 a.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 > >