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

Polish Behat DataContext file to improve performance #734

Merged
merged 1 commit into from
Jan 2, 2014

Conversation

stloyd
Copy link
Contributor

@stloyd stloyd commented Dec 20, 2013

Speed gain is at least 1m (or more, it depends on machine you use + php)

@ghost ghost assigned stloyd Dec 20, 2013
pjedrzejewski pushed a commit that referenced this pull request Jan 2, 2014
Polish Behat DataContext file to improve performance
@pjedrzejewski pjedrzejewski merged commit 0f7ad1b into Sylius:master Jan 2, 2014
@stloyd stloyd deleted the bugfix/behat_datacontext branch January 2, 2014 11:30
@pjedrzejewski
Copy link
Member

Thanks Joseph! In perfect scenario I'd like to refactor the contexts using PageObjectExtension (see Akeneo), BUT... Behat V3 is coming and not sure how to proceed... :)

@jjanvier
Copy link
Contributor

jjanvier commented Jan 3, 2014

@pjedrzejewski Agree we should take a look at Behat v3. We wanted to refactor all the tests with @Arn0d, but as Behat v3 was coming, we waited.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Enhancement Minor issues and PRs improving the current solutions (optimizations, typo fixes, etc.). UX Issues and PRs aimed at improving User eXperience.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants