4 files changed, 97 insertions, 18 deletions

diff --git a/packages/main/src/infrastructure/config/impl/__tests__/ConfigFile.integ.test.ts b/packages/main/src/infrastructure/config/impl/__tests__/ConfigFile.integ.test.ts
index dd4aaaa..c443d99 100644
--- a/packages/main/src/infrastructure/config/impl/__tests__/ConfigFile.integ.test.ts
+++ b/packages/main/src/infrastructure/config/impl/__tests__/ConfigFile.integ.test.ts
@@ -29,10 +29,11 @@ import {
   utimes,
   writeFile,
 } from 'node:fs/promises';
-import { tmpdir } from 'node:os';
+import { platform, tmpdir, userInfo } from 'node:os';
 import path from 'node:path';
 
 import { jest } from '@jest/globals';
+import { testIf } from '@sophie/test-utils';
 import { mocked } from 'jest-mock';
 
 import Disposer from '../../../../utils/Disposer.js';
@@ -249,19 +250,24 @@ describe('watchConfig', () => {
       expect(callback).toHaveBeenCalled();
     });
 
-    test('handles other filesystem errors', async () => {
-      const { mode } = await stat(userDataDir!);
-      await writeFile(configFilePath, 'Hi Mars!', 'utf8');
-      // Remove permission to force a filesystem error.
-      // eslint-disable-next-line no-bitwise -- Compute reduced permissions.
-      await chmod(userDataDir!, mode & 0o666);
-      try {
-        await catchUpWithFilesystem();
-        expect(callback).not.toHaveBeenCalled();
-      } finally {
-        await chmod(userDataDir!, mode);
-      }
-    });
+    // We can only cause a filesystem error by changing permissions if we run on a POSIX-like
+    // system and we aren't root (i.e., not in CI).
+    testIf(platform() !== 'win32' && userInfo().uid !== 0)(
+      'handles other filesystem errors',
+      async () => {
+        const { mode } = await stat(userDataDir!);
+        await writeFile(configFilePath, 'Hi Mars!', 'utf8');
+        // Remove permission to force a filesystem error.
+        // eslint-disable-next-line no-bitwise -- Compute reduced permissions.
+        await chmod(userDataDir!, mode & 0o666);
+        try {
+          await catchUpWithFilesystem();
+          expect(callback).not.toHaveBeenCalled();
+        } finally {
+          await chmod(userDataDir!, mode);
+        }
+      },
+    );
 
     test('does not notify when the modification date is prior to the last write', async () => {
       await repository.writeConfig('Hello World!');
diff --git a/packages/test-utils/package.json b/packages/test-utils/package.json
index 93f1626..8243cc5 100644
--- a/packages/test-utils/package.json
+++ b/packages/test-utils/package.json
@@ -10,6 +10,10 @@
     "types": "yarn g:types"
   },
   "dependencies": {
+    "@jest/globals": "^28.1.0",
     "type-fest": "^2.13.0"
+  },
+  "devDependencies": {
+    "@jest/types": "^28.1.0"
   }
 }
diff --git a/packages/test-utils/src/index.ts b/packages/test-utils/src/index.ts
index 52ac98d..528b872 100644
--- a/packages/test-utils/src/index.ts
+++ b/packages/test-utils/src/index.ts
@@ -18,8 +18,7 @@
  * SPDX-License-Identifier: AGPL-3.0-only
  */
 
-/*
-  eslint-disable-next-line import/prefer-default-export --
-  Keep API consistent for more helpers in the future.
-*/
 export { default as fake } from './fake.js';
+
+export type { TestIf, TestIfConcurrent } from './testIf.js';
+export { default as testIf } from './testIf.js';
diff --git a/packages/test-utils/src/testIf.ts b/packages/test-utils/src/testIf.ts
new file mode 100644
index 0000000..2309ddc
--- /dev/null
+++ b/packages/test-utils/src/testIf.ts
@@ -0,0 +1,70 @@
+/*
+ * Copyright (C)  2022 Kristóf Marussy <kristof@marussy.com>
+ *
+ * This file is part of Sophie.
+ *
+ * Sophie is free software: you can redistribute it and/or modify
+ * it under the terms of the GNU Affero General Public License as
+ * published by the Free Software Foundation, version 3.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * GNU Affero General Public License for more details.
+ *
+ * You should have received a copy of the GNU Affero General Public License
+ * along with this program.  If not, see <https://www.gnu.org/licenses/>.
+ *
+ * SPDX-License-Identifier: AGPL-3.0-only
+ */
+
+import { test } from '@jest/globals';
+import type { Global } from '@jest/types';
+
+export interface TestIfConcurrent {
+  (condition: boolean): Global.ItConcurrentBase;
+
+  only(condition: boolean): Global.ItConcurrentBase;
+
+  skip(condition: boolean): Global.ItConcurrentBase;
+}
+
+export interface TestIf {
+  (condition: boolean): Global.ItBase;
+
+  only(condition: boolean): Global.ItBase;
+
+  skip(condition: boolean): Global.ItBase;
+
+  concurrent: TestIfConcurrent;
+}
+
+const testIfConcurrent: TestIfConcurrent = function concurrent(
+  condition: boolean,
+) {
+  return condition ? test.concurrent : test.concurrent.skip;
+};
+
+testIfConcurrent.only = function only() {
+  return test.concurrent.only;
+};
+
+testIfConcurrent.skip = function skip() {
+  return test.concurrent.skip;
+};
+
+const testIf: TestIf = function testIf(condition: boolean) {
+  return condition ? test : test.skip;
+};
+
+testIf.only = function only() {
+  return test.only;
+};
+
+testIf.skip = function skip() {
+  return test.skip;
+};
+
+testIf.concurrent = testIfConcurrent;
+
+export default testIf;