More CI

Author: pokey

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

@@ -16,6 +16,7 @@ import path from "path";
 import sinon from "sinon";
 import { commands } from "vscode";
 import { endToEndTestSetup } from "../../endToEndTestSetup";
+import { isEqual, uniqWith } from "lodash-es";
 
 suite("tutorial", async function () {
   // Retry doesn't make sense because we need to capture initial load events of
@@ -146,7 +147,9 @@ async function runBasicTutorialTest(spyIde: SpyIDE) {
   // Allow for debounce
   await sleep(250);
 
-  const log = getTutorialWebviewEventLog();
+  // We allow duplicate messages because they're idempotent. Not sure why some
+  // platforms get the init message twice but it doesn't matter.
+  const log = uniqWith(getTutorialWebviewEventLog(), isEqual);
   assert.equal(log.length, 3, JSON.stringify(log, null, 2));
   const lastMessage = log[log.length - 1];
   assert(

packages/cursorless-vscode/src/VscodeTutorial.ts

@@ -76,7 +76,6 @@ export class VscodeTutorial implements WebviewViewProvider {
     _context: WebviewViewResolveContext,
     _token: CancellationToken,
   ) {
-    console.log("resolveWebviewView");
     if (this.view != null && this.view instanceof SpyWebviewView) {
       this.view.view = webviewView;
     } else {
@@ -117,8 +116,8 @@ export class VscodeTutorial implements WebviewViewProvider {
     return [];
   }
 
-  public async start(tutorialId: TutorialId) {
-    await this.tutorial.start(tutorialId);
+  public async start(id: TutorialId | number) {
+    await this.tutorial.start(id);
     this.revealTutorial();
   }