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

Change the behat command to use depending on whether ddev is available. #180

Merged
merged 1 commit into from
Jan 27, 2022

Conversation

becw
Copy link
Member

@becw becw commented Jan 7, 2022

Followup to #174; this does the same thing for behat as we did for drush, so that behat is run in an environment that can access the Drupal application directly.

To test...

Set up the codebase

composer create-project palantirnet/drupal-skeleton example dev-develop --no-interaction
cd example
composer require --dev palantirnet/the-build:dev-run-behat-inside-ddev
ddev start
vendor/bin/the-build-installer

Install Drupal at the prompt

When you're prompted to Install Drupal now (y,n)?, say y.

The site should be available at https://example.ddev.site/

Run the behat test command:

vendor/bin/phing behat

@becw becw requested a review from byrond January 7, 2022 19:57
@eric-schmidt eric-schmidt self-requested a review January 20, 2022 16:55
Copy link

@eric-schmidt eric-schmidt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Worked like a charm for me!

Copy link
Contributor

@byrond byrond left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Works for me as well!

@becw becw merged commit d917e82 into develop Jan 27, 2022
@becw becw deleted the run-behat-inside-ddev branch January 27, 2022 16:36
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.

3 participants