From c047624b683175651c894e1124e6734eeac90a8b Mon Sep 17 00:00:00 2001
From: Andrey Lushnikov <lushnikov@chromium.org>
Date: Wed, 7 Aug 2019 10:26:53 -0700
Subject: [PATCH] chore: generate testIds on CIs only (#4817)

---
 test/utils.js | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/test/utils.js b/test/utils.js
index fbc5528c..ef8be3a2 100644
--- 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.