tests: fixing test failures

fix: `tests/inodes/INodeManager.test.ts` change to using `await expect().rejects.toThrow`

Author: tegefaulkes

tests/EncryptedFS.concurrent.test.ts

@@ -2941,7 +2941,7 @@ describe(`${EncryptedFS.name} Concurrency`, () => {
         { status: 'fulfilled', value: undefined },
       ]);
       // All A, in multiples of 5
-      expect(contents).toMatch(RegExp('^[^A]*((AAAAA)+[^A]*)$'));
+      expect(contents).toMatch(RegExp('^(AAAAA)*$'));
       // Contents length between 0 and 10*5
       expect(contents.length).toBeGreaterThanOrEqual(0);
       expect(contents.length).toBeLessThanOrEqual(50);
@@ -2977,7 +2977,7 @@ describe(`${EncryptedFS.name} Concurrency`, () => {
         { status: 'fulfilled', value: undefined },
       ]);
       // All A, in multiples of 5
-      expect(contents).toMatch(RegExp('^[^A]*((AAAAA)+[^A]*)$'));
+      expect(contents).toMatch(RegExp('^(AAAAA)*$'));
       // Contents length between 0 and 10*5
       expect(contents.length).toBeGreaterThanOrEqual(0);
       expect(contents.length).toBeLessThanOrEqual(50);

tests/inodes/INodeManager.test.ts

@@ -2,7 +2,7 @@ import os from 'os';
 import pathNode from 'path';
 import fs from 'fs';
 import Logger, { LogLevel, StreamHandler } from '@matrixai/logger';
-import { DB } from '@matrixai/db';
+import { DB, errors as dbErrors } from '@matrixai/db';
 import INodeManager from '@/inodes/INodeManager';
 import * as utils from '@/utils';
 import * as permissions from '@/permissions';
@@ -89,16 +89,24 @@ describe('INodeManager', () => {
     await iNodeMgr.withTransactionF(async (tran) => {
       await tran.put([...iNodeMgr.mgrDbPath, 'test'], 0);
     });
-    await Promise.all([
-      iNodeMgr.withTransactionF(async (tran) => {
-        const num = (await tran.get<number>([...iNodeMgr.mgrDbPath, 'test']))!;
-        await tran.put([...iNodeMgr.mgrDbPath, 'test'], num + 1);
-      }),
-      iNodeMgr.withTransactionF(async (tran) => {
-        const num = (await tran.get<number>([...iNodeMgr.mgrDbPath, 'test']))!;
-        await tran.put([...iNodeMgr.mgrDbPath, 'test'], num + 1);
-      }),
-    ]);
+    await expect(
+      Promise.all([
+        iNodeMgr.withTransactionF(async (tran) => {
+          const num = (await tran.get<number>([
+            ...iNodeMgr.mgrDbPath,
+            'test',
+          ]))!;
+          await tran.put([...iNodeMgr.mgrDbPath, 'test'], num + 1);
+        }),
+        iNodeMgr.withTransactionF(async (tran) => {
+          const num = (await tran.get<number>([
+            ...iNodeMgr.mgrDbPath,
+            'test',
+          ]))!;
+          await tran.put([...iNodeMgr.mgrDbPath, 'test'], num + 1);
+        }),
+      ]),
+    ).rejects.toThrow(dbErrors.ErrorDBTransactionConflict);
     await iNodeMgr.withTransactionF(async (tran) => {
       const num = (await tran.get<number>([...iNodeMgr.mgrDbPath, 'test']))!;
       // Race condition clobbers the counter