Changeset 53349 for trunk/tests/phpunit/includes/abstract-testcase.php

Timestamp:

05/04/2022 12:38:18 AM (3 years ago)

Author:

SergeyBiryukov

Message:

Build/Test Tools: Pass GitHub Actions environment variables to the Docker container.

This ensures that WP_UnitTestCase::skipOnAutomatedBranches() has access to the GITHUB_REF and GITHUB_EVENT_NAME variables, so that some tests can be skipped when appropriate.

Additionally, account for renaming the master branch to trunk in November 2021.

Follow-up to [40241], [46999], [49264], [49267], [51868].

Fixes #55668.

File:

• trunk/tests/phpunit/includes/abstract-testcase.php (modified) (2 diffs)

trunk/tests/phpunit/includes/abstract-testcase.php

@@ -203 +203 @@
      * Allows tests to be skipped on some automated runs.
      *
-     * For test runs on GitHub Actions for something other than trunk/master,
-     * we want to skip tests that only need to run for master.
+     * For test runs on GitHub Actions for something other than trunk,
+     * we want to skip tests that only need to run for trunk.
      */
     public function skipOnAutomatedBranches() {
@@ -211 +211 @@
         $github_ref        = getenv( 'GITHUB_REF' );
 
-        if ( $github_event_name && 'false' !== $github_event_name ) {
+        if ( $github_event_name ) {
             // We're on GitHub Actions.
             $skipped = array( 'pull_request', 'pull_request_target' );
 
-            if ( in_array( $github_event_name, $skipped, true ) || 'refs/heads/master' !== $github_ref ) {
-                $this->markTestSkipped( 'For automated test runs, this test is only run on trunk/master' );
+            if ( in_array( $github_event_name, $skipped, true ) || 'refs/heads/trunk' !== $github_ref ) {
+                $this->markTestSkipped( 'For automated test runs, this test is only run on trunk' );
             }
         }