Review Request 129519: Improve autotests for kpackage
Bhushan Shah
bhush94 at gmail.com
Mon Nov 21 12:57:02 UTC 2016
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/129519/
-----------------------------------------------------------
(Updated Nov. 21, 2016, 6:27 p.m.)
Status
------
This change has been marked as submitted.
Review request for Plasma, Aleix Pol Gonzalez and Marco Martin.
Repository: kpackage
Description
-------
There are multiple changes in this,
a) Previously it used to install all packages in one go and then just verify number. now this tries to do it step by step.
b) Added testcase for invalid metadata
c) This tries to install the package with dependencies specified, which as expected would fail. One would need to mock dep resolver for testcases. (I will address this in next patch)
Diffs
-----
autotests/data/testinvalidmetadata/contents/ui/main.qml PRE-CREATION
autotests/data/testinvalidmetadata/metadata.json PRE-CREATION
autotests/data/testpackagesdep/metadata.json a634d9d
autotests/querytest.h 6628eff
autotests/querytest.cpp 7765c7f
Diff: https://git.reviewboard.kde.org/r/129519/diff/
Testing
-------
make test works, and coverage is also bit up now.
Thanks,
Bhushan Shah
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20161121/f3de9347/attachment-0001.html>
More information about the Plasma-devel
mailing list