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 --junit-suite-name CLI option #2273

Closed
wants to merge 38 commits into from
Closed

Add --junit-suite-name CLI option #2273

wants to merge 38 commits into from

Conversation

dimp-gh
Copy link
Contributor

@dimp-gh dimp-gh commented Feb 22, 2017

It can be used to control <testsuite> 'name' attribute. Resolves #533.

barneygale and others added 30 commits January 2, 2017 22:01
Replace 'raise StopIteration' usages in the code by 'return's in accordance to PEP-479
…lobals

'xfail' markers without a condition no longer rely on the underlying `Item` objects deriving from `PyobjMixin`
Do not asssume `Item.obj` in 'skipping' plugin
KeyboardInterrupt is a subclass of BaseException, but not of Exception.
Hence if we remove this except, KeyboardInterrupts will still be raised
so the behavior stays the same.
Descriptors (e.g. properties) such as in the added test case are
triggered during collection, executing arbitrary code which can raise.
Previously, such exceptions were propagated and failed the collection.
Now these exceptions are caught and the corresponding attributes are
silently ignored.

A better solution would be to completely skip access to all custom
descriptors, such that the offending code doesn't even trigger. However
I think this requires manually going through the instance and all of its
MRO for each and every attribute checking if it might be a proper
fixture before accessing it. So I took the easy route here.

In other words, putting something like this in your test class is still
a bad idea...:

    @Property
    def innocent(self):
        os.system('rm -rf /')

Fixes #2234.
ignore property errors when parsing fixure factories
…active

Allow to skip unittests if --pdb active
Mention next training event.
--override-ini now correctly overrides some fundamental options like "python_files"
Python 3.6 invalid escape sequence deprecation fixes
vmuriart and others added 7 commits February 16, 2017 22:52
Document pytest_report_header and conftest behavior
Simplify travis.yml with tox environment variables
In the xml report we now have two occurences for the system-out tag if
the testcase writes to stdout both on call and teardown and fails in
teardown.
This behaviour is against the xsd.
This patch makes sure that the system-out section exists only
once per testcase.
junitxml: Fix double system-out tags per testcase
@dimp-gh dimp-gh changed the base branch from master to features February 22, 2017 20:00
@coveralls
Copy link

Coverage Status

Coverage decreased (-0.02%) to 92.71% when pulling 6db84b5 on dmand:feat/junitxml/suite-name into 44ad369 on pytest-dev:features.

@dimp-gh
Copy link
Contributor Author

dimp-gh commented Feb 22, 2017

Looks like i kind of messed up with master/feature base branches. I'll try to reopen it.

@dimp-gh dimp-gh closed this Feb 22, 2017
@coveralls
Copy link

Coverage Status

Coverage decreased (-0.1%) to 92.608% when pulling 31b7632 on dmand:feat/junitxml/suite-name into 44ad369 on pytest-dev:features.

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.