Merge pull request #11 from kodado/typescript-improvements

Typescript improvements
kodado · May 8, 2024 · 5e5700d · 5e5700d
2 parents 5d729ad + 742d563
commit 5e5700d
Show file tree

Hide file tree

Showing 3 changed files with 23 additions and 5 deletions.
diff --git a/src/auth/AuthClient.ts b/src/auth/AuthClient.ts
@@ -6,6 +6,7 @@ import {
   EmailAndPasswordRequiredError,
   AlreadySignedInError,
   NotSignedInError,
+  WrongCredentialsError,
 } from "../errors/authErrors";
 
 import {
@@ -72,7 +73,7 @@ export class AuthClient {
     });
 
     if (!session) {
-      return;
+      throw new WrongCredentialsError();
     }
 
     if (session instanceof CognitoUserSession) {
@@ -122,6 +123,10 @@ export class AuthClient {
       };
     }
 
+    if (!this.user) {
+      throw new WrongCredentialsError();
+    }
+
     return this.user;
   }
 
@@ -225,7 +230,7 @@ export class AuthClient {
     this.session = null;
     this.keys = null;
 
-    return this.cognitoClient.deleteCognitoUser(user);
+    await this.cognitoClient.deleteCognitoUser(user);
   }
 
   async getCurrentAuthorizationToken() {

diff --git a/test/auth.spec.ts b/test/auth.spec.ts
@@ -193,8 +193,19 @@ describe("deleteUser", () => {
       email: "auth-lib-user@turingpoint.de",
       password: "Abcd1234!",
     });
-    const success = await client.auth.deleteUser();
 
-    expect(success).toBe(true);
+    await client.auth.deleteUser();
+
+    try {
+      await client.auth.signIn({
+        email: "auth-lib-user@turingpoint.de",
+        password: "Abcd1234!",
+      });
+      expect(false).toBe(true);
+    } catch (e) {
+      if (e instanceof WrongCredentialsError) {
+        expect(true).toBe(true);
+      }
+    }
   });
 });
diff --git a/test/bulkRevokeItem.spec.ts b/test/bulkRevokeItem.spec.ts
@@ -128,7 +128,9 @@ describe("bulkRevoke", () => {
         id: ids[0],
       });
     } catch (e) {
-      expect(e.message).toBe("Item not found.");
+      if (e instanceof Error) {
+        expect(e.message).toBe("Item not found.");
+      }
     }
     client.auth.signOut();
   });