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

Fix simpletest usage for Drupal 8 #322

Merged
merged 4 commits into from
Jun 18, 2020

Conversation

kierenevans
Copy link
Contributor

@kierenevans kierenevans commented Jun 16, 2020

The container running the website (php-fpm) needs to be the container
running the simpletest phpunit run, as it does calls with basic auth
that let the sqlite database be used instead of MySQL.

Without this, drupal is detecting the lack of a sessions table in the php-fpm sqlite database (despite it present in the console sqlite database) and redirecting to the installer.

The container running the website (php-fpm) needs to be the container
running the simpletest phpunit run, as it does calls with basic auth
that let the sqlite database be used instead of MySQL.
@kierenevans kierenevans added enhancement New feature or request harness-drupal8 Drupal 8.x harness, obsolete as of 1.5.x labels Jun 16, 2020
@kierenevans kierenevans added this to the 0.8.0 milestone Jun 16, 2020
stage('Test') {
parallel {
stage('quality') { steps { sh 'ws exec composer test-quality' } }
stage('unit') { steps { sh 'ws test-unit' } }
Copy link
Contributor Author

Choose a reason for hiding this comment

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

This is the difference to _base version

@kierenevans kierenevans merged commit 9002deb into 0.8.x Jun 18, 2020
@kierenevans kierenevans deleted the feature/fix-simpletest-usage-for-drupal8 branch June 18, 2020 07:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request harness-drupal8 Drupal 8.x harness, obsolete as of 1.5.x
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants