@teamkeel/testing-runtime 0.474.0 → 0.475.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@teamkeel/testing-runtime",
-  "version": "0.474.0",
+  "version": "0.475.0",
   "description": "Internal package used by the generated @teamkeel/testing package",
   "exports": "./src/index.mjs",
   "typings": "src/index.d.ts",

package/src/Executor.mjs

@@ -20,6 +20,15 @@ export class Executor {
         if (v !== undefined) {
           return v;
         }
+        // An executor must not look like a promise. Handing back a function for `then` makes
+        // `await executor` treat it as a thenable and dispatch an action called "then", which
+        // 404s and never settles the await. Symbols are never action names either.
+        // An executor must not look like a promise. Handing back a function for `then` makes
+        // `await executor` treat it as a thenable and dispatch an action called "then", which
+        // 404s and never settles the await. Symbols are never action names either.
+        if (prop === "then" || typeof prop === "symbol") {
+          return undefined;
+        }
         return target._execute.bind(target, prop);
       },
     });

package/src/index.test.ts

@@ -1,5 +1,6 @@
 import { expect, test } from "vitest";
 import "./index";
+import { Executor } from "./Executor.mjs";
 
 test("toHaveAuthorizationError", async () => {
   const p = Promise.reject({
@@ -70,3 +71,31 @@ test("not.toHaveError", async () => {
     code: "ERR_INVALID_INPUT",
   });
 });
+
+// An executor answers to any property with an action call, so it must explicitly refuse the ones
+// the language itself reaches for. Answering `then` with a function makes `await executor` treat
+// it as a thenable: it dispatches an action called "then" and never settles.
+test("an executor is not a thenable", async () => {
+  const executor = new Executor({
+    apiBaseUrl: "http://localhost",
+    parseJsonResult: false,
+  });
+
+  expect(executor.then).toBeUndefined();
+
+  const resolved = await Promise.resolve(executor);
+  expect(resolved).toBe(executor);
+
+  const returned = await (async () => executor)();
+  expect(returned).toBe(executor);
+});
+
+test("an executor does not answer to symbols", () => {
+  const executor = new Executor({
+    apiBaseUrl: "http://localhost",
+    parseJsonResult: false,
+  });
+
+  expect(executor[Symbol.toPrimitive]).toBeUndefined();
+  expect(executor[Symbol.iterator]).toBeUndefined();
+});