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

!!!TASK: Upgrade Behat #14

Merged
merged 12 commits into from
Jan 28, 2019
Merged

!!!TASK: Upgrade Behat #14

merged 12 commits into from
Jan 28, 2019

Conversation

johannessteu
Copy link
Contributor

This upgrades Behat from 2.4 to the latest 3.5.x. This change is breaking as behat changed some things under the hood regarding contexts. Therefore the old FlowContext was refactored to a FlowContextTrait wich can be included in FeatureContexts

@baschny baschny mentioned this pull request Jan 15, 2019
@johannessteu
Copy link
Contributor Author

@johannessteu
Copy link
Contributor Author

Adjust travis scripts

TASK: test kickstart of behat config

TASK: Fix travis ci scripts

TASK: Implement correct Context in dummy files

Adjust default behat configuration file

TASK: Upgrade behat dependenices

TASK: Make sure to remove old behat config from dsitri collection when testing

Debug

TASK: Don’t link package in ci as the includes paths won’t match

Install before delete ...

Come on travis ...

Travis

Travis
@johannessteu
Copy link
Contributor Author

I added a travis-ci.yml to check the kickstart command and actually execute those behat tests:

https://travis-ci.com/johannessteu/behat

@bwaidelich bwaidelich self-assigned this Jan 28, 2019
@bwaidelich
Copy link
Member

Thanks for the contribution!
I merge this one, but please don't release a new version yet as I found some other gremlins that I'll try to zap with a follow up

@bwaidelich bwaidelich merged commit 87faef6 into neos:master Jan 28, 2019
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.

4 participants