Differential D16349

[kdirlistertest] Wait a little longer for the lister to finish
ClosedPublic
Actions

Authored by jtamate on Oct 21 2018, 9:55 AM.

Details

Reviewers
dfaure
Group Reviewers
Frameworks
Commits
R241:87a2f084be83: [kdirlistertest] Wait a little longer for the lister to finish
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

Diff Detail

Repository
R241 KIO
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
jtamate created this revision.Oct 21 2018, 9:55 AM
Restricted Application added a project: Frameworks. · View Herald TranscriptOct 21 2018, 9:55 AM
Restricted Application added a subscriber: kde-frameworks-devel. · View Herald Transcript
jtamate requested review of this revision.Oct 21 2018, 9:55 AM
dfaure accepted this revision.Oct 22 2018, 8:11 AM

Thanks!!

This revision is now accepted and ready to land.Oct 22 2018, 8:11 AM
Closed by commit R241:87a2f084be83: [kdirlistertest] Wait a little longer for the lister to finish (authored by jtamate). · Explain WhyOct 22 2018, 5:20 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathPackages
Mautotests/kdirlistertest.cpp (1 line)

Diff 44081

View Options

autotests/kdirlistertest.cpp

Loading...