[Differential] [Accepted] D3707: add simple test rig for service runner

dfaure (David Faure) noreply at phabricator.kde.org
Thu Dec 22 21:03:01 UTC 2016


dfaure accepted this revision.
dfaure added inline comments.
This revision is now accepted and ready to land.

INLINE COMMENTS

> servicerunnertest.cpp:49
> +    QDir(appsPath).removeRecursively();
> +    Q_ASSERT(QDir().mkpath(appsPath));
> +    auto fixtureDir = QDir(QFINDTESTDATA("fixtures"));

Use QVERIFY rather than Q_ASSERT in unittests.

(better integrated with the framework, and works even in a release build)

(replace all Q_ASSERTs in this file, there are others)

> servicerunnertest.cpp:54
> +        auto target = appsPath + QDir::separator() + fileInfo.fileName();
> +        Q_ASSERT_X(QFile::copy(fileInfo.absoluteFilePath(), target),
> +                   "copy",

QVERIFY2

REPOSITORY
  R120 Plasma Workspace

BRANCH
  master

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

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

To: sitter, broulik, dfaure
Cc: dfaure, plasma-devel, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20161222/1dac4410/attachment.html>


More information about the Plasma-devel mailing list