Skip to content
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

Add skip_on_spawning_platform and skip_unless_on_spawning_platform #81

Merged
merged 2 commits into from
Sep 14, 2021

Conversation

s0undt3ch
Copy link
Contributor

No description provided.

@codecov
Copy link

codecov bot commented Sep 14, 2021

Codecov Report

Merging #81 (2858eec) into master (47d53ff) will decrease coverage by 1.10%.
The diff coverage is 80.56%.

❗ Current head 2858eec differs from pull request most recent head 2709569. Consider uploading reports for the commit 2709569 to get more accurate results
Impacted file tree graph

@@            Coverage Diff             @@
##           master      #81      +/-   ##
==========================================
- Coverage   88.49%   87.39%   -1.09%     
==========================================
  Files         135      137       +2     
  Lines        7371     7443      +72     
  Branches      854      865      +11     
==========================================
- Hits         6522     6504      -18     
- Misses        623      682      +59     
- Partials      226      257      +31     
Flag Coverage Δ
Linux 87.01% <80.56%> (-0.41%) ⬇️
Py35 64.31% <19.45%> (-0.43%) ⬇️
Py36 85.04% <80.56%> (-0.33%) ⬇️
Py37 64.40% <19.45%> (-21.45%) ⬇️
Py38 44.84% <19.45%> (-42.20%) ⬇️
Py39 ?
PyTest60 64.43% <19.45%> (-1.95%) ⬇️
PyTest61 64.35% <19.45%> (-2.03%) ⬇️
PyTest62 86.91% <80.56%> (-0.87%) ⬇️
Salt3002_6 85.09% <80.56%> (-1.33%) ⬇️
Salt3003 64.33% <19.45%> (-22.09%) ⬇️
Salt3003.3 44.84% <19.45%> (-0.25%) ⬇️
Windows ?
macOS ?
src 80.08% <58.83%> (-1.30%) ⬇️
system 44.84% <19.45%> (-0.25%) ⬇️
tests 96.00% <100.00%> (-0.35%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
src/saltfactories/utils/platform.py 89.62% <25.00%> (-7.49%) ⬇️
src/saltfactories/utils/markers.py 74.53% <66.67%> (-1.31%) ⬇️
src/saltfactories/plugins/markers.py 100.00% <100.00%> (ø)
...nctional/markers/test_skip_on_spawning_platform.py 100.00% <100.00%> (ø)
...l/markers/test_skip_unless_on_spawning_platform.py 100.00% <100.00%> (ø)
tests/unit/utils/markers/test_skip_if_not_root.py 60.00% <0.00%> (-40.00%) ⬇️
src/saltfactories/utils/processes.py 68.43% <0.00%> (-21.05%) ⬇️
tests/functional/markers/test_skip_if_not_root.py 84.62% <0.00%> (-15.38%) ⬇️
src/saltfactories/plugins/log_server.py 92.31% <0.00%> (-5.98%) ⬇️
src/saltfactories/utils/ports.py 88.24% <0.00%> (-5.88%) ⬇️
... and 14 more

@s0undt3ch s0undt3ch merged commit 615e49e into saltstack:master Sep 14, 2021
s0undt3ch added a commit that referenced this pull request Sep 15, 2021
s0undt3ch added a commit to s0undt3ch/pytest-salt-factories that referenced this pull request Sep 15, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant