fix(rtl): fix noPolyfill arg in scrollToItem

#241

package.json

@@ -141,7 +141,7 @@
       ]
     },
     "test:lint": {
-      "command": "tsc --noEmit && eslint src"
+      "command": "eslint src && tsc --noEmit"
     },
     "test:unit": {
       "command": "jest",

src/createApi.test.ts

@@ -52,6 +52,7 @@ const setup = (ratio = [0.3, 1, 0.7]) => {
 describe('createApi', () => {
   afterEach(() => {
     jest.clearAllMocks();
+    jest.restoreAllMocks();
   });
 
   test('visibleElementsWithSeparators', () => {
@@ -130,6 +131,39 @@ describe('createApi', () => {
         });
       });
 
+      test('with noPolyfill', () => {
+        const { items, visibleElementsWithSeparators } = setup([0.7, 0, 0]);
+        const scrollToItemSpy = jest
+          .spyOn(helpers, 'scrollToItem')
+          .mockReturnValue(jest.fn());
+
+        const boundary = { current: document.createElement('div') };
+        const transitionOptions = undefined;
+
+        const noPolyfill = true;
+
+        const elem = document.createElement('div');
+        createApi(
+          items,
+          visibleElementsWithSeparators,
+          boundary,
+          transitionOptions,
+          undefined,
+          noPolyfill
+        ).scrollToItem(elem);
+
+        expect(scrollToItemSpy).toHaveBeenCalledTimes(1);
+        expect(scrollToItemSpy).toHaveBeenNthCalledWith(
+          1,
+          elem,
+          undefined,
+          undefined,
+          undefined,
+          expect.objectContaining({ boundary: expect.anything() }),
+          noPolyfill
+        );
+      });
+
       test('arguments should have priority over transitionOptions', () => {
         const { items, visibleElementsWithSeparators } = setup([0.7, 0, 0]);
 

src/createApi.ts

@@ -130,12 +130,19 @@ export default function createApi(
       options?: scrollToItemOptions
     ) => {
       const _behavior = behavior ?? transitionOptions?.behavior;
-      return scrollToItem(target, _behavior, inline, block, {
-        boundary: boundaryElement?.current,
-        ...options,
-        duration: options?.duration ?? transitionOptions?.duration,
-        ease: options?.ease ?? transitionOptions?.ease,
-      });
+      return scrollToItem(
+        target,
+        _behavior,
+        inline,
+        block,
+        {
+          boundary: boundaryElement?.current,
+          ...options,
+          duration: options?.duration ?? transitionOptions?.duration,
+          ease: options?.ease ?? transitionOptions?.ease,
+        },
+        noPolyfill
+      );
     },
     visibleElements,
     visibleElementsWithSeparators,