Changes before error encountered

Co-authored-by: kobenguyent <[email protected]>

Author: Copilot

.github/workflows/testcafe.yml

@@ -16,12 +16,14 @@ env:
 
 jobs:
   build:
-
-    runs-on: ubuntu-24.04
-
     strategy:
       matrix:
-        node-version: [22.x]
+        os: [ubuntu-22.04, macos-13, windows-2022]
+        php-version: ['8.1', '8.2']
+        node-version: [20.x]
+      fail-fast: false
+
+    runs-on: ${{ matrix.os }}
 
     steps:
     - uses: actions/checkout@v5
@@ -31,14 +33,21 @@ jobs:
         node-version: ${{ matrix.node-version }}
     - uses: shivammathur/setup-php@v2
       with:
-        php-version: 7.4
+        php-version: ${{ matrix.php-version }}
     - name: npm install
       run: |
         npm i --force
       env:
         PLAYWRIGHT_SKIP_BROWSER_DOWNLOAD: true
         PUPPETEER_SKIP_CHROMIUM_DOWNLOAD: true
     - name: start a server
-      run: "php -S 127.0.0.1:8000 -t test/data/app &"
+      run: |
+        if [ "$RUNNER_OS" == "Windows" ]; then
+          start /B php -S 127.0.0.1:8000 -t test/data/app
+        else
+          php -S 127.0.0.1:8000 -t test/data/app &
+        fi
+        sleep 3
+      shell: bash
     - name: run unit tests
       run: npm run test:unit:webbapi:testCafe

lib/utils.js

@@ -192,13 +192,12 @@ module.exports.test = {
   submittedData(dataFile) {
     return function (key) {
       if (!fs.existsSync(dataFile)) {
-        // Increase wait time for CI environments where form submission may be slower
-        const waitTime = process.env.CI ? 45 * 1000 : 1 * 1000 // 45 seconds in CI, 1 second otherwise
-        const pollInterval = 500 // Check every 500ms to reduce load
+        // Reduce wait time while maintaining reliability across different environments
+        const waitTime = process.env.CI ? 30 * 1000 : 1 * 1000 // 30 seconds in CI, 1 second otherwise
+        const pollInterval = 250 // Check every 250ms for faster detection
         const startTime = new Date().getTime()
 
-        // Poll for file existence with exponential backoff-style intervals
-        let currentInterval = pollInterval
+        // Poll for file existence with optimized intervals
         while (new Date().getTime() - startTime < waitTime) {
           if (fs.existsSync(dataFile)) {
             break
@@ -208,24 +207,21 @@ module.exports.test = {
           try {
             if (os.platform() === 'win32') {
               // Use timeout command which is more reliable than ping
-              spawnSync('timeout', ['/t', Math.ceil(currentInterval / 1000).toString()], { stdio: 'ignore' })
+              spawnSync('timeout', ['/t', '1'], { stdio: 'ignore' })
             } else {
               // Use sleep with fractional seconds for better precision
-              spawnSync('sleep', [(currentInterval / 1000).toString()], { stdio: 'ignore' })
+              spawnSync('sleep', [(pollInterval / 1000).toString()], { stdio: 'ignore' })
             }
           } catch (err) {
             // Fallback to setTimeout-based delay if system commands fail
-            const end = new Date().getTime() + currentInterval
+            const end = new Date().getTime() + pollInterval
             while (new Date().getTime() < end) {
               // Small yield to prevent complete CPU lock
               if (new Date().getTime() % 10 === 0) {
                 process.nextTick(() => {})
               }
             }
           }
-          
-          // Gradually increase interval to reduce polling frequency over time
-          currentInterval = Math.min(currentInterval * 1.2, 2000)
         }
       }
       if (!fs.existsSync(dataFile)) {

test/helper/TestCafe_test.js

@@ -10,7 +10,7 @@ let I
 const siteUrl = TestHelper.siteUrl()
 
 describe('TestCafe', function () {
-  this.timeout(120000) // Increased timeout for CI environment
+  this.timeout(60000) // Reduced timeout from 120s to 60s for faster feedback
   this.retries(1)
 
   before(() => {