Skip performance tests on Mac CI for now, since they're flaky there (#3183)

Author: AndreasArvidsson

packages/cursorless-vscode-e2e/src/suite/performance.vscode.test.ts

@@ -29,6 +29,12 @@ suite(`Performance ${thresholds.join("/")} ms`, async function () {
   // Before each test, print the test title. This is done so we have the test
   // title before the test run time / duration.
   this.beforeEach(function () {
+    // FIXME: This test is flaky on Mac CI, so we skip it there for now
+    if (isCI() && isMac()) {
+      this.skip();
+      return;
+    }
+
     const title = this.currentTest!.title;
     if (title !== previousTitle) {
       console.log(`    ${title}`);
@@ -261,13 +267,8 @@ function generateTestData(n: number): string {
 
 function calculateMultiplier() {
   // The GitHub test runner is generally slower than running tests locally, so
-  // we increase the thresholds in CI. We do this for all platforms, but
-  // especially for macOS since the GitHub test runner for macOS is particularly
-  // slow.
+  // we increase the thresholds in CI.
   if (isCI()) {
-    if (isMac()) {
-      return 3;
-    }
     return 2;
   }
   return 1;