fix: fixed linting errors regarding no-unused-expressions.

Author: CDeltakai

src/keys/KeyRing.ts

@@ -947,8 +947,8 @@ class KeyRing {
         `${this.dbKeyPath}.tmp`,
         dbJWEJSON,
         'utf-8',
-      ),
-        await this.fs.promises.rename(`${this.dbKeyPath}.tmp`, this.dbKeyPath);
+      );
+      await this.fs.promises.rename(`${this.dbKeyPath}.tmp`, this.dbKeyPath);
     } catch (e) {
       throw new keysErrors.ErrorDBKeyWrite(
         `DB key path ${this.dbKeyPath} cannot be written to`,

tests/client/handlers/identities.test.ts

@@ -476,7 +476,9 @@ describe('identitiesClaim', () => {
       .mockImplementation(async (payload, _, func) => {
         const token = Token.fromPayload(payload);
         // We need to call the function to resolve a promise in the code
-        func != null && (await func(token as unknown as Token<Claim>));
+        if (func) {
+          await func(token as unknown as Token<Claim>);
+        }
         return [claimId, signedClaim];
       });
 
@@ -1750,7 +1752,9 @@ describe('identitiesInvite', () => {
       .mockImplementation(async (payload, _, func) => {
         const token = Token.fromPayload(payload);
         // We need to call the function to resolve a promise in the code
-        func != null && (await func(token as unknown as Token<Claim>));
+        if (func) {
+          await func(token as unknown as Token<Claim>);
+        }
         return [claimId, signedClaim];
       });
 

tests/keys/KeyRing.test.ts

@@ -45,7 +45,7 @@ describe(KeyRing.name, () => {
       keysErrors.ErrorKeyRingDestroyed,
     );
     expect(() => {
-      keyRing.keyPair;
+      void keyRing.keyPair;
     }).toThrow(keysErrors.ErrorKeyRingNotRunning);
     await expect(async () => {
       await keyRing.checkPassword(password);