D26878: Add missing renamed event when a destination file already existed

David Faure noreply at phabricator.kde.org
Fri Jan 24 17:51:19 GMT 2020


dfaure accepted this revision.
dfaure added a comment.


  Nice work! Thanks for the unittest extension.

INLINE COMMENTS

> jobtest.cpp:1731
> +
> +    qDebug() << sourceFile << renamedFile;
> +    QVERIFY2(job->exec(), qPrintable(job->errorString()));

Please clean up before committing

> jobtest.cpp:1740
> +
> +    QFile(sourceFile).remove();
> +    QDir(dest).removeRecursively();

QVERIFY(...)

> jobtest.cpp:1741
> +    QFile(sourceFile).remove();
> +    QDir(dest).removeRecursively();
> +}

QVERIFY(...)

REPOSITORY
  R241 KIO

BRANCH
  arcpatch-D26878

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

To: meven, #frameworks, ervin, dfaure
Cc: kde-frameworks-devel, LeGast00n, GB_2, michaelh, ngraham, bruns
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20200124/bdae031c/attachment-0001.html>


More information about the Kde-frameworks-devel mailing list