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

Report "pytestmark = skip" only once #2549

Closed
blueyed opened this issue Jul 4, 2017 · 1 comment
Closed

Report "pytestmark = skip" only once #2549

blueyed opened this issue Jul 4, 2017 · 1 comment
Labels
status: help wanted developers would like help from experts on this topic topic: reporting related to terminal output and user-facing messages and errors type: enhancement new feature or API change, should be merged into features branch type: feature-branch new feature or API change, should be merged into features branch

Comments

@blueyed
Copy link
Contributor

blueyed commented Jul 4, 2017

When using pytestmark = pytest.mark.skip("reason") on a module you will get a SKIP report for every test in that module (in the short test summary info).

There happens some folding already, which could be extended:

pytest/_pytest/skipping.py

Lines 351 to 360 in 6908d93

def folded_skips(skipped):
d = {}
for event in skipped:
key = event.longrepr
assert len(key) == 3, (event, key)
d.setdefault(key, []).append(event)
l = []
for key, events in d.items():
l.append((len(events),) + key)
return l

@blueyed blueyed added the type: enhancement new feature or API change, should be merged into features branch label Jul 4, 2017
@RonnyPfannschmidt RonnyPfannschmidt added type: feature-branch new feature or API change, should be merged into features branch status: help wanted developers would like help from experts on this topic labels Jul 4, 2017
@nicoddemus nicoddemus added the topic: reporting related to terminal output and user-facing messages and errors label Jul 5, 2017
@nicoddemus nicoddemus added type: enhancement new feature or API change, should be merged into features branch and removed type: enhancement new feature or API change, should be merged into features branch labels Sep 14, 2017
@bilderbuchi
Copy link
Contributor

@nicoddemus @RonnyPfannschmidt as the PR is merged this can be closed I think.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
status: help wanted developers would like help from experts on this topic topic: reporting related to terminal output and user-facing messages and errors type: enhancement new feature or API change, should be merged into features branch type: feature-branch new feature or API change, should be merged into features branch
Projects
None yet
Development

No branches or pull requests

4 participants