D16349: [kdirlistertest] Wait a little longer for the lister to finish

2018-10-22 Thread Jaime Torres Amate
This revision was automatically updated to reflect the committed changes.
Closed by commit R241:87a2f084be83: [kdirlistertest] Wait a little longer for 
the lister to finish (authored by jtamate).

REPOSITORY
  R241 KIO

CHANGES SINCE LAST UPDATE
  https://phabricator.kde.org/D16349?vs=44011=44081

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

AFFECTED FILES
  autotests/kdirlistertest.cpp

To: jtamate, dfaure, #frameworks
Cc: kde-frameworks-devel, michaelh, ngraham, bruns


D16349: [kdirlistertest] Wait a little longer for the lister to finish

2018-10-22 Thread David Faure
dfaure accepted this revision.
dfaure added a comment.
This revision is now accepted and ready to land.


  Thanks!!

REPOSITORY
  R241 KIO

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

To: jtamate, dfaure, #frameworks
Cc: kde-frameworks-devel, michaelh, ngraham, bruns


D16349: [kdirlistertest] Wait a little longer for the lister to finish

2018-10-21 Thread Jaime Torres Amate
jtamate created this revision.
jtamate added reviewers: dfaure, Frameworks.
Herald added a project: Frameworks.
Herald added a subscriber: kde-frameworks-devel.
jtamate requested review of this revision.

REVISION SUMMARY
  Wait a little longer and let m_dirLister finish before checking it has 
finished.

TEST PLAN
  Before:
  FAIL!  : KDirListerTest::testConcurrentListingAndStop() Compared values are 
not the same
  
Actual   (m_dirLister.spyCompleted.count()): 0
Expected (1)   : 1
Loc: [/home/jenkins/workspace/Frameworks/kio/kf5-qt5 
SUSEQt5.10/autotests/kdirlistertest.cpp(798)]
  
  After (at home):
   Totals: 32 passed, 0 failed, 0 skipped, 0 blacklisted

REPOSITORY
  R241 KIO

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

AFFECTED FILES
  autotests/kdirlistertest.cpp

To: jtamate, dfaure, #frameworks
Cc: kde-frameworks-devel, michaelh, ngraham, bruns