D21249: Test current filter before setting a new one

[David Faure]

dfaure requested changes to this revision.
dfaure added a comment.
This revision now requires changes to proceed.


  Thanks!
  
  One minor issue: unittests (those based on QTEST_MAIN like yours) go into the autotests directory.
  tests/ is for interactive test programs.
  Can you move it there?
  It could even be just a new method in the existing autotests/kfilewidgettest.cpp
  [if you do keep it separate for some reason, add a copyright header, but my recommendation is to merge it anyway, to keep the usual structure of one test file per class being tested]

REPOSITORY
  R241 KIO

REVISION DETAIL
  https://phabricator.kde.org/D21249

To: jglogowski, #frameworks, ngraham, dfaure
Cc: elvisangelaccio, ngraham, michaelweghorn, kde-frameworks-devel, michaelh, bruns
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20190520/4cb2c111/attachment-0001.html>