Skip to content
This repository has been archived by the owner on Aug 20, 2018. It is now read-only.

Pytest selenium 1.2.0 #329

Closed
wants to merge 4 commits into from
Closed

Pytest selenium 1.2.0 #329

wants to merge 4 commits into from

Conversation

stephendonner
Copy link
Contributor

r? @bobsilverberg @davehunt - I've done an ad-hoc job, which passes: http://webqa-ci-staging1.qa.scl3.mozilla.com:8080/job/oneanddone.adhoc/66/console

I'm not confident nor happy about downgrading pytest, just to get this to pass (see http://webqa-ci-staging1.qa.scl3.mozilla.com:8080/job/oneanddone.adhoc/65/ for failures with pytest 2.8.5).

I'm really out of my realm here, sorry; still getting up-to-speed, and thought this would be easy!

Feel free to take over completely, as well!

@davehunt
Copy link
Member

Yeah, I discovered this today with bedrock. The issue is pytest-dev/pytest#1062 which is fixed in the upcoming pytest 2.9. I'm happy with pinning pytest to 2.7.3 for now or pinning pytest-selenium to 1.1. I guess it's a good thing we're not on 2.8 elsewhere. 😄

pytest==2.8.5
pytest-selenium
pytest==2.7.3
pytest-selenium>=1.2.0rc1
Copy link
Member

Choose a reason for hiding this comment

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

No need for the >=1.2.0rc1 now as 1.2.1 is released and will be picked up.

@davehunt
Copy link
Member

r+ wc

@davehunt
Copy link
Member

Replaced by #330

@davehunt davehunt closed this Feb 26, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants