From 143d4bb1715f248f7471ddeda5011ba160ff8ebd Mon Sep 17 00:00:00 2001 From: Chris Huber Date: Tue, 27 Oct 2026 13:40:43 -0400 Subject: [PATCH 1/3] Give the concurrent server-files install test time on Windows CI --- .../lib/dependency-management/tests/setup-server-files.test.ts | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/apps/cli/lib/dependency-management/tests/setup-server-files.test.ts b/apps/cli/lib/dependency-management/tests/setup-server-files.test.ts index d98815944e..525766eba3 100644 --- a/apps/cli/lib/dependency-management/tests/setup-server-files.test.ts +++ b/apps/cli/lib/dependency-management/tests/setup-server-files.test.ts @@ -16,6 +16,7 @@ const count = ( dir: string ) => fs.readdirSync( dir, { recursive: true } ).leng // The app runs several CLI commands at once, and each installs the bundled WordPress. Any reader // that finds version.php copies `latest` into a new site, so it must never see a partial copy. +// Eight concurrent copies of 400 files can take longer than the default 5 s on Windows CI. it( 'never shows a partial latest WordPress while several processes install it', async () => { const bundled = path.join( root, 'wp-files', 'latest', 'wordpress', 'wp-includes' ); fs.mkdirSync( bundled, { recursive: true } ); @@ -46,4 +47,4 @@ it( 'never shows a partial latest WordPress while several processes install it', expect( count( latest ) ).toBe( count( path.dirname( bundled ) ) ); expect( fs.readdirSync( path.dirname( latest ) ) ).toEqual( [ 'latest' ] ); fs.rmSync( root, { recursive: true, force: true } ); -} ); +}, 30_000 ); From 47337ba622748791ac721b71852711d2baf6e013 Mon Sep 17 00:00:00 2001 From: Chris Huber Date: Tue, 27 Oct 2026 13:50:28 -0400 Subject: [PATCH 2/3] Shrink the concurrent server-files install test instead of extending its timeout --- .../dependency-management/tests/setup-server-files.test.ts | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/apps/cli/lib/dependency-management/tests/setup-server-files.test.ts b/apps/cli/lib/dependency-management/tests/setup-server-files.test.ts index 525766eba3..547b5108b8 100644 --- a/apps/cli/lib/dependency-management/tests/setup-server-files.test.ts +++ b/apps/cli/lib/dependency-management/tests/setup-server-files.test.ts @@ -21,7 +21,7 @@ it( 'never shows a partial latest WordPress while several processes install it', const bundled = path.join( root, 'wp-files', 'latest', 'wordpress', 'wp-includes' ); fs.mkdirSync( bundled, { recursive: true } ); fs.writeFileSync( path.join( bundled, 'version.php' ), " setImmediate( resolve ) ); } } )(); - await Promise.all( Array.from( { length: 8 }, () => setupServerFiles() ) ); + await Promise.all( Array.from( { length: 2 }, () => setupServerFiles() ) ); installing = false; await watch; From 65745fd5eed878b4ec1371d9e94f5ddec8854201 Mon Sep 17 00:00:00 2001 From: Chris Huber Date: Tue, 27 Oct 2026 13:50:35 -0400 Subject: [PATCH 3/3] Drop the extended timeout --- .../lib/dependency-management/tests/setup-server-files.test.ts | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/apps/cli/lib/dependency-management/tests/setup-server-files.test.ts b/apps/cli/lib/dependency-management/tests/setup-server-files.test.ts index 547b5108b8..60449d6913 100644 --- a/apps/cli/lib/dependency-management/tests/setup-server-files.test.ts +++ b/apps/cli/lib/dependency-management/tests/setup-server-files.test.ts @@ -16,7 +16,6 @@ const count = ( dir: string ) => fs.readdirSync( dir, { recursive: true } ).leng // The app runs several CLI commands at once, and each installs the bundled WordPress. Any reader // that finds version.php copies `latest` into a new site, so it must never see a partial copy. -// Eight concurrent copies of 400 files can take longer than the default 5 s on Windows CI. it( 'never shows a partial latest WordPress while several processes install it', async () => { const bundled = path.join( root, 'wp-files', 'latest', 'wordpress', 'wp-includes' ); fs.mkdirSync( bundled, { recursive: true } ); @@ -47,4 +46,4 @@ it( 'never shows a partial latest WordPress while several processes install it', expect( count( latest ) ).toBe( count( path.dirname( bundled ) ) ); expect( fs.readdirSync( path.dirname( latest ) ) ).toEqual( [ 'latest' ] ); fs.rmSync( root, { recursive: true, force: true } ); -}, 30_000 ); +} );