@tui-sandbox/library 2.0.2 → 2.1.0

This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
Files changed (47) hide show
  1. package/CHANGELOG.md +7 -0
  2. package/dist/src/client/neovim-client.d.ts +2 -1
  3. package/dist/src/client/neovim-client.js +12 -3
  4. package/dist/src/server/TestServer.d.ts +1 -1
  5. package/dist/src/server/TestServer.js +15 -24
  6. package/dist/src/server/connection/trpc.d.ts +7 -15
  7. package/dist/src/server/connection/trpc.js +0 -3
  8. package/dist/src/server/neovim/NeovimApplication.d.ts +3 -2
  9. package/dist/src/server/neovim/NeovimApplication.js +21 -13
  10. package/dist/src/server/neovim/environment/createTempDir.test.js +1 -0
  11. package/dist/src/server/neovim/index.d.ts +1 -2
  12. package/dist/src/server/neovim/index.js +15 -21
  13. package/dist/src/server/server.d.ts +4 -4
  14. package/dist/src/server/server.js +1 -1
  15. package/dist/src/server/updateTestdirectorySchemaFile.d.ts +2 -1
  16. package/dist/src/server/updateTestdirectorySchemaFile.js +4 -0
  17. package/dist/src/server/updateTestdirectorySchemaFile.test.d.ts +1 -0
  18. package/dist/src/server/updateTestdirectorySchemaFile.test.js +34 -0
  19. package/dist/src/server/utilities/DisposableSingleApplication.d.ts +6 -8
  20. package/dist/src/server/utilities/DisposableSingleApplication.js +9 -9
  21. package/dist/src/server/utilities/DisposableSingleApplication.test.d.ts +1 -0
  22. package/dist/src/server/utilities/DisposableSingleApplication.test.js +55 -0
  23. package/dist/src/server/utilities/TerminalApplication.d.ts +2 -1
  24. package/dist/src/server/utilities/generator.d.ts +2 -0
  25. package/dist/src/server/utilities/generator.js +8 -0
  26. package/dist/src/server/utilities/generator.test.d.ts +1 -0
  27. package/dist/src/server/utilities/generator.test.js +41 -0
  28. package/dist/tsconfig.tsbuildinfo +1 -1
  29. package/dist/vitest.config.d.ts +2 -0
  30. package/dist/vitest.config.js +8 -0
  31. package/package.json +11 -10
  32. package/src/client/neovim-client.ts +14 -4
  33. package/src/server/TestServer.ts +17 -25
  34. package/src/server/connection/trpc.ts +1 -15
  35. package/src/server/neovim/NeovimApplication.ts +24 -14
  36. package/src/server/neovim/environment/createTempDir.test.ts +2 -0
  37. package/src/server/neovim/index.ts +25 -27
  38. package/src/server/server.ts +1 -1
  39. package/src/server/updateTestdirectorySchemaFile.test.ts +43 -0
  40. package/src/server/updateTestdirectorySchemaFile.ts +7 -2
  41. package/src/server/utilities/DisposableSingleApplication.test.ts +70 -0
  42. package/src/server/utilities/DisposableSingleApplication.ts +17 -12
  43. package/src/server/utilities/TerminalApplication.ts +2 -1
  44. package/src/server/utilities/generator.test.ts +49 -0
  45. package/src/server/utilities/generator.ts +13 -0
  46. package/tsconfig.json +2 -2
  47. package/vitest.config.ts +9 -0
package/CHANGELOG.md CHANGED
@@ -1,5 +1,12 @@
1
1
  # Changelog
2
2
 
