Update @wordpress/env to 8.1.1 and re-enable PHP Unit Tests for PHP v…

… 8.1 and 8.2 (#9875) Co-authored-by: Manish Menaria <[email protected]> Co-authored-by: Alexandre Lara <[email protected]> Co-authored-by: Luigi Teschio <[email protected]> Co-authored-by: Alba Rincón <[email protected]> Co-authored-by: Alba Rincón <[email protected]> Co-authored-by: Tung Du <[email protected]>
woocommerce · Sep 1, 2023 · 0b44765 · 0b44765
1 parent aed4423
commit 0b44765
Show file tree

Hide file tree

Showing 11 changed files with 4,751 additions and 11,005 deletions.
diff --git a/.github/workflows/unit-tests.yml b/.github/workflows/unit-tests.yml
@@ -80,38 +80,28 @@ jobs:
       ##
       - name: Set up PHP
         uses: shivammathur/setup-php@v2
-        # 8.1 and 8.2 are temporarily disabled as docker-images for those cause the errors when pipeline is running
-        if: ${{ matrix.php == '7.4' || matrix.php == '8.0' }}
         with:
           php-version: '${{ matrix.php }}'
           ini-file: development
           coverage: none
 
       # Ensure that Composer installs the correct versions of packages.
       - name: Override PHP version in composer.json
-        # 8.1 and 8.2 are temporarily disabled as docker-images for those cause the errors when pipeline is running
-        if: ${{ matrix.php == '7.4' || matrix.php == '8.0' }}
         run: |
           composer config platform.php ${{ matrix.php }}
           composer update
 
       - name: Install npm dependencies
-        # 8.1 and 8.2 are temporarily disabled as docker-images for those cause the errors when pipeline is running
-        if: ${{ matrix.php == '7.4' || matrix.php == '8.0' }}
         run: |
           npm ci
           npm run build
           
       - name: Docker debug information
-        # 8.1 and 8.2 are temporarily disabled as docker-images for those cause the errors when pipeline is running
-        if: ${{ matrix.php == '7.4' || matrix.php == '8.0' }}
         run: |
           docker -v
           docker-compose -v
           
       - name: General debug information
-        # 8.1 and 8.2 are temporarily disabled as docker-images for those cause the errors when pipeline is running
-        if: ${{ matrix.php == '7.4' || matrix.php == '8.0' }}
         run: |
           npm --version
           node --version
@@ -121,26 +111,18 @@ jobs:
           locale -a
           
       - name: Start Docker environment
-        # 8.1 and 8.2 are temporarily disabled as docker-images for those cause the errors when pipeline is running
-        if: ${{ matrix.php == '7.4' || matrix.php == '8.0' }}
         run: npm run wp-env start --update
 
       - name: Log running Docker containers
-        # 8.1 and 8.2 are temporarily disabled as docker-images for those cause the errors when pipeline is running
-        if: ${{ matrix.php == '7.4' || matrix.php == '8.0' }}
         run: docker ps -a
 
       - name: Docker container debug information
-        # 8.1 and 8.2 are temporarily disabled as docker-images for those cause the errors when pipeline is running
-        if: ${{ matrix.php == '7.4' || matrix.php == '8.0' }}
         run: |
           npm run wp-env run tests-mysql mysql -- --version
-          npm run wp-env run tests-wordpress "php --version"
-          npm run wp-env run tests-wordpress "php -m"
-          npm run wp-env run tests-wordpress "php -i"
-          npm run wp-env run tests-wordpress "locale -a"
+          npm run wp-env run tests-wordpress php -- --version
+          npm run wp-env run tests-wordpress php -- -m
+          npm run wp-env run tests-wordpress php -- -i
+          npm run wp-env run tests-wordpress locale -- -a
           
       - name: Run PHPUnit tests
-        # 8.1 and 8.2 are temporarily disabled as docker-images for those cause the errors when pipeline is running
-        if: ${{ matrix.php == '7.4' || matrix.php == '8.0' }}
         run: npm run test:php