fix(carousel): fix problem with direction when there are only two images

Author: rui.franclimsilva

src/context/CarouselProvider.js

@@ -61,13 +61,26 @@ class CarouselProvider extends Component {
     }
 
     _getDirection(newActiveItem, activeItem = this.state.activeItem) {
-        const lastItem = this.state.itemsLength - 1;
-
-        if (newActiveItem === 0 && lastItem === activeItem) {
+        const { isInfinite, direction, itemsLength } = this.state;
+        const lastItem = itemsLength - 1;
+
+        // Going from last to first
+        if (
+            isInfinite &&
+            newActiveItem === 0 &&
+            lastItem === activeItem &&
+            direction === 'next'
+        ) {
             return 'next';
         }
 
-        if (activeItem === 0 && newActiveItem === lastItem) {
+        // Going from first to last
+        if (
+            isInfinite &&
+            activeItem === 0 &&
+            newActiveItem === lastItem &&
+            direction === 'prev'
+        ) {
             return 'prev';
         }
 

tests/context/CarouselProvider.test.js

@@ -671,38 +671,38 @@ describe('<CarouselProvider/>', () => {
             expect(result).toBe(prevDirection);
         });
 
-        test('should return prev if new active item is the last item of items', () => {
+        test('should return next when is infinite and user moves in next direction while at the end of the carousel', () => {
             const wrapper = shallow(
-                <CarouselProvider>
+                <CarouselProvider isInfinite>
                     <MockComponent />
                 </CarouselProvider>
             );
 
             const instance = wrapper.instance();
 
             instance._setItemsLength(2);
-            instance._setActiveItem(0);
-
-            const result = instance._getDirection(1);
+            instance._goNext();
+            instance._setActiveItem(1);
+            const result = instance._getDirection(0);
 
-            expect(result).toBe(prevDirection);
+            expect(result).toBe(nextDirection);
         });
 
-        test('should return next if new active item is the first item of items', () => {
+        test('should return prev when is infinite and user moves in prev direction while at the start of the carousel', () => {
             const wrapper = shallow(
-                <CarouselProvider>
+                <CarouselProvider isInfinite>
                     <MockComponent />
                 </CarouselProvider>
             );
 
             const instance = wrapper.instance();
 
             instance._setItemsLength(2);
-            instance._setActiveItem(1);
-
-            const result = instance._getDirection(0);
+            instance._goPrev();
+            instance._setActiveItem(0);
+            const result = instance._getDirection(1);
 
-            expect(result).toBe(nextDirection);
+            expect(result).toBe(prevDirection);
         });
     });