Refactor PHP version specification and enhance testRigor workflow with connection verification and artifact upload

Author: Felix-Suarez-Bonilla

.github/workflows/main.yml

@@ -16,7 +16,7 @@ jobs:
     strategy:
       fail-fast: false
       matrix:
-        php-versions: ['8.2', '8.3', '8.4']
+        php-versions: ["8.2", "8.3", "8.4"]
     steps:
       - uses: actions/checkout@v2
 
@@ -35,30 +35,30 @@ jobs:
           key: ${{ runner.os }}-php-${{ hashFiles('**/composer.lock') }}
           restore-keys: |
             ${{ runner.os }}-php-
-      
+
       - name: Install dependencies
         if: steps.composer-cache.outputs.cache-hit != 'true'
         run: composer install --prefer-dist --no-progress --no-suggest
 
       - name: Run tests
         run: vendor/bin/phpunit --configuration dev/tests/phpunit.xml --testsuite unit --coverage-clover clover.xml
 
-#      - name: Monitor coverage
-#        if: github.event_name == 'pull_request'
-#        uses: slavcodev/coverage-monitor-action@1.2.0
-#        with:
-#          github_token: ${{ secrets.GITHUB_TOKEN }}
-#          clover_file: "clover.xml"
-#          threshold_alert: 10
-#          threshold_warning: 20
+  #      - name: Monitor coverage
+  #        if: github.event_name == 'pull_request'
+  #        uses: slavcodev/coverage-monitor-action@1.2.0
+  #        with:
+  #          github_token: ${{ secrets.GITHUB_TOKEN }}
+  #          clover_file: "clover.xml"
+  #          threshold_alert: 10
+  #          threshold_warning: 20
 
   verification-tests:
     name: Verification Tests
     runs-on: ubuntu-latest
     strategy:
       fail-fast: false
       matrix:
-        php-versions: ['8.2', '8.3', '8.4']
+        php-versions: ["8.2", "8.3", "8.4"]
     steps:
       - uses: actions/checkout@v2
 
@@ -76,7 +76,7 @@ jobs:
           key: ${{ runner.os }}-php-${{ hashFiles('**/composer.lock') }}
           restore-keys: |
             ${{ runner.os }}-php-
-      
+
       - name: Install dependencies
         if: steps.composer-cache.outputs.cache-hit != 'true'
         run: composer install --prefer-dist --no-progress --no-suggest
@@ -90,7 +90,7 @@ jobs:
     strategy:
       fail-fast: false
       matrix:
-        php-versions: ['8.2', '8.3', '8.4']
+        php-versions: ["8.2", "8.3", "8.4"]
     steps:
       - uses: actions/checkout@v2
 
@@ -108,7 +108,7 @@ jobs:
           key: ${{ runner.os }}-php-${{ hashFiles('**/composer.lock') }}
           restore-keys: |
             ${{ runner.os }}-php-
-      
+
       - name: Install dependencies
         if: steps.composer-cache.outputs.cache-hit != 'true'
         run: composer install --prefer-dist --no-progress --no-suggest
@@ -122,7 +122,7 @@ jobs:
     strategy:
       fail-fast: false
       matrix:
-        php-versions: ['8.2', '8.3', '8.4']
+        php-versions: ["8.2", "8.3", "8.4"]
 
     services:
       chrome:
@@ -146,7 +146,7 @@ jobs:
           key: ${{ runner.os }}-php-${{ hashFiles('**/composer.lock') }}
           restore-keys: |
             ${{ runner.os }}-php-
-      
+
       - name: Install dependencies
         if: steps.composer-cache.outputs.cache-hit != 'true'
         run: composer install --prefer-dist --no-progress --no-suggest
@@ -157,18 +157,42 @@ jobs:
   testrigor-tests:
     name: testRigor Tests
     runs-on: ubuntu-latest
+
     env:
+      # testRigor variables
       MAGENTO_TEST_SUITE_ID: ${{vars.MAGENTO_TEST_SUITE_ID}}
       MAGENTO_AUTH_TOKEN: ${{secrets.MAGENTO_AUTH_TOKEN}}
-      LOCALHOST_URL: ${{vars.LOCALHOST_URL}}
+
+      # MFTF Magento connection variables
+      MAGENTO_BASE_URL: ${{vars.MAGENTO_BASE_URL}}
+      MAGENTO_BACKEND_NAME: ${{vars.MAGENTO_BACKEND_NAME}}
+      MAGENTO_ADMIN_USERNAME: ${{vars.MAGENTO_ADMIN_USERNAME}}
+      MAGENTO_ADMIN_PASSWORD: ${{secrets.MAGENTO_ADMIN_PASSWORD}}
+
+      # MFTF configuration
+      SELENIUM_CLOSE_ALL_SESSIONS: true
+      BROWSER: chrome
+      WINDOW_WIDTH: 1920
+      WINDOW_HEIGHT: 1080
+      WAIT_TIMEOUT: 60
+      MAGENTO_CLI_WAIT_TIMEOUT: 60
+      TEST_ENV: ci
+      HEADLESS: true
+
+    services:
+      chrome:
+        image: selenium/standalone-chrome:3.141.59-zirconium
+        ports:
+          - 4444:4444
 
     steps:
       - uses: actions/checkout@v4
+
       - name: Setup PHP
         uses: shivammathur/setup-php@master
         with:
-          php-version: ${{ matrix.php-versions }}
-          extensions: curl, dom, intl, json, openssl
+          php-version: "8.2"
+          extensions: curl, dom, intl, json, openssl, zip
 
       - name: Cache Composer packages
         id: composer-cache
@@ -178,10 +202,33 @@ jobs:
           key: ${{ runner.os }}-php-${{ hashFiles('**/composer.lock') }}
           restore-keys: |
             ${{ runner.os }}-php-
-      
+
       - name: Install dependencies
         if: steps.composer-cache.outputs.cache-hit != 'true'
         run: composer install --prefer-dist --no-progress --no-suggest
 
-      - name: Run tests
-        run: bin/mftf run:testrigor
+      - name: Verify Magento connection
+        run: |
+          echo "Testing connection to $MAGENTO_BASE_URL"
+          curl -f -s -o /dev/null $MAGENTO_BASE_URL || echo "Warning: Cannot connect to Magento"
+
+      - name: Build MFTF project
+        run: php bin/mftf build:project
+
+      - name: Verify MFTF configuration
+        run: php bin/mftf doctor || true
+
+      - name: Run testRigor tests
+        run: |
+          echo "Running MFTF testRigor tests..."
+          php bin/mftf run:testrigor --debug
+
+      - name: Upload test artifacts
+        if: failure()
+        uses: actions/upload-artifact@v4
+        with:
+          name: mftf-test-artifacts
+          path: |
+            dev/tests/acceptance/_output/
+            var/log/
+          retention-days: 7