Fix phpunit SIMPLETEST_BASE_URL / D12 core-tests failures in drupal-core and drupal-contrib

[rfay]

Summary

• Fixes Running tests on a ddev-drupal-dev based instance fails due to a missing env variable #211: drupal-core always omits ddev-router, so DDEV never populates DDEV_PRIMARY_URL. amateescu/ddev-drupal-dev's own config.drupal-dev.yaml sets SIMPLETEST_BASE_URL=${DDEV_PRIMARY_URL}, which is therefore always empty, failing every phpunit functional/JS test with "You must provide a SIMPLETEST_BASE_URL environment variable". Root-caused and reproduced live on staging, including the exact SettingSummariesContentTypeTest::testWorkflowSummary test from the issue.
• Overrides SIMPLETEST_BASE_URL to the web container's own Docker-network hostname (http://web), reachable regardless of routing, via a .ddev/config.zzz-coder-test-env.yaml file named to sort alphabetically after the add-on's own config file so it wins.
• Also points BROWSERTEST_OUTPUT_BASE_URL at the real Coder proxy URL instead of the container-only http://web, so failed-test debug links are actually clickable from outside the container.
• drupal-contrib was not affected — its add-on already hardcodes SIMPLETEST_BASE_URL=http://web. No fix needed there, but added CI coverage anyway (see below) since it's the same regression class.
• Adds a phpunit smoke-test step to both drupal-integration-test.yml and drupal-contrib-integration-test.yml's GH-runner jobs so this doesn't silently regress again.
• Filed Consolidate drupal-core/drupal-contrib routing onto freeform's router-based dispatch #213 to track the longer-term fix: switching drupal-core/drupal-contrib to the router-based Host-header dispatch freeform already uses, which avoids this whole class of bug rather than patching around it.
• Update: the new drupal-contrib CI step above caught a second, unrelated bug: the Contrib skipto D12 (plain, GH) cell failed CI with Cannot open bootstrap script core/tests/bootstrap.php. Root cause: drupal_version=12 installs drupal/core from the 12.x-dev branch (no stable D12 tag exists yet), and Composer's default dist install for a -dev version fetches a GitHub zipball — a git archive snapshot that honors drupal/core's own .gitattributes export-ignore rules, which strip core/tests entirely. D10/D11 are unaffected since they resolve to real tagged releases packaged differently. Fixed by forcing composer config preferred-install.drupal/core source (a real git clone, which doesn't apply export-ignore) for drupal_version=12 only. Verified end-to-end on a fresh workspace (see manual testing below). Filed drupal-contrib D12: core/tests missing because dev-branch dist zip strips it via .gitattributes export-ignore #216 for traceability.

Manual testing instructions

These were run against staging-coder.ddev.com before opening this PR; repeating them here for review/reproduction.

drupal-core

1. Push this branch as an inactive template version:

   coder templates push drupal-core --directory drupal-core --activate=false \
     --name pr-verify-core --yes \
     --variable image_version="$(cat VERSION)" \
     --variable workspace_image_registry=index.docker.io/ddev/coder-ddev
   

2. Create a workspace from it:

   coder create verify-core --template drupal-core --template-version pr-verify-core \
     --parameter install_profile=minimal --parameter drupal_version=12 \
     --use-parameter-defaults --yes
   

3. Wait for setup to finish: ssh verify-core.coder 'tail -f /tmp/drupal-setup.log' until you see the admin credentials block.
4. Confirm the env override took effect:

   ssh verify-core.coder 'cd ~/drupal-core && ddev exec printenv | grep -i "SIMPLETEST_BASE_URL\|BROWSERTEST_OUTPUT_BASE_URL"'
   

   Expect SIMPLETEST_BASE_URL=http://web and BROWSERTEST_OUTPUT_BASE_URL=https://drupal-site--verify-core--<you>.<coder-domain>.
5. Run a plain functional test:

   ssh verify-core.coder 'cd ~/drupal-core && ddev phpunit --filter testTitle core/modules/system/tests/src/Functional/System/PageTitleTest.php'
   

   Expect OK.
6. Run the exact JS test from the issue report:

   ssh verify-core.coder 'cd ~/drupal-core && ddev phpunit --filter testWorkflowSummary core/modules/node/tests/src/FunctionalJavascript/SettingSummariesContentTypeTest.php'
   

   Expect OK.
7. Clean up: coder delete verify-core --yes && coder templates versions archive drupal-core pr-verify-core --yes.

drupal-contrib (regression check — should already pass, confirming CI coverage doesn't break it)

1. Push this branch's drupal-contrib template (unchanged by this PR, but push it fresh so CI's new step is exercised against the current code):

   coder templates push drupal-contrib --directory drupal-contrib --activate=false \
     --name pr-verify-contrib --yes \
     --variable image_version="$(cat VERSION)" \
     --variable workspace_image_registry=index.docker.io/ddev/coder-ddev
   

2. Create a workspace for a small module (pathauto):

   coder create verify-contrib --template drupal-contrib --template-version pr-verify-contrib \
     --parameter project_name=pathauto --parameter project_type=module \
     --parameter drupal_version=11 --parameter install_profile=minimal \
     --use-parameter-defaults --yes
   

3. Wait for setup: ssh verify-contrib.coder 'tail -f /tmp/drupal-setup.log' until it finishes.
4. Confirm SIMPLETEST_BASE_URL is already correct (no change expected here):

   ssh verify-contrib.coder 'cd ~/pathauto && ddev exec printenv | grep -i SIMPLETEST_BASE_URL'
   

   Expect SIMPLETEST_BASE_URL=http://web.
5. Run the same project-agnostic core smoke test the new CI step uses:

   ssh verify-contrib.coder 'cd ~/pathauto && ddev phpunit --filter testTitle web/core/modules/system/tests/src/Functional/System/PageTitleTest.php'
   

   Expect OK.
6. Run a module-specific functional test for good measure:

   ssh verify-contrib.coder 'cd ~/pathauto && ddev phpunit tests/src/Functional/PathautoUserWebTest.php'
   

   Expect OK.
7. Clean up: coder delete verify-contrib --yes && coder templates versions archive drupal-contrib pr-verify-contrib --yes.

drupal-contrib D12 (the actual regression this PR fixes — #216)

1. Create a second workspace from the same pr-verify-contrib template version, this time for skipto on D12 (the exact cell that failed CI):

   coder create verify-contrib-d12 --template drupal-contrib --template-version pr-verify-contrib \
     --parameter project_name=skipto --parameter project_type=module \
     --parameter drupal_version=12 --parameter install_profile=minimal \
     --use-parameter-defaults --yes
   

2. Wait for setup: ssh verify-contrib-d12.coder 'tail -f /tmp/drupal-setup.log' until it finishes.
3. Confirm core/tests is present (previously missing before this fix):

   ssh verify-contrib-d12.coder 'cd ~/skipto && test -d web/core/tests && echo EXISTS || echo MISSING'
   

   Expect EXISTS.
4. Run the same core smoke test the CI step uses:

   ssh verify-contrib-d12.coder 'cd ~/skipto && ddev phpunit --filter testTitle web/core/modules/system/tests/src/Functional/System/PageTitleTest.php'
   

   Expect OK.
5. Clean up: coder delete verify-contrib-d12 --yes.

Test plan

• terraform fmt -recursive clean
• make validate passes for all templates
• make test-templates passes for all templates
• Manual verification above run against staging-coder.ddev.com for both templates (see PR description)
• New CI steps added to drupal-integration-test.yml and drupal-contrib-integration-test.yml will exercise this automatically going forward

🤖 Generated with Claude Code

[github-actions]

PR Preview Action v1.8.1
Preview removed because the pull request was closed.
2026-09-27 00:43 UTC