Skip to content

Commit 1d967a3

Browse files
committed
Install purge in CI, so the kernel tests can actually run.
Wiring the phpunit job was verified by running the unit tests inside the CI image locally. That missed the kernel tests, which install quant_purger and therefore need drupal/purge. CI only installed drupal/token, so all 14 purger kernel tests errored with "Unavailable module: 'purge'" on every push since the job was added. The suite passes locally because the harness has purge installed for the end-to-end work. Verifying a CI change by running part of it locally was not verifying it; the run on the pull request was.
1 parent 6770485 commit 1d967a3

1 file changed

Lines changed: 3 additions & 1 deletion

File tree

.github/workflows/ci.yml

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -35,8 +35,10 @@ jobs:
3535
run: drush si --db-url=mysql://root:drupal@mariadb:3306/drupal11 -y
3636
working-directory: /var/www/drupal
3737

38+
# token is required by quant_search. purge is required by quant_purger,
39+
# whose kernel tests install it, so the suite cannot run without it.
3840
- name: Install module dependencies
39-
run: composer --no-interaction --no-progress require drupal/token
41+
run: composer --no-interaction --no-progress require drupal/token drupal/purge
4042
working-directory: /var/www/drupal
4143

4244
# The image ships phpunit globally, but its bootstrap needs the test

0 commit comments

Comments
 (0)