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

CONTRIB-6619 behat: make them pass for 32 and up #38

Merged
merged 1 commit into from Dec 9, 2016

Conversation

stronk7
Copy link
Member

@stronk7 stronk7 commented Dec 9, 2016

Finally it has been done by:

  1. Using the new steps (MDL-56614) for 31 and up.
  2. Disable running behats for 30 and down (fair enough,
    no UI changes are expected there and they have been
    working ok. Plus all them are under security-only support
    so...

@stronk7
Copy link
Member Author

stronk7 commented Dec 9, 2016

(note this is WIP / testing travis changes, yet)

Finally it has been done by:

1) Using the new steps (MDL-56614) for 31 and up.
2) Disable running behats for 30 and down (fair enough,
   no UI changes are expected there and they have been
   working ok. Plus all them are under security-only support
   so...
@stronk7
Copy link
Member Author

stronk7 commented Dec 9, 2016

I'd consider this ready for integration:

  1. Ignoring the "weird" travis timeout bug.
  2. 31, 32 and master are running behat tests.
  3. 31 and older are not.

@danpoltawski danpoltawski merged commit 355b5fe into moodlehq:master Dec 9, 2016
@danpoltawski
Copy link
Contributor

Merged thanks

@stronk7 stronk7 deleted the CONTRIB-6619 branch December 9, 2016 12:20
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.

None yet

2 participants