ban 0 for short public key test cases to stop AbortOnZeroSrpError cau…

…sing flaking tests
simonmcallister0210 · Mar 12, 2024 · 59fd456 · 59fd456
1 parent 93ca42f
commit 59fd456
Show file tree

Hide file tree

Showing 3 changed files with 8 additions and 4 deletions.
diff --git a/src/__tests__/test-cases/initiate-auth-response.ts b/src/__tests__/test-cases/initiate-auth-response.ts
@@ -19,7 +19,8 @@ export const positiveInitiateAuthResponses: Record<string, InitiateAuthResponse>
   largeBShort: mockInitiateAuthResponseFactory({
     ChallengeParameters: {
       ...ChallengeParameters,
-      SRP_B: faker.random.alphaNumeric(1, { casing: "lower" }),
+      // 1 / 62 chance to return "0" which will trigger a AbortOnZeroBSrpError, so ban the char
+      SRP_B: faker.random.alphaNumeric(1, { casing: "lower", bannedChars: "0" }),
     },
   }),
   largeBLong: mockInitiateAuthResponseFactory({

diff --git a/src/__tests__/test-cases/srp-sessions-signed.ts b/src/__tests__/test-cases/srp-sessions-signed.ts
@@ -90,7 +90,8 @@ export const positiveSrpSessionsSigned: Record<string, SrpSessionSigned> = {
     largeA: faker.random.alphaNumeric(1024, { casing: "lower" }),
   }),
   largeAShort: mockSrpSessionSignedFactory({
-    largeA: faker.random.alphaNumeric(1, { casing: "lower" }),
+    // 1 / 62 chance to return "0" which will trigger a AbortOnZeroASrpError, so ban the char
+    largeA: faker.random.alphaNumeric(1, { casing: "lower", bannedChars: "0" }),
   }),
   largeALong: mockSrpSessionSignedFactory({
     largeA: faker.random.alphaNumeric(10000, { casing: "lower" }),
@@ -100,7 +101,8 @@ export const positiveSrpSessionsSigned: Record<string, SrpSessionSigned> = {
     largeB: faker.random.alphaNumeric(1024, { casing: "lower" }),
   }),
   largeBShort: mockSrpSessionSignedFactory({
-    largeB: faker.random.alphaNumeric(1, { casing: "lower" }),
+    // 1 / 62 chance to return "0" which will trigger a AbortOnZeroBSrpError, so ban the char
+    largeB: faker.random.alphaNumeric(1, { casing: "lower", bannedChars: "0" }),
   }),
   largeBLarge: mockSrpSessionSignedFactory({
     largeB: faker.random.alphaNumeric(10000, { casing: "lower" }),

diff --git a/src/__tests__/test-cases/srp-sessions.ts b/src/__tests__/test-cases/srp-sessions.ts
@@ -89,7 +89,8 @@ export const positiveSrpSessions: Record<string, SrpSession> = {
     largeA: faker.random.alphaNumeric(1024, { casing: "lower" }),
   }),
   largeAShort: mockSrpSessionFactory({
-    largeA: faker.random.alphaNumeric(1, { casing: "lower" }),
+    // 1 / 62 chance to return "0" which will trigger a AbortOnZeroASrpError, so ban the char
+    largeA: faker.random.alphaNumeric(1, { casing: "lower", bannedChars: "0" }),
   }),
   largeALong: mockSrpSessionFactory({
     largeA: faker.random.alphaNumeric(10000, { casing: "lower" }),