From db84cb5ccbfdbf6a74bbdc943c5fceb3bd399ebd Mon Sep 17 00:00:00 2001 From: "Michael B. Gale" Date: Fri, 15 May 2026 11:22:17 +0100 Subject: [PATCH] Remove outdated comments for `analyze-action` tests --- src/analyze-action-env.test.ts | 7 ------- src/analyze-action-input.test.ts | 7 ------- 2 files changed, 14 deletions(-) diff --git a/src/analyze-action-env.test.ts b/src/analyze-action-env.test.ts index 93992c4a8..bcdcf9723 100644 --- a/src/analyze-action-env.test.ts +++ b/src/analyze-action-env.test.ts @@ -17,13 +17,6 @@ import * as util from "./util"; setupTests(test); -// This test needs to be in its own file so that ava would run it in its own -// nodejs process. The code being tested is in analyze-action.ts, which runs -// immediately on load. So the file needs to be loaded during part of the test, -// and that can happen only once per nodejs process. If multiple such tests are -// in the same test file, ava would run them in the same nodejs process, and all -// but the first test would fail. - test("analyze action with RAM & threads from environment variables", async (t) => { // This test frequently times out on Windows with the default timeout, so we bump // it a bit to 20s. diff --git a/src/analyze-action-input.test.ts b/src/analyze-action-input.test.ts index b0c2f90c0..9aee1a844 100644 --- a/src/analyze-action-input.test.ts +++ b/src/analyze-action-input.test.ts @@ -17,13 +17,6 @@ import * as util from "./util"; setupTests(test); -// This test needs to be in its own file so that ava would run it in its own -// nodejs process. The code being tested is in analyze-action.ts, which runs -// immediately on load. So the file needs to be loaded during part of the test, -// and that can happen only once per nodejs process. If multiple such tests are -// in the same test file, ava would run them in the same nodejs process, and all -// but the first test would fail. - test("analyze action with RAM & threads from action inputs", async (t) => { t.timeout(1000 * 20); await util.withTmpDir(async (tmpDir) => {