test: use test instead of it

Signed-off-by: Kristóf Marussy <kristof@marussy.com>

10 files changed, 69 insertions, 69 deletions

diff --git a/config/jest.config.base.cjs b/config/jest.config.base.cjs
index ee85d66..7bee8dd 100644
--- a/config/jest.config.base.cjs
+++ b/config/jest.config.base.cjs
@@ -5,7 +5,7 @@ module.exports = {
   transform: {
     '\\.[jt]sx?$': path.join(__dirname, 'jestEsbuildCjsTransformer.cjs'),
   },
-  testRegex: '(\\.|/)integ\\.spec\\.ts$',
+  testRegex: '(\\.|/)integ\\.test\\.ts$',
   transformIgnorePatterns: ['/node_modules/(?!(chalk|lodash-es)/)'],
   moduleNameMapper: {
     '^@sophie/(.+)$': path.join(__dirname, '../packages/$1/src/index.ts'),
diff --git a/config/jest.config.base.js b/config/jest.config.base.js
index 32d2ecb..7524ab3 100644
--- a/config/jest.config.base.js
+++ b/config/jest.config.base.js
@@ -9,7 +9,7 @@ export default {
   transform: {
     '\\.tsx?$': path.join(thisDir, 'jestEsbuildTransformer.js'),
   },
-  testRegex: '((?<!integ)\\.|/)spec\\.tsx?$',
+  testRegex: '((?<!integ)\\.|/)test\\.tsx?$',
   extensionsToTreatAsEsm: ['.ts', '.tsx'],
   moduleNameMapper: {
     '^@sophie/(.+)$': path.join(thisDir, '../packages/$1/src/index.ts'),
diff --git a/packages/main/src/infrastructure/electron/impl/__tests__/electron.integ.spec.ts b/packages/main/src/infrastructure/electron/impl/__tests__/electron.integ.test.ts
index f6bd440..67cf689 100644
--- a/packages/main/src/infrastructure/electron/impl/__tests__/electron.integ.spec.ts
+++ b/packages/main/src/infrastructure/electron/impl/__tests__/electron.integ.test.ts
@@ -2,7 +2,7 @@ import { BrowserWindow } from 'electron';
 
 import '../ElectronMainWindow';
 
-it('should create a BrowserWindow', async () => {
+test('create a BrowserWindow', async () => {
   const w = new BrowserWindow();
   await expect(w.loadURL('https://example.org')).resolves.toBeUndefined();
   w.close();
diff --git a/packages/main/src/infrastructure/electron/impl/__tests__/hardenSession.spec.ts b/packages/main/src/infrastructure/electron/impl/__tests__/hardenSession.test.ts
index dc15d68..7b70d10 100644
--- a/packages/main/src/infrastructure/electron/impl/__tests__/hardenSession.spec.ts
+++ b/packages/main/src/infrastructure/electron/impl/__tests__/hardenSession.test.ts
@@ -94,14 +94,14 @@ beforeEach(() => {
   onBeforeRequest = undefined;
 });
 
-it('should set permission request and before request handlers', () => {
+test('set permission request and before request handlers', () => {
   hardenSession(getFakeResources(false), false, fakeSession);
   expect(permissionRequestHandler).toBeDefined();
   expect(onBeforeRequest).toBeDefined();
 });
 
-it.each(permissions.map((permission) => [permission]))(
-  'should reject %s permission requests',
+test.each(permissions.map((permission) => [permission]))(
+  'reject %s permission requests',
   (permission: Permission) => {
     hardenSession(getFakeResources(false), false, fakeSession);
     const callback = jest.fn();
@@ -114,7 +114,7 @@ it.each(permissions.map((permission) => [permission]))(
   },
 );
 
-it.each([
+test.each([
   [
     false,
     'GET',
diff --git a/packages/main/src/infrastructure/electron/impl/__tests__/lockWebContentsToFile.spec.ts b/packages/main/src/infrastructure/electron/impl/__tests__/lockWebContentsToFile.test.ts
index c9aeaab..47525d3 100644
--- a/packages/main/src/infrastructure/electron/impl/__tests__/lockWebContentsToFile.spec.ts
+++ b/packages/main/src/infrastructure/electron/impl/__tests__/lockWebContentsToFile.test.ts
@@ -12,7 +12,7 @@
  * 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
+ * You 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
@@ -85,16 +85,16 @@ describe('when loadURL does not throw', () => {
     await lockWebContentsToFile(fakeResources, 'index.html', fakeWebContents);
   });
 
-  it('should load the specified file', () => {
+  test('loads the specified file', () => {
     expect(fakeWebContents.loadURL).toHaveBeenCalledWith(urlToLoad);
   });
 
-  it('should set up will navigate and window open listeners', () => {
+  test('sets up will navigate and window open listeners', () => {
     expect(willNavigate).toBeDefined();
     expect(windowOpenHandler).toBeDefined();
   });
 
-  it('should prevent opening a window', () => {
+  test('prevents opening a window', () => {
     const { action } = windowOpenHandler!({
       url: 'https://example.com',
       frameName: 'newWindow',
@@ -108,12 +108,12 @@ describe('when loadURL does not throw', () => {
     expect(action).toBe('deny');
   });
 
-  it('should allow navigation to the loaded URL', () => {
+  test('allows navigation to the loaded URL', () => {
     willNavigate!(event, urlToLoad);
     expect(event.preventDefault).not.toHaveBeenCalled();
   });
 
-  it('should not allow navigation to another URL', () => {
+  test('does not allow navigation to another URL', () => {
     willNavigate!(
       event,
       'file:///opt/sophie/resources/app.asar/packages/renderer/not-allowed.html',
@@ -132,7 +132,7 @@ describe('when loadURL throws', () => {
   describe('when the URL points at a file', () => {
     const fakeResources = createFakeResources('http://localhost:3000');
 
-    it('should swallow ERR_ABORTED errors', async () => {
+    test('swallows ERR_ABORTED errors', async () => {
       const error = createAbortedError();
       mocked(fakeWebContents.loadURL).mockRejectedValueOnce(error);
       await expect(
@@ -140,7 +140,7 @@ describe('when loadURL throws', () => {
       ).resolves.not.toThrow();
     });
 
-    it('should pass through other errors', async () => {
+    test('passes through other errors', async () => {
       mocked(fakeWebContents.loadURL).mockRejectedValueOnce(
         new Error('other error'),
       );
@@ -153,7 +153,7 @@ describe('when loadURL throws', () => {
   describe('when the URL points at a local server', () => {
     const fakeResources = createFakeResources(filePrefix);
 
-    it('should pass through ERR_ABORTED errors', async () => {
+    test('passes through ERR_ABORTED errors', async () => {
       const error = createAbortedError();
       mocked(fakeWebContents.loadURL).mockRejectedValueOnce(error);
       await expect(
@@ -161,7 +161,7 @@ describe('when loadURL throws', () => {
       ).rejects.toBeInstanceOf(Error);
     });
 
-    it('should pass through other errors', async () => {
+    test('passes through other errors', async () => {
       mocked(fakeWebContents.loadURL).mockRejectedValueOnce(
         new Error('other error'),
       );
diff --git a/packages/main/src/infrastructure/resources/impl/__tests__/getDistResources.spec.ts b/packages/main/src/infrastructure/resources/impl/__tests__/getDistResources.test.ts
index 8040601..649536b 100644
--- a/packages/main/src/infrastructure/resources/impl/__tests__/getDistResources.spec.ts
+++ b/packages/main/src/infrastructure/resources/impl/__tests__/getDistResources.test.ts
@@ -81,23 +81,23 @@ describe.each([
       resources = getDistResources(devMode, thisDir, devServerURL);
     });
 
-    it('getPath should return the path to the requested resource', () => {
+    test('getPath returns the path to the requested resource', () => {
       const path = resources.getPath('preload', 'index.cjs');
       expect(path).toBe(preloadIndexPath);
     });
 
-    it('getFileURL should return the file URL to the requested resource', () => {
+    test('getFileURL returns the file URL to the requested resource', () => {
       const url = resources.getFileURL('preload', 'index.cjs');
       expect(url).toBe(preloadIndexFileURL);
     });
 
     describe('getRendererURL', () => {
-      it('should return the URL to the requested resource', () => {
+      test('returns the URL to the requested resource', () => {
         const url = resources.getRendererURL('index.html');
         expect(url).toBe(rendererIndexURL);
       });
 
-      it('should return the root URL', () => {
+      test('returns the root URL', () => {
         const url = resources.getRendererURL('/');
         expect(url).toBe(rendererRootURL);
       });
diff --git a/packages/main/src/reactions/__tests__/synchronizeConfig.spec.ts b/packages/main/src/reactions/__tests__/synchronizeConfig.test.ts
index 403a608..f0e88c1 100644
--- a/packages/main/src/reactions/__tests__/synchronizeConfig.spec.ts
+++ b/packages/main/src/reactions/__tests__/synchronizeConfig.test.ts
@@ -51,12 +51,12 @@ describe('when synchronizeializing', () => {
       });
     });
 
-    it('should create a new config file', async () => {
+    test('should create a new config file', async () => {
       await synchronizeConfig(store, repository);
       expect(repository.writeConfig).toHaveBeenCalledTimes(1);
     });
 
-    it('should bail if there is an an error creating the config file', async () => {
+    test('should bail if there is an an error creating the config file', async () => {
       mocked(repository.writeConfig).mockRejectedValue(new Error('boo'));
       await expect(() =>
         synchronizeConfig(store, repository),
@@ -76,13 +76,13 @@ describe('when synchronizeializing', () => {
       });
     });
 
-    it('should read the existing config file is there is one', async () => {
+    test('should read the existing config file is there is one', async () => {
       await synchronizeConfig(store, repository);
       expect(repository.writeConfig).not.toHaveBeenCalled();
       expect(store.settings.themeSource).toBe('dark');
     });
 
-    it('should bail if it cannot set up a watcher', async () => {
+    test('should bail if it cannot set up a watcher', async () => {
       mocked(repository.watchConfig).mockImplementationOnce(() => {
         throw new Error('boo');
       });
@@ -92,7 +92,7 @@ describe('when synchronizeializing', () => {
     });
   });
 
-  it('should update the config file if new details are added during read', async () => {
+  test('should update the config file if new details are added during read', async () => {
     mocked(repository.readConfig).mockResolvedValueOnce({
       found: true,
       contents: `{
@@ -109,7 +109,7 @@ describe('when synchronizeializing', () => {
     expect(repository.writeConfig).toHaveBeenCalledTimes(1);
   });
 
-  it('should not apply an invalid config file but should not overwrite it', async () => {
+  test('should not apply an invalid config file but should not overwrite it', async () => {
     mocked(repository.readConfig).mockResolvedValueOnce({
       found: true,
       contents: `{
@@ -122,7 +122,7 @@ describe('when synchronizeializing', () => {
     expect(repository.writeConfig).not.toHaveBeenCalled();
   });
 
-  it('should bail if it cannot determine whether there is a config file', async () => {
+  test('should bail if it cannot determine whether there is a config file', async () => {
     mocked(repository.readConfig).mockRejectedValue(new Error('boo'));
     await expect(() =>
       synchronizeConfig(store, repository),
@@ -146,7 +146,7 @@ describe('when it has loaded the config', () => {
     jest.resetAllMocks();
   });
 
-  it('should throttle saving changes to the config file', () => {
+  test('should throttle saving changes to the config file', () => {
     mocked(repository.writeConfig).mockResolvedValue();
     store.settings.setThemeSource('dark');
     jest.advanceTimersByTime(lessThanThrottleMs);
@@ -155,14 +155,14 @@ describe('when it has loaded the config', () => {
     expect(repository.writeConfig).toHaveBeenCalledTimes(1);
   });
 
-  it('should handle config writing errors gracefully', () => {
+  test('should handle config writing errors gracefully', () => {
     mocked(repository.writeConfig).mockRejectedValue(new Error('boo'));
     store.settings.setThemeSource('dark');
     jest.advanceTimersByTime(throttleMs);
     expect(repository.writeConfig).toHaveBeenCalledTimes(1);
   });
 
-  it('should read the config file when it has changed', async () => {
+  test('should read the config file when it has changed', async () => {
     mocked(repository.readConfig).mockResolvedValueOnce({
       found: true,
       contents: serializeConfig({
@@ -177,7 +177,7 @@ describe('when it has loaded the config', () => {
     expect(store.settings.themeSource).toBe('dark');
   });
 
-  it('should update the config file if new details are added', async () => {
+  test('should update the config file if new details are added', async () => {
     mocked(repository.readConfig).mockResolvedValueOnce({
       found: true,
       contents: `{
@@ -194,7 +194,7 @@ describe('when it has loaded the config', () => {
     expect(repository.writeConfig).toHaveBeenCalledTimes(1);
   });
 
-  it('should not apply an invalid config file when it has changed but should not overwrite it', async () => {
+  test('should not apply an invalid config file when it has changed but should not overwrite it', async () => {
     mocked(repository.readConfig).mockResolvedValueOnce({
       found: true,
       contents: `{
@@ -207,7 +207,7 @@ describe('when it has loaded the config', () => {
     expect(repository.writeConfig).not.toHaveBeenCalled();
   });
 
-  it('should handle config reading errors gracefully', async () => {
+  test('should handle config reading errors gracefully', async () => {
     mocked(repository.readConfig).mockRejectedValue(new Error('boo'));
     await expect(configChangedCallback()).resolves.not.toThrow();
   });
@@ -217,11 +217,11 @@ describe('when it has loaded the config', () => {
       sutDisposer();
     });
 
-    it('should dispose the watcher', () => {
+    test('should dispose the watcher', () => {
       expect(watcherDisposer).toHaveBeenCalled();
     });
 
-    it('should not listen to store changes any more', () => {
+    test('should not listen to store changes any more', () => {
       store.settings.setThemeSource('dark');
       jest.advanceTimersByTime(2 * throttleMs);
       expect(repository.writeConfig).not.toHaveBeenCalled();
diff --git a/packages/main/src/reactions/__tests__/synchronizeNativeTheme.spec.ts b/packages/main/src/reactions/__tests__/synchronizeNativeTheme.test.ts
index cf37568..05f14fb 100644
--- a/packages/main/src/reactions/__tests__/synchronizeNativeTheme.spec.ts
+++ b/packages/main/src/reactions/__tests__/synchronizeNativeTheme.test.ts
@@ -50,16 +50,16 @@ beforeEach(() => {
   disposeSut = synchronizeNativeTheme(store);
 });
 
-it('should register a nativeTheme updated listener', () => {
+test('should register a nativeTheme updated listener', () => {
   expect(nativeTheme.on).toHaveBeenCalledWith('updated', expect.anything());
 });
 
-it('should synchronize themeSource changes to the nativeTheme', () => {
+test('should synchronize themeSource changes to the nativeTheme', () => {
   store.settings.setThemeSource('dark');
   expect(nativeTheme.themeSource).toBe('dark');
 });
 
-it('should synchronize shouldUseDarkColors changes to the store', () => {
+test('should synchronize shouldUseDarkColors changes to the store', () => {
   const listener = mocked(nativeTheme.on).mock.calls.find(
     ([event]) => event === 'updated',
   )![1];
@@ -68,7 +68,7 @@ it('should synchronize shouldUseDarkColors changes to the store', () => {
   expect(store.shouldUseDarkColors).toBe(true);
 });
 
-it('should remove the listener on dispose', () => {
+test('should remove the listener on dispose', () => {
   const listener = mocked(nativeTheme.on).mock.calls.find(
     ([event]) => event === 'updated',
   )![1];
diff --git a/packages/main/src/stores/__tests__/SharedStore.spec.ts b/packages/main/src/stores/__tests__/SharedStore.test.ts
index 268ce3f..0ed63e3 100644
--- a/packages/main/src/stores/__tests__/SharedStore.spec.ts
+++ b/packages/main/src/stores/__tests__/SharedStore.test.ts
@@ -39,7 +39,7 @@ beforeEach(() => {
 });
 
 describe('loadConfig', () => {
-  it('should load profiles with an ID', () => {
+  test('loads profiles with an ID', () => {
     sut.loadConfig({
       profiles: [
         {
@@ -51,14 +51,14 @@ describe('loadConfig', () => {
     expect(sut.profiles[0].id).toBe('someId');
   });
 
-  it('should generate an ID for profiles without and ID', () => {
+  test('generates an ID for profiles without and ID', () => {
     sut.loadConfig({
       profiles: [profileProps],
     });
     expect(sut.profiles[0].id).toBeDefined();
   });
 
-  it('should load services with an ID and a profile', () => {
+  test('loads services with an ID and a profile', () => {
     sut.loadConfig({
       profiles: [
         {
@@ -78,7 +78,7 @@ describe('loadConfig', () => {
     expect(sut.services[0].settings.profile).toBe(sut.profiles[0]);
   });
 
-  it('should refuse to load a profile without a name', () => {
+  test('refuses to load a profile without a name', () => {
     expect(() => {
       sut.loadConfig({
         profiles: [
@@ -93,7 +93,7 @@ describe('loadConfig', () => {
     expect(sut.profiles).toHaveLength(0);
   });
 
-  it('should load services without an ID but with a profile', () => {
+  test('loads services without an ID but with a profile', () => {
     sut.loadConfig({
       profiles: [
         {
@@ -112,7 +112,7 @@ describe('loadConfig', () => {
     expect(sut.services[0].settings.profile).toBe(sut.profiles[0]);
   });
 
-  it('should create a profile for a service with an ID but no profile', () => {
+  test('creates a profile for a service with an ID but no profile', () => {
     sut.loadConfig({
       services: [
         {
@@ -128,7 +128,7 @@ describe('loadConfig', () => {
     );
   });
 
-  it('should create a profile for a service without an ID or profile', () => {
+  test('creates a profile for a service without an ID or profile', () => {
     sut.loadConfig({
       services: [
         {
@@ -143,7 +143,7 @@ describe('loadConfig', () => {
     );
   });
 
-  it('should refuse to load a service without a name', () => {
+  test('refuses to load a service without a name', () => {
     expect(() => {
       sut.loadConfig({
         services: [
diff --git a/packages/preload/src/contextBridge/__tests__/createSophieRenderer.spec.ts b/packages/preload/src/contextBridge/__tests__/createSophieRenderer.test.ts
index 4411789..21620ed 100644
--- a/packages/preload/src/contextBridge/__tests__/createSophieRenderer.spec.ts
+++ b/packages/preload/src/contextBridge/__tests__/createSophieRenderer.test.ts
@@ -73,7 +73,7 @@ beforeAll(() => {
 });
 
 describe('createSophieRenderer', () => {
-  it('registers a shared store patch listener', () => {
+  test('registers a shared store patch listener', () => {
     createSophieRenderer(false);
     expect(ipcRenderer.on).toHaveBeenCalledWith(
       MainToRendererIpcMessage.SharedStorePatch,
@@ -103,7 +103,7 @@ describe('SharedStoreConnector', () => {
   });
 
   describe('onSharedStoreChange', () => {
-    it('should request a snapshot from the main process', async () => {
+    test('requests a snapshot from the main process', async () => {
       mocked(ipcRenderer.invoke).mockResolvedValueOnce(snapshot);
       await sut.onSharedStoreChange(listener);
       expect(ipcRenderer.invoke).toHaveBeenCalledWith(
@@ -112,7 +112,7 @@ describe('SharedStoreConnector', () => {
       expect(listener.onSnapshot).toHaveBeenCalledWith(snapshot);
     });
 
-    it('should catch IPC errors without exposing them', async () => {
+    test('catches IPC errors without exposing them', async () => {
       mocked(ipcRenderer.invoke).mockRejectedValue(new Error('s3cr3t'));
       await expect(
         sut.onSharedStoreChange(listener),
@@ -122,7 +122,7 @@ describe('SharedStoreConnector', () => {
   });
 
   describe('dispatchAction', () => {
-    it('should dispatch valid actions', () => {
+    test('dispatches valid actions', () => {
       sut.dispatchAction(action);
       expect(ipcRenderer.send).toHaveBeenCalledWith(
         RendererToMainIpcMessage.DispatchAction,
@@ -130,31 +130,31 @@ describe('SharedStoreConnector', () => {
       );
     });
 
-    it('should not dispatch invalid actions', () => {
+    test('does not dispatch invalid actions', () => {
       expect(() => sut.dispatchAction(invalidAction)).toThrow();
       expect(ipcRenderer.send).not.toHaveBeenCalled();
     });
   });
 
   describe('when no listener is registered', () => {
-    it('should discard the received patch without any error', () => {
+    test('discards the received patch without any error', () => {
       expect(() => onSharedStorePatch(event, patch)).not.toThrow();
     });
   });
 
-  function itRefusesToRegisterAnotherListener(): void {
-    it('should refuse to register another listener', async () => {
+  function testRefusesToRegisterAnotherListener(): void {
+    test('refuses to register another listener', async () => {
       await expect(sut.onSharedStoreChange(listener)).rejects.toBeInstanceOf(
         Error,
       );
     });
   }
 
-  function itDoesNotPassPatchesToTheListener(
-    name = 'should not pass patches to the listener',
+  function testDoesNotPassPatchesToTheListener(
+    name = 'does not pass patches to the listener',
   ): void {
     // eslint-disable-next-line jest/valid-title -- Title is a string parameter.
-    it(name, () => {
+    test(name, () => {
       onSharedStorePatch(event, patch);
       expect(listener.onPatch).not.toHaveBeenCalled();
     });
@@ -166,19 +166,19 @@ describe('SharedStoreConnector', () => {
       await sut.onSharedStoreChange(listener);
     });
 
-    it('should pass patches to the listener', () => {
+    test('passes patches to the listener', () => {
       onSharedStorePatch(event, patch);
       expect(listener.onPatch).toHaveBeenCalledWith(patch);
     });
 
-    it('should catch listener errors', () => {
+    test('catches listener errors', () => {
       mocked(listener.onPatch).mockImplementation(() => {
         throw new Error('listener error');
       });
       expect(() => onSharedStorePatch(event, patch)).not.toThrow();
     });
 
-    itRefusesToRegisterAnotherListener();
+    testRefusesToRegisterAnotherListener();
 
     describe('after the listener threw in onPatch', () => {
       beforeEach(() => {
@@ -189,7 +189,7 @@ describe('SharedStoreConnector', () => {
         listener.onPatch.mockRestore();
       });
 
-      itDoesNotPassPatchesToTheListener('should not pass on patches any more');
+      testDoesNotPassPatchesToTheListener('does not pass on patches any more');
     });
   });
 
@@ -205,9 +205,9 @@ describe('SharedStoreConnector', () => {
       }
     });
 
-    itRefusesToRegisterAnotherListener();
+    testRefusesToRegisterAnotherListener();
 
-    itDoesNotPassPatchesToTheListener();
+    testDoesNotPassPatchesToTheListener();
   });
 
   describe('when a listener failed to register due to listener error', () => {
@@ -223,9 +223,9 @@ describe('SharedStoreConnector', () => {
       }
     });
 
-    itRefusesToRegisterAnotherListener();
+    testRefusesToRegisterAnotherListener();
 
-    itDoesNotPassPatchesToTheListener();
+    testDoesNotPassPatchesToTheListener();
   });
 
   describe('when it is allowed to replace listeners', () => {
@@ -239,7 +239,7 @@ describe('SharedStoreConnector', () => {
       onPatch: jest.fn((_patch: IJsonPatch[]) => {}),
     };
 
-    it('should fetch a second snapshot', async () => {
+    test('fetches a second snapshot', async () => {
       mocked(ipcRenderer.invoke).mockResolvedValueOnce(snapshot2);
       await sut.onSharedStoreChange(listener2);
       expect(ipcRenderer.invoke).toHaveBeenCalledWith(
@@ -248,7 +248,7 @@ describe('SharedStoreConnector', () => {
       expect(listener2.onSnapshot).toHaveBeenCalledWith(snapshot2);
     });
 
-    it('should pass the second snapshot to the new listener', async () => {
+    test('passes the second snapshot to the new listener', async () => {
       mocked(ipcRenderer.invoke).mockResolvedValueOnce(snapshot2);
       await sut.onSharedStoreChange(listener2);
       onSharedStorePatch(event, patch);