fix: regex match condition for primary type (#353)

MetaMask · Dec 19, 2024 · 73db8a7 · 73db8a7
1 parent 2f7d139
commit 73db8a7
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 1 deletion.
diff --git a/src/utils/common.test.ts b/src/utils/common.test.ts
@@ -10,6 +10,10 @@ describe('CommonUtils', () => {
     it('return types which are not array without any change', () => {
       expect(stripArrayTypeIfPresent('string')).toBe('string');
       expect(stripArrayTypeIfPresent('string []')).toBe('string []');
+      expect(
+        // @ts-expect-error Intentionally testing invalid input
+        stripArrayTypeIfPresent(undefined),
+      ).toBeUndefined();
     });
   });
 });
diff --git a/src/utils/common.ts b/src/utils/common.ts
@@ -5,7 +5,7 @@
  * @returns Parameter string with array brackets [] removed.
  */
 export const stripArrayTypeIfPresent = (typeString: string) => {
-  if (typeString?.match(/\S\[\d*\]$/u) !== null) {
+  if (typeString?.match(/\S\[\d*\]$/u)) {
     return typeString.replace(/\[\d*\]$/gu, '').trim();
   }
   return typeString;