add tests to ensure ignored files are ignored by the child process as…

… well

spec/Supervisor.spec.ts

@@ -42,7 +42,7 @@ test("it proxies ipc messages", async () => {
     });
   });
 
-  for (let number of messagesToChild) {
+  for (const number of messagesToChild) {
     child.send(number);
   }
 

spec/SwcCompiler.test.ts

@@ -1,40 +1,86 @@
-import { MissingDestinationError, SwcCompiler } from '../src/SwcCompiler'
 import * as fs from "fs/promises";
 import os from "os";
 import path from "path";
+import { MissingDestinationError, SwcCompiler } from "../src/SwcCompiler";
+import { log } from "../src/utils";
 
 const compile = async (filename: string, root = "fixtures/src") => {
   const workDir = await fs.mkdtemp(path.join(os.tmpdir(), "wds-test"));
   const rootDir = path.join(__dirname, root);
   const fullPath = path.join(rootDir, filename);
 
-  const compiler = new SwcCompiler(rootDir, workDir)
+  const compiler = new SwcCompiler(rootDir, workDir);
   await compiler.compile(fullPath);
   const compiledFilePath = (await compiler.fileGroup(fullPath))[fullPath]!;
 
   return await fs.readFile(compiledFilePath, "utf-8");
-}
+};
 
 test("compiles simple files", async () => {
   const content = await compile("./simple.ts");
-  expect(content).toContain('console.log("success")')
+  expect(content).toContain('console.log("success")');
 });
 
 test("throws if the compilation fails", async () => {
   await expect(compile("./failing/failing.ts", "fixtures/failing")).rejects.toThrow(MissingDestinationError);
-})
+});
+
+test("throws if the file is ignored", async () => {
+  let error: MissingDestinationError | null = null;
+  try {
+    await compile("./files_with_config/ignored.ts");
+  } catch (e) {
+    if (e instanceof MissingDestinationError) {
+      error = e;
+    } else {
+      throw e;
+    }
+  }
+
+  expect(error).toBeDefined();
+  expect(error?.ignoredFile).toBeTruthy();
+  expect(error?.message).toMatch(
+    /File .+ignored\.ts is imported but not being built because it is explicitly ignored in the wds project config\. It is being ignored by the provided glob pattern '!ignored\.ts', remove this pattern from the project config or don't import this file to fix./
+  );
+});
+
+test("logs error when a file in group fails compilation but continues", async () => {
+  const errorLogs: any[] = [];
+
+  const mock = jest.spyOn(log, "error").mockImplementation((...args: any[]) => {
+    errorLogs.push(args);
+  });
+
+  const workDir = await fs.mkdtemp(path.join(os.tmpdir(), "wds-test"));
+  const rootDir = path.join(__dirname, "fixtures/failing");
+  const fullPath = path.join(rootDir, "successful.ts");
+  const compiler = new SwcCompiler(rootDir, workDir);
+  await compiler.compile(fullPath);
+  const group = await compiler.fileGroup(fullPath);
+
+  expect(group[fullPath]).toBeDefined();
+  expect(Object.entries(group).filter(([path]) => /.+(bar|successful)\.ts$/.test(path))).toHaveLength(2);
+  const error = errorLogs[0][0];
+  expect(error.code).toBe("GenericFailure");
+  expect(error.message).toMatch(/.+failing\.ts/);
+  expect(error.message).toMatch(/Syntax Error/);
+
+  mock.mockRestore();
+});
 
 test("compiles lazy import", async () => {
   const content = await compile("./lazy_import.ts");
-  expect(content).toContain(`
+  expect(content).toContain(
+    `
 function _child_process() {
-    const data = require(\"child_process\");
+    const data = require("child_process");
     _child_process = function() {
         return data;
     };
     return data;
 }
-`.trim());
+`.trim()
+  );
 });
 
 test("uses the swc config file from wds.js", async () => {

spec/fixtures/failing/bar.ts

@@ -0,0 +1 @@
+export const foo = "foo";

spec/fixtures/failing/successful.ts

@@ -0,0 +1,3 @@
+export function success() {
+  return !!(1 + 1);
+}

spec/fixtures/src/files_with_config/wds.js

@@ -10,5 +10,6 @@ module.exports = {
       "type": "commonjs",
       "strictMode": false
     }
-  }
+  },
+  ignore: ["ignored.ts"]
 }

spec/fixtures/src/wds.js

@@ -13,5 +13,5 @@ module.exports = {
       "strictMode": true,
       "lazy": true
     }