3
+ ## [2.1.0](https://github.com/mikavilpas/tui-sandbox/compare/library-v2.0.2...library-v2.1.0) (2024-11-02)
4
+
5
+
6
+ ### Features
7
+
8
+ * client connects to server using server-sent events ([e07765e](https://github.com/mikavilpas/tui-sandbox/commit/e07765e20dba6394538cf29b1b16463b62ba4b7a))
9
+
3
10
  ## [2.0.2](https://github.com/mikavilpas/tui-sandbox/compare/library-v2.0.1...library-v2.0.2) (2024-10-20)
4
11
 
5
12
 
@@ -1,4 +1,5 @@
1
1
  import "@xterm/xterm/css/xterm.css";
2
+ import type { Except } from "type-fest";
2
3
  import type { StartNeovimGenericArguments } from "../server/neovim/NeovimApplication.ts";
3
4
  import type { TestDirectory } from "../server/types.ts";
4
5
  import "./style.css";
@@ -8,5 +9,5 @@ export declare class NeovimClient {
8
9
  private readonly terminal;
9
10
  private readonly trpc;
10
11
  constructor(app: HTMLElement);
11
- startNeovim(args: StartNeovimGenericArguments): Promise<TestDirectory>;
12
+ startNeovim(args: Except<StartNeovimGenericArguments, "terminalDimensions">): Promise<TestDirectory>;
12
13
  }
@@ -1,4 +1,4 @@
1
- import { createTRPCClient, createWSClient, wsLink } from "@trpc/client";
1
+ import { createTRPCClient, httpBatchLink, splitLink, unstable_httpSubscriptionLink } from "@trpc/client";
2
2
  import "@xterm/xterm/css/xterm.css";
3
3
  import "./style.css";
4
4
  import { getTabId, startTerminal } from "./websocket-client.js";
@@ -8,9 +8,18 @@ export class NeovimClient {
8
8
  terminal;
9
9
  trpc;
10
10
  constructor(app) {
11
- const wsClient = createWSClient({ url: `ws://localhost:3000`, WebSocket });
12
11
  const trpc = createTRPCClient({
13
- links: [wsLink({ client: wsClient })],
12
+ links: [
13
+ splitLink({
14
+ condition: operation => operation.type === "subscription",
15
+ true: unstable_httpSubscriptionLink({
16
+ url: "http://localhost:3000",
17
+ }),
18
+ false: httpBatchLink({
19
+ url: "http://localhost:3000",
20
+ }),
21
+ }),
22
+ ],
14
23
  });
15
24
  this.trpc = trpc;
16
25
  this.tabId = getTabId();
@@ -4,5 +4,5 @@ import type { TestServerConfig } from "./updateTestdirectorySchemaFile";
4
4
  export declare class TestServer {
5
5
  private readonly port;
6
6
  constructor(port: number);
7
- startAndRun<TRouter extends AnyTRPCRouter>(appRouter: TRouter, config: TestServerConfig): Promise<void>;
7
+ startAndRun(appRouter: AnyTRPCRouter, config: TestServerConfig): Promise<void>;
8
8
  }
@@ -1,48 +1,39 @@
1
- import { applyWSSHandler } from "@trpc/server/adapters/ws";
1
+ import { createHTTPServer } from "@trpc/server/adapters/standalone";
2
2
  import "core-js/proposals/async-explicit-resource-management";
3
+ import cors from "cors";
3
4
  import { once } from "events";
4
- import { WebSocketServer } from "ws";
5
- import { createContext } from "./connection/trpc";
6
5
  import { updateTestdirectorySchemaFile } from "./updateTestdirectorySchemaFile";
7
6
  export class TestServer {
8
7
  port;
9
8
  constructor(port) {
10
9
  this.port = port;
11
10
  }
12
- // eslint-disable-next-line @typescript-eslint/no-unnecessary-type-parameters
13
11
  async startAndRun(appRouter, config) {
14
12
  console.log("🚀 Server starting");
15
13
  await updateTestdirectorySchemaFile(config);
16
- const wss = new WebSocketServer({ port: this.port });
17
- const handler = applyWSSHandler({
18
- wss,
14
+ const server = createHTTPServer({
19
15
  router: appRouter,
20
- createContext,
21
- // Enable heartbeat messages to keep connection open (disabled by default)
22
- keepAlive: {
23
- enabled: true,
24
- // server ping message interval in milliseconds
25
- pingMs: 30_000,
26
- // connection is terminated if pong message is not received in this many milliseconds
27
- pongWaitMs: 5000,
28
- },
16
+ createContext: () => ({}),
17
+ middleware: cors({
18
+ origin: "*",
19
+ }),
29
20
  });
30
- wss.on("connection", socket => {
31
- console.log(`➕➕ Connection (${wss.clients.size})`);
21
+ server.listen(this.port);
22
+ server.on("connection", socket => {
23
+ console.log(`➕➕ Connection`);
32
24
  socket.once("close", () => {
33
- console.log(`➖➖ Connection (${wss.clients.size})`);
25
+ console.log(`➖➖ Connection`);
34
26
  });
35
27
  });
36
- console.log(`✅ WebSocket Server listening on ws://localhost:${this.port}`);
28
+ console.log(`✅ Server listening on ws://localhost:${this.port}`);
37
29
  await Promise.race([once(process, "SIGTERM"), once(process, "SIGINT")]);
38
30
  console.log("Shutting down...");
39
- handler.broadcastReconnectNotification();
40
- wss.close(error => {
31
+ server.close(error => {
41
32
  if (error) {
42
- console.error("Error closing WebSocket server", error);
33
+ console.error("Error closing server", error);
43
34
  process.exit(1);
44
35
  }
45
- console.log("WebSocket server closed");
36
+ console.log("Server closed");
46
37
  process.exit(0);
47
38
  });
48
39
  }
@@ -1,29 +1,21 @@
1
- import type { CreateWSSContextFnOptions } from "@trpc/server/adapters/ws";
2
- import type { Socket } from "net";
3
- import type { WebSocket } from "ws";
4
- export type Connection = {
5
- clientId: WebSocket;
6
- socket: Socket;
7
- };
8
- export declare function createContext(opts: CreateWSSContextFnOptions): Connection;
9
1
  export declare const trpc: {
10
2
  _config: import("@trpc/server/unstable-core-do-not-import").RootConfig<{
11
- ctx: Connection;
3
+ ctx: object;
12
4
  meta: object;
13
5
  errorShape: import("@trpc/server/unstable-core-do-not-import").DefaultErrorShape;
14
6
  transformer: false;
15
7
  }>;
16
- procedure: import("@trpc/server/unstable-core-do-not-import").ProcedureBuilder<Connection, object, object, typeof import("@trpc/server/unstable-core-do-not-import").unsetMarker, typeof import("@trpc/server/unstable-core-do-not-import").unsetMarker, typeof import("@trpc/server/unstable-core-do-not-import").unsetMarker, typeof import("@trpc/server/unstable-core-do-not-import").unsetMarker, false>;
17
- middleware: <$ContextOverrides>(fn: import("@trpc/server/unstable-core-do-not-import").MiddlewareFunction<Connection, object, object, $ContextOverrides, unknown>) => import("@trpc/server/unstable-core-do-not-import").MiddlewareBuilder<Connection, object, $ContextOverrides, unknown>;
8
+ procedure: import("@trpc/server/unstable-core-do-not-import").ProcedureBuilder<object, object, object, typeof import("@trpc/server/unstable-core-do-not-import").unsetMarker, typeof import("@trpc/server/unstable-core-do-not-import").unsetMarker, typeof import("@trpc/server/unstable-core-do-not-import").unsetMarker, typeof import("@trpc/server/unstable-core-do-not-import").unsetMarker, false>;
9
+ middleware: <$ContextOverrides>(fn: import("@trpc/server/unstable-core-do-not-import").MiddlewareFunction<object, object, object, $ContextOverrides, unknown>) => import("@trpc/server/unstable-core-do-not-import").MiddlewareBuilder<object, object, $ContextOverrides, unknown>;
18
10
  router: {
19
11
  <TInput extends import("@trpc/server").RouterRecord>(input: TInput): import("@trpc/server/unstable-core-do-not-import").BuiltRouter<{
20
- ctx: Connection;
12
+ ctx: object;
21
13
  meta: object;
22
14
  errorShape: import("@trpc/server/unstable-core-do-not-import").DefaultErrorShape;
23
15
  transformer: false;
24
16
  }, TInput>;
25
17
  <TInput extends import("@trpc/server/unstable-core-do-not-import").CreateRouterOptions>(input: TInput): import("@trpc/server/unstable-core-do-not-import").BuiltRouter<{
26
- ctx: Connection;
18
+ ctx: object;
27
19
  meta: object;
28
20
  errorShape: import("@trpc/server/unstable-core-do-not-import").DefaultErrorShape;
29
21
  transformer: false;
@@ -31,12 +23,12 @@ export declare const trpc: {
31
23
  };
32
24
  mergeRouters: typeof import("@trpc/server/unstable-core-do-not-import").mergeRouters;
33
25
  createCallerFactory: <TRecord extends import("@trpc/server").RouterRecord>(router: Pick<import("@trpc/server/unstable-core-do-not-import").Router<{
34
- ctx: Connection;
26
+ ctx: object;
35
27
  meta: object;
36
28
  errorShape: import("@trpc/server/unstable-core-do-not-import").DefaultErrorShape;
37
29
  transformer: false;
38
30
  }, TRecord>, "_def">) => import("@trpc/server/unstable-core-do-not-import").RouterCaller<{
39
- ctx: Connection;
31
+ ctx: object;
40
32
  meta: object;
41
33
  errorShape: import("@trpc/server/unstable-core-do-not-import").DefaultErrorShape;
42
34
  transformer: false;
@@ -1,5 +1,2 @@
1
1
  import { initTRPC } from "@trpc/server";
2
- export function createContext(opts) {
3
- return { clientId: opts.res, socket: opts.req.socket };
4
- }
5
2
  export const trpc = initTRPC.context().create();
@@ -12,11 +12,12 @@ export type StartNeovimGenericArguments = {
12
12
  };
13
13
  startupScriptModifications?: string[];
14
14
  };
15
- export declare class NeovimApplication extends DisposableSingleApplication {
15
+ export declare class NeovimApplication {
16
16
  private readonly testEnvironmentPath;
17
+ readonly application: DisposableSingleApplication;
17
18
  private testDirectory;
18
19
  readonly events: EventEmitter;
19
- constructor(testEnvironmentPath: string);
20
+ constructor(testEnvironmentPath: string, application?: DisposableSingleApplication);
20
21
  /**
21
22
  * Kill the current application and start a new one with the given arguments.
22
23
  */
@@ -1,16 +1,18 @@
1
+ import assert from "assert";
1
2
  import { exec } from "child_process";
2
3
  import EventEmitter from "events";
3
4
  import { existsSync } from "fs";
4
5
  import path from "path";
5
6
  import { DisposableSingleApplication } from "../utilities/DisposableSingleApplication";
6
7
  import { TerminalApplication } from "../utilities/TerminalApplication";
7
- export class NeovimApplication extends DisposableSingleApplication {
8
+ export class NeovimApplication {
8
9
  testEnvironmentPath;
10
+ application;
9
11
  testDirectory;
10
12
  events;
11
- constructor(testEnvironmentPath) {
12
- super();
13
+ constructor(testEnvironmentPath, application = new DisposableSingleApplication()) {
13
14
  this.testEnvironmentPath = testEnvironmentPath;
15
+ this.application = application;
14
16
  this.events = new EventEmitter();
15
17
  }
16
18
  /**
@@ -42,19 +44,25 @@ export class NeovimApplication extends DisposableSingleApplication {
42
44
  }
43
45
  }
44
46
  const stdout = this.events;
45
- this.application = TerminalApplication.start({
46
- command: "nvim",
47
- args: neovimArguments,
48
- cwd: this.testEnvironmentPath,
49
- env: process.env,
50
- dimensions: startArgs.terminalDimensions,
51
- onStdoutOrStderr(data) {
52
- stdout.emit("stdout", data);
53
- },
47
+ await this.application.startNextAndKillCurrent(async () => {
48
+ return TerminalApplication.start({
49
+ command: "nvim",
50
+ args: neovimArguments,
51
+ cwd: this.testEnvironmentPath,
52
+ env: process.env,
53
+ dimensions: startArgs.terminalDimensions,
54
+ onStdoutOrStderr(data) {
55
+ data;
56
+ stdout.emit("stdout", data);
57
+ },
58
+ });
54
59
  });
60
+ const processId = this.application.processId();
61
+ assert(processId !== undefined, "Neovim was started without a process ID. This is a bug - please open an issue.");
62
+ console.log(`🚀 Started Neovim instance ${processId}`);
55
63
  }
56
64
  async [Symbol.asyncDispose]() {
57
- await super.killCurrent();
65
+ await this.application[Symbol.asyncDispose]();
58
66
  if (this.testDirectory) {
59
67
  exec(`rm -rf ${this.testDirectory.rootPathAbsolute}`);
60
68
  }
@@ -2,6 +2,7 @@ import fs, { rmdirSync } from "fs";
2
2
  import nodePath from "path";
3
3
  import { expect, it } from "vitest";
4
4
  import { createTempDir } from "./createTempDir";
5
+ vi.spyOn(console, "log").mockImplementation(vi.fn());
5
6
  class TempDirectory {
6
7
  path;
7
8
  constructor(path) {
@@ -1,11 +1,10 @@
1
- import type { Observable } from "@trpc/server/observable";
2
1
  import type { TestDirectory } from "../types";
3
2
  import type { TestServerConfig } from "../updateTestdirectorySchemaFile";
4
3
  import type { TabId } from "../utilities/tabId";
5
4
  import type { StartNeovimGenericArguments } from "./NeovimApplication";
6
5
  export declare function onStdout(options: {
7
6
  client: TabId;
8
- }, testEnvironmentPath: string): Observable<string, unknown>;
7
+ }, signal: AbortSignal | undefined, testEnvironmentPath: string): Promise<AsyncGenerator<string, void, unknown>>;
9
8
  export declare function start(options: StartNeovimGenericArguments, tabId: TabId, config: TestServerConfig): Promise<TestDirectory>;
10
9
  export declare function sendStdin(options: {
11
10
  tabId: TabId;
@@ -1,40 +1,34 @@
1
- import { observable } from "@trpc/server/observable";
2
1
  import assert from "assert";
2
+ import { convertEventEmitterToAsyncGenerator } from "../utilities/generator";
3
3
  import { createTempDir } from "./environment/createTempDir";
4
4
  import { NeovimApplication } from "./NeovimApplication";
5
5
  const neovims = new Map();
6
- export function onStdout(options, testEnvironmentPath) {
7
- return observable(emit => {
8
- const tabId = options.client.tabId;
9
- const neovim = neovims.get(tabId) ?? new NeovimApplication(testEnvironmentPath);
10
- if (neovims.get(tabId) === undefined) {
11
- neovims.set(tabId, neovim);
12
- }
13
- const send = (data) => {
14
- assert(typeof data === "string");
15
- emit.next(data);
16
- };
17
- neovim.events.on("stdout", send);
18
- return () => {
19
- neovim.events.off("stdout", send);
6
+ export async function onStdout(options, signal, testEnvironmentPath) {
7
+ const tabId = options.client.tabId;
8
+ const neovim = neovims.get(tabId) ?? new NeovimApplication(testEnvironmentPath);
9
+ if (neovims.get(tabId) === undefined) {
10
+ neovims.set(tabId, neovim);
11
+ }
12
+ const stdout = convertEventEmitterToAsyncGenerator(neovim.events, "stdout");
13
+ if (signal) {
14
+ signal.addEventListener("abort", () => {
20
15
  void neovim[Symbol.asyncDispose]().finally(() => {
21
16
  neovims.delete(tabId);
22
17
  });
23
- };
24
- });
18
+ });
19
+ }
20
+ return stdout;
25
21
  }
26
22
  export async function start(options, tabId, config) {
27
23
  const neovim = neovims.get(tabId.tabId);
28
24
  assert(neovim, `Neovim instance not found for client id ${tabId.tabId}`);
29
25
  const testDirectory = await createTempDir(config);
30
26
  await neovim.startNextAndKillCurrent(testDirectory, options);
31
- const processId = neovim.processId();
32
- assert(processId !== undefined, "Neovim was started without a process ID. This is a bug - please open an issue.");
33
- console.log(`🚀 Started Neovim instance ${processId}`);
34
27
  return testDirectory;
35
28
  }
36
29
  export async function sendStdin(options) {
37
30
  const neovim = neovims.get(options.tabId.tabId);
38
31
  assert(neovim !== undefined, `Neovim instance for clientId not found - cannot send stdin. Maybe it's not started yet?`);
39
- await neovim.write(options.data);
32
+ assert(neovim.application, `Neovim application not found for client id ${options.tabId.tabId}. Maybe it's not started yet?`);
33
+ await neovim.application.write(options.data);
40
34
  }
@@ -6,13 +6,13 @@ import type { TestServerConfig } from "./updateTestdirectorySchemaFile";
6
6
  declare const autocleanup: AsyncDisposableStack;
7
7
  export { autocleanup };
8
8
  declare function createAppRouter(config: TestServerConfig): import("@trpc/server/unstable-core-do-not-import").BuiltRouter<{
9
- ctx: import("./connection/trpc").Connection;
9
+ ctx: object;
10
10
  meta: object;
11
11
  errorShape: import("@trpc/server/unstable-core-do-not-import").DefaultErrorShape;
12
12
  transformer: false;
13
13
  }, import("@trpc/server/unstable-core-do-not-import").DecorateCreateRouterOptions<{
14
14
  neovim: import("@trpc/server/unstable-core-do-not-import").BuiltRouter<{
15
- ctx: import("./connection/trpc").Connection;
15
+ ctx: object;
16
16
  meta: object;
17
17
  errorShape: import("@trpc/server/unstable-core-do-not-import").DefaultErrorShape;
18
18
  transformer: false;
@@ -35,13 +35,13 @@ declare function createAppRouter(config: TestServerConfig): import("@trpc/server
35
35
  };
36
36
  output: import("./types").TestDirectory;
37
37
  }>;
38
- onStdout: import("@trpc/server/unstable-core-do-not-import").LegacyObservableSubscriptionProcedure<{
38
+ onStdout: import("@trpc/server").TRPCSubscriptionProcedure<{
39
39
  input: {
40
40
  client: {
41
41
  tabId: string;
42
42
  };
43
43
  };
44
- output: string;
44
+ output: AsyncGenerator<string, void, unknown>;
45
45
  }>;
46
46
  sendStdin: import("@trpc/server").TRPCMutationProcedure<{
47
47
  input: {
@@ -34,7 +34,7 @@ function createAppRouter(config) {
34
34
  return neovim.start(options.input.startNeovimArguments, options.input.tabId, config);
35
35
  }),
36
36
  onStdout: trpc.procedure.input(z.object({ client: tabIdSchema })).subscription(options => {
37
- return neovim.onStdout(options.input, config.testEnvironmentPath);
37
+ return neovim.onStdout(options.input, options.signal, config.testEnvironmentPath);
38
38
  }),
39
39
  sendStdin: trpc.procedure.input(z.object({ tabId: tabIdSchema, data: z.string() })).mutation(options => {
40
40
  return neovim.sendStdin(options.input);
@@ -3,4 +3,5 @@ export type TestServerConfig = {
3
3
  testEnvironmentPath: string;
4
4
  outputFilePath: string;
5
5
  };
6
- export declare function updateTestdirectorySchemaFile({ testEnvironmentPath, outputFilePath, }: TestServerConfig): Promise<void>;
6
+ export type UpdateTestdirectorySchemaFileResult = "updated" | "did-nothing";
7
+ export declare function updateTestdirectorySchemaFile({ testEnvironmentPath, outputFilePath, }: TestServerConfig): Promise<UpdateTestdirectorySchemaFileResult>;
@@ -15,5 +15,9 @@ export async function updateTestdirectorySchemaFile({ testEnvironmentPath, outpu
15
15
  // because file watchers will trigger on file changes and we don't want to
16
16
  // trigger a build if the schema hasn't changed
17
17
  writeFileSync(outputFilePath, newSchema);
18
+ return "updated";
19
+ }
20
+ else {
21
+ return "did-nothing";
18
22
  }
19
23
  }
@@ -0,0 +1,34 @@
1
+ import { readFileSync, writeFileSync } from "fs";
2
+ import { buildTestDirectorySchema } from "./dirtree";
3
+ import { updateTestdirectorySchemaFile } from "./updateTestdirectorySchemaFile";
4
+ vi.mock("fs");
5
+ vi.mock("./dirtree");
6
+ vi.spyOn(console, "log").mockImplementation(vi.fn());
7
+ const mock = {
8
+ readFileSync: vi.mocked(readFileSync),
9
+ writeFileSync: vi.mocked(writeFileSync),
10
+ buildTestDirectorySchema: vi.mocked(buildTestDirectorySchema),
11
+ };
12
+ describe("when the schema has not changed", () => {
13
+ it("does not write the file", async () => {
14
+ mock.buildTestDirectorySchema.mockResolvedValue("schema");
15
+ mock.readFileSync.mockImplementation(() => "schema");
16
+ const result = await updateTestdirectorySchemaFile({
17
+ testEnvironmentPath: "path",
18
+ outputFilePath: "path",
19
+ });
20
+ expect(result).toBe("did-nothing");
21
+ });
22
+ });
23
+ describe("when the schema has changed", () => {
24
+ it("writes the file", async () => {
25
+ mock.buildTestDirectorySchema.mockResolvedValue("new schema");
26
+ mock.readFileSync.mockImplementation(() => "old schema");
27
+ const result = await updateTestdirectorySchemaFile({
28
+ testEnvironmentPath: "path",
29
+ outputFilePath: "path",
30
+ });
31
+ expect(result).toBe("updated");
32
+ expect(mock.writeFileSync).toHaveBeenCalledWith("path", "new schema");
33
+ });
34
+ });
@@ -1,16 +1,14 @@
1
1
  import type { TerminalApplication } from "./TerminalApplication";
2
+ export type StartableApplication = Pick<TerminalApplication, "write" | "processId" | "killAndWait">;
2
3
  /** A testable application that can be started, killed, and given input. For a
3
4
  * single instance of this interface, only a single instance can be running at
4
- * a time (1 to 1 mapping).
5
- *
6
- * @typeParam T The type of context the tests should have, e.g. information
7
- * about a custom directory that the application is running in.
8
- *
5
+ * a time.
9
6
  */
10
- export declare abstract class DisposableSingleApplication implements AsyncDisposable {
11
- protected application: TerminalApplication | undefined;
12
- killCurrent(): Promise<void>;
7
+ export declare class DisposableSingleApplication implements AsyncDisposable {
8
+ protected application: StartableApplication | undefined;
9
+ startNextAndKillCurrent(startNext: () => Promise<StartableApplication>): Promise<void>;
13
10
  write(input: string): Promise<void>;
14
11
  processId(): number | undefined;
12
+ /** Kill the current application if it exists. */
15
13
  [Symbol.asyncDispose](): Promise<void>;
16
14
  }
@@ -1,27 +1,27 @@
1
+ import assert from "assert";
1
2
  /** A testable application that can be started, killed, and given input. For a
2
3
  * single instance of this interface, only a single instance can be running at
3
- * a time (1 to 1 mapping).
4
- *
5
- * @typeParam T The type of context the tests should have, e.g. information
6
- * about a custom directory that the application is running in.
7
- *
4
+ * a time.
8
5
  */
9
6
  export class DisposableSingleApplication {
10
7
  application;
11
- async killCurrent() {
12
- await this.application?.killAndWait();
8
+ async startNextAndKillCurrent(startNext) {
9
+ await this[Symbol.asyncDispose]();
10
+ this.application = await startNext();
13
11
  }
14
12
  async write(input) {
15
- return this.application?.write(input);
13
+ assert(this.application, "The application not started yet. It makes no sense to write to it, so this looks like a bug.");
14
+ this.application.write(input);
16
15
  }
17
16
  processId() {
18
17
  return this.application?.processId;
19
18
  }
19
+ /** Kill the current application if it exists. */
20
20
  async [Symbol.asyncDispose]() {
21
21
  if (this.processId() === undefined) {
22
22
  return;
23
23
  }
24
24
  console.log(`Killing current application ${this.processId()}...`);
25
- await this.killCurrent();
25
+ await this.application?.killAndWait();
26
26
  }
27
27
  }
@@ -0,0 +1,55 @@
1
+ import { DisposableSingleApplication } from "./DisposableSingleApplication";
2
+ vi.spyOn(console, "log").mockImplementation(vi.fn());
3
+ class TestDisposableSingleApplication extends DisposableSingleApplication {
4
+ getApplication() {
5
+ return this.application;
6
+ }
7
+ }
8
+ const fakeApp = {
9
+ processId: 123,
10
+ write: vi.fn(),
11
+ killAndWait: vi.fn(),
12
+ };
13
+ describe("DisposableSingleApplication", () => {
14
+ it("has no application when created", () => {
15
+ const app = new TestDisposableSingleApplication();
16
+ expect(app.processId()).toBeUndefined();
17
+ expect(app.getApplication()).toBeUndefined();
18
+ });
19
+ it("can start an application", async () => {
20
+ const app = new TestDisposableSingleApplication();
21
+ await app.startNextAndKillCurrent(async () => fakeApp);
22
+ expect(app.processId()).toBe(123);
23
+ expect(app.getApplication()).toBe(fakeApp);
24
+ });
25
+ it("can write to the application", async () => {
26
+ const app = new TestDisposableSingleApplication();
27
+ await app.startNextAndKillCurrent(async () => fakeApp);
28
+ await app.write("hello");
29
+ expect(fakeApp.write).toHaveBeenCalledWith("hello");
30
+ });
31
+ it("fails to write if the application is not started", async () => {
32
+ // there is no need to support soft failing in the ui, so we do hard
33
+ // failing to make this error obvious
34
+ const app = new TestDisposableSingleApplication();
35
+ await expect(app.write("hello")).rejects.toThrowErrorMatchingInlineSnapshot(`[AssertionError: The application not started yet. It makes no sense to write to it, so this looks like a bug.]`);
36
+ });
37
+ describe("disposing", () => {
38
+ it("disposes the application when disposed", async () => {
39
+ // it's important to make sure there are no dangling applications when
40
+ // starting new tests or ending the user session entirely and closing the
41
+ // application
42
+ const app = new TestDisposableSingleApplication();
43
+ await app.startNextAndKillCurrent(async () => fakeApp);
44
+ expect(app.getApplication()).toBe(fakeApp);
45
+ await app[Symbol.asyncDispose]();
46
+ expect(fakeApp.killAndWait).toHaveBeenCalledOnce();
47
+ });
48
+ it("does nothing if there is no application to dispose", async () => {
49
+ const app = new TestDisposableSingleApplication();
50
+ expect(app.getApplication()).toBeUndefined();
51
+ expect(app.processId()).toBeUndefined();
52
+ expect(() => app[Symbol.asyncDispose]()).not.toThrow();
53
+ });
54
+ });
55
+ });
@@ -1,6 +1,7 @@
1
1
  import type winston from "winston";
2
2
  import type { ITerminalDimensions } from "@xterm/addon-fit";
3
- export declare class TerminalApplication {
3
+ import type { StartableApplication } from "./DisposableSingleApplication";
4
+ export declare class TerminalApplication implements StartableApplication {
4
5
  private readonly subProcess;
5
6
  readonly onStdoutOrStderr: (data: string) => void;
6
7
  readonly processId: number;
@@ -0,0 +1,2 @@
1
+ import type EventEmitter from "events";
2
+ export declare function convertEventEmitterToAsyncGenerator(emitter: EventEmitter, eventName: string): AsyncGenerator<string, void, unknown>;
@@ -0,0 +1,8 @@
1
+ export async function* convertEventEmitterToAsyncGenerator(emitter, eventName) {
2
+ // eslint-disable-next-line @typescript-eslint/no-unnecessary-condition
3
+ while (true) {
4
+ yield await new Promise(resolve => {
5
+ emitter.once(eventName, resolve);
6
+ });
7
+ }
8
+ }
@@ -0,0 +1 @@
1
+ export {};
@@ -0,0 +1,41 @@
1
+ import { EventEmitter } from "stream";
2
+ import { convertEventEmitterToAsyncGenerator } from "./generator";
3
+ describe("when a listener is attached", () => {
4
+ it("forwards events from an EventEmitter to an async generator", async () => {
5
+ const eventEmitter = new EventEmitter();
6
+ const generator = convertEventEmitterToAsyncGenerator(eventEmitter, "test");
7
+ {
8
+ const promise = generator.next();
9
+ eventEmitter.emit("test", "message");
10
+ expect(await promise).toMatchInlineSnapshot(`
11
+ {
12
+ "done": false,
13
+ "value": "message",
14
+ }
15
+ `);
16
+ const promise2 = generator.next();
17
+ eventEmitter.emit("test", "message2");
18
+ expect(await promise2).toMatchInlineSnapshot(`
19
+ {
20
+ "done": false,
21
+ "value": "message2",
22
+ }
23
+ `);
24
+ }
25
+ });
26
+ });
27
+ describe("when no listener is attached, messages are lost permanently", () => {
28
+ it("does not resend lost events after a subscriber attaches", async () => {
29
+ const eventEmitter = new EventEmitter();
30
+ eventEmitter.emit("test", "this message should be lost");
31
+ const generator = convertEventEmitterToAsyncGenerator(eventEmitter, "test");
32
+ const promise = generator.next();
33
+ eventEmitter.emit("test", "new message that will be received");
34
+ expect(await promise).toMatchInlineSnapshot(`
35
+ {
36
+ "done": false,
37
+ "value": "new message that will be received",
38
+ }
39
+ `);
40
+ });
41
+ });