Revert "remove redundancy"

This reverts commit de71406.
yomidevs · Dec 29, 2023 · f58d427 · f58d427
1 parent de71406
commit f58d427
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 1 deletion.
diff --git a/ext/js/general/cache-map.js b/ext/js/general/cache-map.js
@@ -29,6 +29,7 @@ export class CacheMap {
      */
     constructor(maxSize) {
         if (!(
+            typeof maxSize === 'number' &&
             Number.isFinite(maxSize) &&
             maxSize >= 0 &&
             Math.floor(maxSize) === maxSize

diff --git a/test/cache-map.test.js b/test/cache-map.test.js
@@ -31,7 +31,9 @@ function testConstructor() {
             [true,  () => new CacheMap(-1)],
             [true,  () => new CacheMap(1.5)],
             [true,  () => new CacheMap(Number.NaN)],
-            [true,  () => new CacheMap(Number.POSITIVE_INFINITY)]
+            [true,  () => new CacheMap(Number.POSITIVE_INFINITY)],
+            // @ts-expect-error - Ignore because it should throw an error
+            [true,  () => new CacheMap('a')]
         ];
 
         for (const [throws, create] of data) {