----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/127952/#review95579 -----------------------------------------------------------
src/kicontheme.h (line 223) <https://git.reviewboard.kde.org/r/127952/#comment64757> what about just a testmode bool like qstandardpaths? - Marco Martin On May 18, 2016, 8:25 a.m., Harald Sitter wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/127952/ > ----------------------------------------------------------- > > (Updated May 18, 2016, 8:25 a.m.) > > > Review request for KDE Frameworks, Christoph Feck, David Edmundson, and Marco > Martin. > > > Repository: kiconthemes > > > Description > ------- > > this allows unit tests to force a specific name to be used regardless of > which branch of the current() resolution would bite first. the override > wins and that is that. > > in particular this allows unit tests to force a fake theme > > > NOTES: > - this approach means tests need to be explicitly adapted to this new thing > (e.g. https://paste.kde.org/pjzwg47s7) > - another approach would be to have a switch to disable qicon::theme > entirely, which has the advantage of not requiring additional changes to > other tests > - the presented approach in the long run (with possibly a bit more rigging in > kiconloader) allow a test to force all it's kiconthemes magic to use a given > theme without having to touch kdeglobals or running manually reconfigures etc. > - both viable, I like this one better though > - ::overrideCurrent would be another possible name but ::force sounds more > like what it ultimately does in terms of impact > > > Diffs > ----- > > autotests/CMakeLists.txt 7af101398774bd138fe57f3765416cfcfb022f75 > src/kicontheme.h dafece91f401ea0f8188613c55ee92d96d293879 > src/kicontheme.cpp f3d4cf71c05fe143464952bec89497fb911f2291 > > Diff: https://git.reviewboard.kde.org/r/127952/diff/ > > > Testing > ------- > > unit test + adapted plasma-framework's themetest to use this > > > Thanks, > > Harald Sitter > >
_______________________________________________ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel