cmd: fix tests that assume /snap mount #3644

Merged
merged 1 commit into from Aug 2, 2017

Conversation

Projects
None yet
3 participants
Contributor

zyga commented Aug 2, 2017

This patch fixes tests on Arch where snaps are mounted on
/var/lib/snapd/snap and tests are not mocking that aspect. The fix is
universal and just switches to the variable directly.

Signed-off-by: Zygmunt Krynicki me@zygoon.pl

cmd: fix tests that assume /snap mount
This patch fixes tests on Arch where snaps are mounted on
/var/lib/snapd/snap and tests are not mocking that aspect. The fix is
universal and just switches to the variable directly.

Signed-off-by: Zygmunt Krynicki <me@zygoon.pl>

Codecov Report

Merging #3644 into master will increase coverage by <.01%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #3644      +/-   ##
==========================================
+ Coverage    75.2%   75.21%   +<.01%     
==========================================
  Files         387      387              
  Lines       33452    33452              
==========================================
+ Hits        25159    25160       +1     
+ Misses       6481     6480       -1     
  Partials     1812     1812
Impacted Files Coverage Δ
cmd/snap/cmd_aliases.go 95% <0%> (+1.66%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update e037060...0c1d6a8. Read the comment docs.

@zyga zyga merged commit df56552 into snapcore:master Aug 2, 2017

5 of 7 checks passed

zesty-amd64 autopkgtest finished (error)
Details
yakkety-amd64 autopkgtest finished (failure)
Details
artful-amd64 autopkgtest finished (success)
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
xenial-amd64 autopkgtest finished (success)
Details
xenial-i386 autopkgtest finished (success)
Details
xenial-ppc64el autopkgtest finished (success)
Details

@zyga zyga deleted the zyga:fix/cmd-bugfix branch Aug 2, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment