test: use test instead of it

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

3 files changed, 15 insertions, 15 deletions

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'),
       );