-  }
+  },
 }

spec/wds.spec.ts

@@ -0,0 +1,90 @@
+import assert from "assert";
+import type { ChildProcess } from "child_process";
+import http from "http";
+import path from "path";
+import { Supervisor } from "../src/Supervisor";
+import { wds } from "../src/index";
+
+describe("wds", () => {
+  let cwd: any;
+  let supervisorRestart: any;
+  let socketPath: string;
+
+  const sendCompileRequest = async (filename: string) => {
+    assert(socketPath, "socketPath must be set");
+    const result = await new Promise((resolve, reject) => {
+      const request = http.request({ socketPath, path: "/compile", method: "POST", timeout: 200 }, (resp) => {
+        let data = "";
+        if (resp.statusCode !== 200) {
+          return reject(`Error compiling`);
+        }
+        resp.on("data", (chunk: string) => (data += chunk));
+        resp.on("end", () => resolve(JSON.parse(data).filenames));
+      });
+
+      request.on("error", (error) => {
+        reject(error);
+      });
+      request.write(filename);
+      request.end();
+    });
+
+    return result;
+  };
+
+  beforeEach(() => {
+    cwd = jest.spyOn(process, "cwd").mockImplementation(() => {
+      return path.resolve(__dirname, "fixtures/src/files_with_config");
+    });
+
+    supervisorRestart = jest.spyOn(Supervisor.prototype, "restart").mockImplementation(function () {
+      const self = this as unknown as Supervisor;
+      socketPath = self.socketPath;
+      self.process = {
+        on: jest.fn(),
+      } as unknown as ChildProcess;
+      return self.process;
+    });
+  });
+
+  afterEach(() => {
+    cwd.mockRestore();
+    supervisorRestart.mockRestore();
+  });
+
+  test("server responds to ignored files", async () => {
+    const server = await wds({
+      argv: [],
+      terminalCommands: false,
+      reloadOnChanges: false,
+    });
+    const result = (await sendCompileRequest(path.resolve(__dirname, "fixtures/src/files_with_config/ignored.ts"))) as Record<
+      string,
+      string | { ignored: boolean }
+    >;
+    const compiledKeys = Object.keys(result).filter((k) => /spec\/fixtures\/src\/files_with_config\/ignored\.ts/.test(k));
+    expect(compiledKeys).toHaveLength(1);
+    expect(result[compiledKeys[0]]).toEqual({
+      ignored: true,
+    });
+
+    server.close();
+  });
+
+  test("server responds to included files", async () => {
+    const server = await wds({
+      argv: [],
+      terminalCommands: false,
+      reloadOnChanges: false,
+    });
+    const result = (await sendCompileRequest(path.resolve(__dirname, "fixtures/src/files_with_config/simple.ts"))) as Record<
+      string,
+      string | { ignored: boolean }
+    >;
+    const compiledKeys = Object.keys(result).filter((k) => /spec\/fixtures\/src\/files_with_config\/simple\.ts/.test(k));
+    expect(compiledKeys).toHaveLength(1);
+    expect(typeof result[compiledKeys[0]]).toBe("string");
+
+    server.close();
+  });
+});

src/Supervisor.ts

@@ -49,7 +49,6 @@ export class Supervisor extends EventEmitter {
         ...process.env,
         WDS_SOCKET_PATH: this.socketPath,
         WDS_EXTENSIONS: this.project.config.extensions.join(","),
-        WDS_IGNORE: JSON.stringify(this.project.config.ignore),
       },
       stdio: stdio,
     });

src/index.ts

@@ -148,7 +148,7 @@ export const wds = async (options: RunOptions) => {
 
   if (options.reloadOnChanges) startFilesystemWatcher(project);
   if (options.terminalCommands) startTerminalCommandListener(project);
-  await startIPCServer(serverSocketPath, project);
+  const server = await startIPCServer(serverSocketPath, project);
 
   // kickoff the first child process
   options.reloadOnChanges && log.info(`Supervision starting for command: node ${options.argv.join(" ")}`);
@@ -174,4 +174,6 @@ export const wds = async (options: RunOptions) => {
     logShutdown("shutting down project since it's no longer needed...");
     project.shutdown(code ?? 1);
   });
+
+  return server;
 };

tsconfig.json

@@ -22,6 +22,6 @@
     "node_modules/@types/jest/index.d.ts"
   ],
   "exclude": [
-    "spec/fixtures",
+    "spec/fixtures"
   ]
 }