From c17c582818b615c64c0d5e18913737b824dcbcfc Mon Sep 17 00:00:00 2001 From: Bastiaan <17990055+bbassie@users.noreply.github.com> Date: Wed, 19 Aug 2026 16:00:55 +0200 Subject: [PATCH] Download Factorio in host tests to temp/test instead of test/file (#972) * Ignore Factorio download directory created by host tests The live-API download test in test/host/server.js writes a full headless Factorio install (~140 MB) to test/file/factorioDownload and does not remove it, which then shows up as untracked files. Co-Authored-By: Claude Fable 5 * Download Factorio in host tests to temp/test instead of test/file The live-API download test wrote a full headless install (~140 MB) into test/file/factorioDownload, which is tracked test fixture territory and showed up as untracked files. temp/ is already gitignored and is where the other tests put generated data. Co-Authored-By: Claude Fable 5 --------- Co-authored-by: Claude Fable 5 --- test/host/server.js | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/test/host/server.js b/test/host/server.js index adbec5e1..94bf78d7 100644 --- a/test/host/server.js +++ b/test/host/server.js @@ -483,7 +483,7 @@ describe("host/server", function() { this.skip(); } - server._factorioDir = path.join("test", "file", "factorioDownload"); + server._factorioDir = path.join("temp", "test", "factorioDownload"); server._targetVersion = "latest"; global.fetch = _fetch; await fs.rm(server._factorioDir, { force: true, recursive: true, maxRetries: 10 });