chore: generate testIds on CIs only (#4817)

puppeteer · Aug 7, 2019 · c047624 · c047624
1 parent 656fe46
commit c047624
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/test/utils.js b/test/utils.js
@@ -165,9 +165,9 @@ const utils = module.exports = {
     // Generate testIDs for all tests and verify they don't clash.
     // This will add |test.testId| for every test.
     //
-    // NOTE: we do this unconditionally so that developers can see problems in
-    // their local setups.
-    generateTestIDs(testRunner);
+    // NOTE: we do this on CI's so that problems arise on PR trybots.
+    if (process.env.CI)
+      generateTestIDs(testRunner);
     // FLAKINESS_DASHBOARD_PASSWORD is an encrypted/secured variable.
     // Encrypted variables get a special treatment in CI's when handling PRs so that
     // secrets are not leaked to untrusted code.