chore: update telemetry test not to fail without key VSCODE-517

Author: tculig

src/test/suite/helpers.ts

@@ -0,0 +1,26 @@
+const isCI =
+  ['true', '1'].includes(String(process.env.CI).toLowerCase()) ||
+  ['true', '1'].includes(String(process.env.GITHUB_ACTIONS).toLowerCase());
+
+const isCISimplified = process.env.CI || process.env.GITHUB_ACTIONS;
+
+export const testIfCI = (
+  name: string,
+  fn: Mocha.Func | Mocha.AsyncFunc,
+): void => {
+  (isCI ? test : test.skip)(name, fn);
+};
+
+export const suiteIfCI = (
+  name: string,
+  fn: (this: Mocha.Suite) => void,
+): void => {
+  (isCI ? suite : suite.skip)(name, fn);
+};
+
+export function testIfCIv2(
+  testName: string,
+  testFn: () => void | Promise<void>,
+): void {
+  isCISimplified ? test(testName, testFn) : test.skip(testName, testFn);
+}

src/test/suite/telemetry/telemetryService.test.ts

@@ -25,6 +25,7 @@ import {
   SavedConnectionsLoadedTelemetryEvent,
 } from '../../../telemetry';
 import type { SegmentProperties } from '../../../telemetry/telemetryService';
+import { suiteIfCI } from '../helpers';
 
 // eslint-disable-next-line @typescript-eslint/no-var-requires
 const { version } = require('../../../../package.json');
@@ -35,7 +36,7 @@ chai.use(sinonChai);
 
 config({ path: resolve(__dirname, '../../../../.env') });
 
-suite('Telemetry Controller Test Suite', () => {
+suiteIfCI('Telemetry Controller Test Suite', () => {
   const testTelemetryService =
     mdbTestExtension.testExtensionController._telemetryService;