[Differential] [Request, 1 line] D2837: Cleanup test case properly

davidedmundson (David Edmundson) noreply at phabricator.kde.org
Thu Sep 22 16:42:42 UTC 2016


davidedmundson created this revision.
davidedmundson added a reviewer: Plasma.
Restricted Application added a project: Plasma.
Restricted Application added a subscriber: plasma-devel.

REVISION SUMMARY
  Otherwise if a test fails it would leave a dangling folder which causes
  subsequent tests to fail, which in turn means it doesn't clean up
  properly.

TEST PLAN
  Broke my test
  Tests then always failed leaving stray dir
  
  Applied patch
  Ran twice
  Second time now passes again
  
  Possibly fix CI

REPOSITORY
  rPLASMASDK Plasma SDK

BRANCH
  master

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

AFFECTED FILES
  plasmate/app/tests/projecthandlertest.cpp

EMAIL PREFERENCES
  https://phabricator.kde.org/settings/panel/emailpreferences/

To: davidedmundson, #plasma
Cc: plasma-devel, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20160922/badecd6c/attachment.html>


More information about the Plasma-devel mailing list