-
Notifications
You must be signed in to change notification settings - Fork 442
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
tests: add spread suite for plainbox plugin #2317
Conversation
reminder, amend the commit message |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice test suite, minor changes proposed.
execute: | | ||
cd "$SNAP_DIR" | ||
snapcraft | ||
sudo snap install checkbox-simple_*.snap --dangerous |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
sudo is not needed here
88452e0
to
21a5487
Compare
Codecov Report
@@ Coverage Diff @@
## master #2317 +/- ##
=========================================
Coverage ? 90.31%
=========================================
Files ? 213
Lines ? 13489
Branches ? 2008
=========================================
Hits ? 12183
Misses ? 894
Partials ? 412 Continue to review full report at Codecov.
|
1da7750
to
f52110e
Compare
Move the plugin and snapd integration tests for the plainbox plugin into a Spread suite. Also pin the checkbox-ng package in the tests checkbox snap. Also removed a test that looks more like a plainbox tests and not a snapcraft one. LP: #1783839 Signed-off-by: Sergio Schvezov <sergio.schvezov@canonical.com>
f52110e
to
1fbfb6d
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good job! Approving
Move the plugin and snapd integration tests for the plainbox plugin into a Spread suite. Also pin the checkbox-ng package in the tests checkbox snap. Also removed a test that looks more like a plainbox tests and not a snapcraft one. LP: #1783839 Signed-off-by: Sergio Schvezov <sergio.schvezov@canonical.com>
Move the plugin and snapd integration tests for the plainbox plugin into a Spread suite. Also pin the checkbox-ng package in the tests checkbox snap. Also removed a test that looks more like a plainbox tests and not a snapcraft one. LP: #1783839 Signed-off-by: Sergio Schvezov <sergio.schvezov@canonical.com>
Move the plugin and snapd integration tests for the CMake plugin into a
Spread suite.
Also pin the checkbox-ng package in the tests checkbox snap.
LP: #1783839
Signed-off-by: Sergio Schvezov sergio.schvezov@canonical.com
./runtests.sh static
?./runtests.sh tests/unit
?