tests/main: enable a first set of test cases for Fedora and openSUSE #3411

Closed
wants to merge 50 commits into
from

Conversation

Projects
None yet
3 participants
Contributor

morphis commented May 30, 2017

This is currently based on #3365 and #3406 enables a first set of low hanging test cases. More will come in future PRs.

Simon Fels added some commits May 19, 2017

spread: disable all test suites for fedora
We will enable them step by step once necessary changes are introduces
to make each suite working.
tests/lib: use less options for user creation
Not all options we use by default are supported on other distributions
like Fedora or openSUSE so we use less there but keep the original set
for Ubuntu/Debian.
spread: extend PATH with snap directory in Fedora
/snap/bin is at /var/lib/snapd/snap/bin in Fedora so we have to add
that to PATH as well. Sadly we can't just do this on the system
level in spread.yaml as this overrides the globally set PATH variable.

Codecov Report

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

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #3411      +/-   ##
==========================================
- Coverage   77.52%   77.51%   -0.01%     
==========================================
  Files         369      369              
  Lines       25475    25475              
==========================================
- Hits        19750    19748       -2     
- Misses       3974     3976       +2     
  Partials     1751     1751
Impacted Files Coverage Δ
interfaces/sorting.go 90% <0%> (-3.34%) ⬇️
cmd/snap/cmd_aliases.go 94% <0%> (-2%) ⬇️

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 1e9a221...9afef11. Read the comment docs.

Simon Fels added some commits Jun 2, 2017

Contributor

zyga commented Jun 8, 2017

This needs to be merged/rebased on master

@morphis morphis closed this Jun 9, 2017

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