test: tweak other test to try and pass on windows

Author: james2doyle

tests/tools/screenshot.test.ts

@@ -4,7 +4,7 @@
  * SPDX-License-Identifier: Apache-2.0
  */
 import assert from 'node:assert';
-import {rm, stat, mkdir, chmod} from 'node:fs/promises';
+import {rm, stat, mkdir, chmod, writeFile} from 'node:fs/promises';
 import {tmpdir} from 'node:os';
 import {join} from 'node:path';
 import {describe, it} from 'node:test';
@@ -145,27 +145,53 @@ describe('screenshot', () => {
     });
 
     it('with unwritable filePath', async () => {
-      const dir = join(tmpdir(), 'readonly-dir-for-screenshot-test');
-      await mkdir(dir, {recursive: true});
-      await chmod(dir, 0o500);
-      const filePath = join(dir, 'test-screenshot.png');
+      if (process.platform === 'win32') {
+        const filePath = join(tmpdir(), 'readonly-file-for-screenshot-test.png');
+        // Create the file and make it read-only.
+        await writeFile(filePath, '');
+        await chmod(filePath, 0o400);
 
-      try {
-        await withBrowser(async (response, context) => {
-          const fixture = screenshots.basic;
-          const page = context.getSelectedPage();
-          await page.setContent(fixture.html);
-          await assert.rejects(
-            screenshot.handler(
-              {params: {format: 'png', filePath}},
-              response,
-              context,
-            ),
-          );
-        });
-      } finally {
-        await chmod(dir, 0o700);
-        await rm(dir, {recursive: true, force: true});
+        try {
+          await withBrowser(async (response, context) => {
+            const fixture = screenshots.basic;
+            const page = context.getSelectedPage();
+            await page.setContent(fixture.html);
+            await assert.rejects(
+              screenshot.handler(
+                {params: {format: 'png', filePath}},
+                response,
+                context,
+              ),
+            );
+          });
+        } finally {
+          // Make the file writable again so it can be deleted.
+          await chmod(filePath, 0o600);
+          await rm(filePath, {force: true});
+        }
+      } else {
+        const dir = join(tmpdir(), 'readonly-dir-for-screenshot-test');
+        await mkdir(dir, {recursive: true});
+        await chmod(dir, 0o500);
+        const filePath = join(dir, 'test-screenshot.png');
+
+        try {
+          await withBrowser(async (response, context) => {
+            const fixture = screenshots.basic;
+            const page = context.getSelectedPage();
+            await page.setContent(fixture.html);
+            await assert.rejects(
+              screenshot.handler(
+                {params: {format: 'png', filePath}},
+                response,
+                context,
+              ),
+            );
+          });
+        } finally {
+          await chmod(dir, 0o700);
+          await rm(dir, {recursive: true, force: true});
+        }
       }
     });