Change error checking code to work

Author: pokey

src/test/suite/fixtures/recorded/nonMatchErrors/takeFunk.yml

@@ -16,4 +16,4 @@ initialState:
 finalState: null
 returnValue: null
 fullTargets: [{type: primitive, mark: {type: cursor}, selectionType: token, position: contents, insideOutsideType: inside, modifier: {type: containingScope, scopeType: namedFunction, includeSiblings: false}, isImplicit: false}]
-errorName: NoContainingScopeError
+thrownError: {name: NoContainingScopeError}

src/test/suite/fixtures/recorded/nonMatchErrors/takeList.yml

@@ -16,4 +16,4 @@ initialState:
 finalState: null
 returnValue: null
 fullTargets: [{type: primitive, mark: {type: cursor}, selectionType: token, position: contents, insideOutsideType: inside, modifier: {type: containingScope, scopeType: list, includeSiblings: false}, isImplicit: false}]
-errorName: Error
+thrownError: {name: NoContainingScopeError}

src/test/suite/recorded.test.ts

@@ -105,19 +105,22 @@ async function runTest(file: string) {
 
   // Assert that recorded decorations are present
   checkMarks(fixture.initialState.marks, readableHatMap);
+
   if (fixture.thrownError != null) {
-    try {
-      await vscode.commands.executeCommand(
-        "cursorless.command",
-        fixture.command
-      );
-    } catch (e) {
-      const err = e as Error;
-      assert.equal(err.name, fixture.thrownError);
-    } finally {
-      return;
-    }
+    await assert.rejects(
+      async () => {
+        await vscode.commands.executeCommand(
+          "cursorless.command",
+          fixture.command
+        );
+      },
+      (err: Error) => {
+        assert.strictEqual(err.name, fixture.thrownError?.name);
+      }
+    );
+    return;
   }
+
   const returnValue = await vscode.commands.executeCommand(
     "cursorless.command",
     fixture.command