refactor: remove double arguments invocation error

toomuchdesign · Dec 8, 2023 · 2b4b6af · 2b4b6af
1 parent 2b6e7d6
commit 2b4b6af
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 22 deletions.
diff --git a/src/__tests__/createCachedSelector.test.ts b/src/__tests__/createCachedSelector.test.ts
@@ -76,21 +76,6 @@ describe('createCachedSelector', () => {
         expect(cachedSelector.keySelector).toBe(generatedKeySelector);
       });
     });
-
-    describe('as second argument', () => {
-      it('throws an error', () => {
-        expect(() => {
-          createCachedSelector(
-            () => {},
-            () => {}
-          )(
-            () => {},
-            // @ts-expect-error
-            {}
-          );
-        }).toThrow(/"options" as second argument is not supported anymore/);
-      });
-    });
   });
 
   describe('created selector', () => {

diff --git a/src/createCachedSelector.js b/src/createCachedSelector.js
@@ -5,13 +5,7 @@ const defaultCacheCreator = FlatObjectCache;
 const defaultCacheKeyValidator = () => true;
 
 function createCachedSelector(...funcs) {
-  return (polymorphicOptions, legacyOptions) => {
-    if (legacyOptions) {
-      throw new Error(
-        '[re-reselect] "options" as second argument is not supported anymore. Please provide an option object as single argument.'
-      );
-    }
-
+  return polymorphicOptions => {
     const options =
       typeof polymorphicOptions === 'function'
         ? {keySelector: polymorphicOptions}