Changeset 63916

Timestamp:

09/24/2026 03:58:03 PM (3 hours ago)

Author:

lancewillett

Message:

Build/Test Tools: Allow dotted groups in PHPUnit v1 and v2.

Allow periods in requested PHPUnit group names so callers can select groups such as functions.php. Continue rejecting shell syntax that legacy Docker wrappers could interpret as commands.

Follow-up to [63897].

Developed in: ​https://github.com/WordPress/wordpress-develop/pull/13711

Props lucatume.
See #66149.

Location:

trunk/.github/workflows

Files:

• reusable-phpunit-tests-v1.yml (modified) (1 diff)
• reusable-phpunit-tests-v2.yml (modified) (1 diff)

trunk/.github/workflows/reusable-phpunit-tests-v1.yml

@@ -126 +126 @@
           TEST_GROUPS: ${{ inputs.phpunit-test-groups }}
         run: |
-          if [[ ! "$TEST_GROUPS" =~ ^[A-Za-z0-9_-]+(,[A-Za-z0-9_-]+)*$ ]]; then
+          if [[ ! "$TEST_GROUPS" =~ ^[A-Za-z0-9_.-]+(,[A-Za-z0-9_.-]+)*$ ]]; then
             echo 'phpunit-test-groups must be a comma-separated list of group names.' >&2
             exit 1

trunk/.github/workflows/reusable-phpunit-tests-v2.yml

@@ -128 +128 @@
           TEST_GROUPS: ${{ inputs.phpunit-test-groups }}
         run: |
-          if [[ ! "$TEST_GROUPS" =~ ^[A-Za-z0-9_-]+(,[A-Za-z0-9_-]+)*$ ]]; then
+          if [[ ! "$TEST_GROUPS" =~ ^[A-Za-z0-9_.-]+(,[A-Za-z0-9_.-]+)*$ ]]; then
             echo 'phpunit-test-groups must be a comma-separated list of group names.' >&2
             exit 1