Clean up snippet test

Author: AndreasArvidsson

packages/lib-engine/src/test/fixtures/communitySnippets.fixture.ts

@@ -17,18 +17,10 @@ const snippetAfterAction: ActionDescriptor = {
   },
   snippetDescription: {
     type: "list",
-    // This will be the current active language
-    fallbackLanguage: "typescript",
     snippets: [
       {
         type: "custom",
-        languages: [
-          "javascript",
-          "typescript",
-          "javascriptreact",
-          "typescriptreact",
-        ],
-        body: 'import * as $0 from "$0";',
+        body: "```$0\n$1\n```",
       },
     ],
   },
@@ -40,5 +32,5 @@ const snippetAfterAction: ActionDescriptor = {
  * Talon tests by relying on our recorded test fixtures alone.
  */
 export const communitySnippetsSpokenFormsFixture = [
-  spokenFormTest("snip import star after air", snippetAfterAction, undefined),
+  spokenFormTest("snip code after air", snippetAfterAction, undefined),
 ];

packages/test-runner/src/runAllTests.ts

@@ -34,19 +34,28 @@ export function runAllTests(type: TestType): Promise<void> {
   const testRoot = path.join(getCursorlessRepoRoot(), "packages");
 
   let filePattern: string;
-  let ignore: string[] | undefined;
-
-  if (type === TestType.unit) {
-    filePattern = "test.ts";
-    ignore = [
-      TestType.vscode,
-      TestType.talon,
-      TestType.talonJs,
-      TestType.neovim,
-    ].map((t) => `**/*.${t}.ts`);
-  } else {
-    filePattern = `${type}.cjs`;
-    ignore = undefined;
+  let ignore: string[] | undefined = undefined;
+
+  switch (type) {
+    case TestType.unit:
+      filePattern = "test.ts";
+      ignore = [
+        TestType.vscode,
+        TestType.talon,
+        TestType.talonJs,
+        TestType.neovim,
+      ].map((t) => `**/*.${t}.ts`);
+      break;
+
+    case TestType.talon:
+    case TestType.talonJs:
+      filePattern = `${type}.ts`;
+      break;
+
+    // These tests have to be .cjs files because they import vscode and neovim modules which don't work with ts files
+    case TestType.vscode:
+    case TestType.neovim:
+      filePattern = `${type}.cjs`;
   }
 
   return runTestsInDir(testRoot, filePattern, ignore);