aperx/factorio-clusterio
1
0 Fork 0
mirror of https://github.com/clusterio/clusterio.git synced 2026-11-13 23:14:53 +09:00
Code Issues Packages Projects Releases Wiki Activity

Download Factorio in host tests to temp/test instead of test/file (#972)

Browse Source 
* 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 <noreply@anthropic.com>

* 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 <noreply@anthropic.com>

---------

Co-authored-by: Claude Fable 5 <noreply@anthropic.com>
This commit is contained in:
BastiaanandClaude Fable 5 authored and GitHub committed 2026-08-19 15:00:55 +01:00
1 parent 0be4fd77e5
commit c17c582818
1 file changed
+1 -1
Download Patch File Download Diff File Expand all files Collapse all files
test/host/server.js
+1 -1
View File
@@ -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 });