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

Explain better how we do automated testing in prestashop and introduce behat tests #333

Merged
merged 6 commits into from Sep 20, 2019

Conversation

@matks
Copy link
Contributor

commented Sep 4, 2019

No description provided.

@matks matks changed the title Explain better how we do automated testing in prestashop Explain better how we do automated testing in prestashop and introduce behat tests Sep 4, 2019
@matks

This comment has been minimized.

Copy link
Contributor Author

commented Sep 4, 2019

Not finished 🙄my network connection failed me this evening

@matks matks force-pushed the matks:explain-better-testing branch from 3193f16 to 742af70 Sep 9, 2019
@matks matks force-pushed the matks:explain-better-testing branch 2 times, most recently from a149c52 to 0b09338 Sep 12, 2019
@matks matks force-pushed the matks:explain-better-testing branch from 0b09338 to 9bf8500 Sep 12, 2019
@matks

This comment has been minimized.

Copy link
Contributor Author

commented Sep 12, 2019

Applied changes suggested by @PierreRambaud and added a step-by-step tutorial

Only missing things, but it can be an extra for later: explain how the database is reset between each feature, explain internal workings of steps (how data are wiped out after a scenario, how we use Shared storage ...)

matks and others added 2 commits Sep 13, 2019
Co-Authored-By: Šarūnas Jonušas <jonusas.sarunas@gmail.com>
@matks

This comment has been minimized.

Copy link
Contributor Author

commented Sep 20, 2019

Thank you @sarjon and @PierreRambaud for the review, everything has been fixed

I think I can safely merge

@matks matks merged commit 517dfd9 into PrestaShop:master Sep 20, 2019
1 check passed
1 check passed
Travis CI - Pull Request Build Passed
Details
@matks matks deleted the matks:explain-better-testing branch Sep 20, 2019
@matks matks referenced this pull request Oct 22, 2019
3 of 5 tasks complete
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants
You can’t perform that action at this time.