From 3f3bc6301b6f7302005ad119f7d702b4a668cd38 Mon Sep 17 00:00:00 2001 From: bcoll Date: Tue, 5 Mar 2024 10:18:45 +0000 Subject: [PATCH] ci: disable file parallelism for fixture tests Some of the `get-{bindings,platform}-proxy` tests have been flaking in CI recently. I think this is because they're trying to connect to the dev registry, but the dev registry was started by an earlier test that just finished. This leads to socket errors like `other side closed`. The proper solution to this problem is to start a dev registry for each test (file). We could do this by assigning a different dev registry port for each test (file/project). In the meantime, this change disables parallelism for the fixture tests, ensuring if a dev registry is started, it is definitely shut down before another test begins. --- package.json | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/package.json b/package.json index 85aa5d0dc52f..b443304364e4 100644 --- a/package.json +++ b/package.json @@ -19,7 +19,7 @@ "fix": "pnpm run prettify && dotenv -- turbo check:lint -- --fix", "prettify": "prettier . --write --ignore-unknown", "test": "dotenv -- turbo test", - "test:ci": "vitest run && dotenv -- turbo test:ci --filter=wrangler --filter=miniflare --filter=kv-asset-handler --filter=@cloudflare/vitest-pool-workers", + "test:ci": "vitest run --no-file-parallelism && dotenv -- turbo test:ci --filter=wrangler --filter=miniflare --filter=kv-asset-handler --filter=@cloudflare/vitest-pool-workers", "test:watch": "turbo test:watch", "type:tests": "dotenv -- turbo type:tests", "gen:package": "turbo gen package" @@ -53,7 +53,7 @@ "node": ">=16.17.0" }, "volta": { - "node": "16.13.0" + "node": "16.17.0" }, "pnpm": { "peerDependencyRules": {