Skip to content
This repository has been archived by the owner on Oct 18, 2023. It is now read-only.

Add behat tests #67

Merged
merged 4 commits into from
Sep 6, 2018
Merged

Add behat tests #67

merged 4 commits into from
Sep 6, 2018

Conversation

miromichalicka
Copy link
Contributor

In this PR we're adding a couple of Behat tests for cart and checkout with a config file.

@malachy-mcconnell
Copy link
Contributor

This is spectacular.

@malachy-mcconnell
Copy link
Contributor

The travis fail is due to PHPCS. Do you want to add // phpcs:ignoreFile to these files? I get the feeling Behat testing using a different pattern to Drupal core.

But I'm ready to merge this. I'm going to test it now but on code review it looks great.

@csibar
Copy link

csibar commented Sep 6, 2018

Looks awesome!

@malachy-mcconnell malachy-mcconnell merged commit bbe63b4 into acquia:8.x-1.x Sep 6, 2018
@miromichalicka
Copy link
Contributor Author

🎉

@miromichalicka miromichalicka deleted the add-behat branch September 6, 2018 23:18
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.

3 participants