@forwardimpact/libmock 0.1.3 → 0.1.4

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@forwardimpact/libmock",
-  "version": "0.1.3",
+  "version": "0.1.4",
   "description": "Shared mocks and test fixtures so every library and service tests the same way.",
   "keywords": [
     "test",

package/src/mock/subprocess.js

@@ -1,3 +1,5 @@
+import { Writable } from "node:stream";
+
 import { spy } from "./spy.js";
 
 function asyncIterableOf(str) {
@@ -9,27 +11,21 @@ function asyncIterableOf(str) {
 }
 
 /**
- * A captured-chunks stub for a spawned child's writable stdin. Records every
- * `write(chunk)` on `chunks`; `end()`/`destroy()` are no-ops. Mirrors the
- * shape `createDefaultSubprocess().spawn` exposes (the child's `node:stream`
- * Writable) closely enough for a supervisor that pipes into it.
+ * A captured-chunks sink for a spawned child's writable stdin. A real
+ * `node:stream` Writable (so it is a valid `pipe()` destination, the way a
+ * supervisor pipes a service's output into its logger) that records every
+ * written chunk on `chunks` instead of forwarding it anywhere. Mirrors the
+ * `stdin` the production `createDefaultSubprocess().spawn` exposes.
  */
 function createMockStdinSink() {
-  const sink = {
-    chunks: [],
-    write(chunk) {
-      sink.chunks.push(typeof chunk === "string" ? chunk : chunk.toString());
-      return true;
-    },
-    end() {},
-    destroy() {},
-    on() {
-      return sink;
-    },
-    once() {
-      return sink;
+  const chunks = [];
+  const sink = new Writable({
+    write(chunk, _encoding, callback) {
+      chunks.push(typeof chunk === "string" ? chunk : chunk.toString());
+      callback();
     },
-  };
+  });
+  sink.chunks = chunks;
   return sink;
 }