Skip to content

Update listener.py #183

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

Merged
merged 7 commits into from
Dec 5, 2017
Merged

Update listener.py #183

merged 7 commits into from
Dec 5, 2017

Conversation

ProSerg
Copy link
Contributor

@ProSerg ProSerg commented Dec 5, 2017

added exception for getting finalizer. fixes #181.

Context

Checklist

added exception for getting finalizer allure-framework#181.
@CLAassistant
Copy link

CLAassistant commented Dec 5, 2017

CLA assistant check
All committers have signed the CLA.

for index, finalizer in enumerate(fixturedef._finalizer or ()):
name = '{fixture}::{finalizer}'.format(fixture=fixturedef.argname, finalizer=finalizer.__name__)
fixturedef._finalizer[index] = allure_commons.fixture(finalizer, parent_uuid=container_uuid, name=name)
try:
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

may be without code duplication and error handling? like this:
...
finalizers = fixturedef._finalizer if hasattr(fixturedef, 'finalizer') else fixturedef._finalizers
for index, finalizer in enumerate(finalizers or ()):
...

cleaned code duplication and error handling.
@ProSerg
Copy link
Contributor Author

ProSerg commented Dec 5, 2017

I would correct, but I can't understand what not so for py33.

@sseliverstov
Copy link
Contributor

Pytest no longer supports Python 2.6 and 3.3. Those Python versions are EOL for some time now and incur maintenance and compatibility costs on the pytest core team, and following up with the rest of the community we decided that they will no longer be supported starting on this version. Users which still require those versions should pin pytest to <3.3. (#2812)

@sseliverstov
Copy link
Contributor

needs some changes in tox.ini (

remove py33
remove py33
remove py33
remove py33
removed a blank line
@sseliverstov sseliverstov merged commit ae23da9 into allure-framework:master Dec 5, 2017
beckerGil pushed a commit to beckerGil/allure-python that referenced this pull request Apr 10, 2023
IvanBuruyane pushed a commit to IvanBuruyane/allure-python that referenced this pull request Mar 19, 2024
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.

allure-python 2.2.3b1 error
3 participants