Set global timeout conditionally

Author: Alex Plischke

src/playwright-runner.ts

@@ -227,10 +227,6 @@ async function runPlaywright(nodeBin: string, runCfg: RunnerConfig): Promise<Run
     playwrightBin, 'test'
   ];
 
-  // Default value for timeout (30min)
-  if (!suite.param.globalTimeout) {
-    suite.param.timeout = 1800000;
-  }
   let args: Record<string, unknown> = _.defaultsDeep(defaultArgs, utils.replaceLegacyKeys(suite.param));
 
   // There is a conflict if the playwright project has a `browser` defined,

src/sauce.config.cjs

@@ -26,6 +26,12 @@ for (const file of configFiles) {
   }
 }
 
+// Set a default timeout of 30 minutes if one is not provided. This protects
+// against tests that hang and never finish.
+if (!userConfig.globalTimeout) {
+  userConfig.globalTimeout = 1000 * 60 * 30; // 30 minutes
+}
+
 const overrides = {
   use: {
     headless: process.env.HEADLESS === 'true',

src/sauce.config.mjs

@@ -27,6 +27,12 @@ for (const file of configFiles) {
   }
 }
 
+// Set a default timeout of 30 minutes if one is not provided. This protects
+// against tests that hang and never finish.
+if (!userConfig.globalTimeout) {
+  userConfig.globalTimeout = 1000 * 60 * 30; // 30 minutes
+}
+
 const overrides = {
   use: {
     headless: process.env.HEADLESS === 'true',