D12784: CoreDirectoryTest

Johnny Jazeix noreply at phabricator.kde.org
Thu May 10 13:44:44 UTC 2018


jjazeix accepted this revision.
jjazeix added a comment.
This revision is now accepted and ready to land.


  Good for me, you can commit it on the master branch.
  
  Thank you

INLINE COMMENTS

> DirectoryTest.cpp:48
> +    file.mkpath("./dummy_directory");
> +    QCOMPARE(file.exists("./dummy_directory"), true);
> +

I think you can directly Q_COMPARE on the mkpath that returns true if the folder has been created

REPOSITORY
  R2 GCompris

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

To: himanshuvishwakarma, jjazeix
Cc: kde-edu, #gcompris, himanshuvishwakarma, harrymecwan, ganeshredcobra, nityanandkumar, echarruau, rahulyadav, narvaez, scagarwal, apol, timotheegiet, hkaelberer, jjazeix, bcoudoin
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-edu/attachments/20180510/7ee6d866/attachment.html>


More information about the kde-edu mailing list