Merge branch 'master' into all-contributors/add-nicksrandall

Author: bcarroll22

.all-contributorsrc

@@ -242,6 +242,13 @@
       "profile": "http://www.nickrandall.com",
       "contributions": [
         "review"
+    },
+      "login": "CarsonF",
+      "name": "Carson Full",
+      "avatar_url": "https://avatars3.githubusercontent.com/u/932566?v=4",
+      "profile": "https://github.com/CarsonF",
+      "contributions": [
+        "code"
       ]
     }
   ],

README.md

@@ -668,6 +668,7 @@ Thanks goes to these wonderful people ([emoji key](https://allcontributors.org/d
     <td align="center"><a href="https://github.com/kevynb"><img src="https://avatars1.githubusercontent.com/u/4941215?v=4" width="100px;" alt="Kevyn Bruyere"/><br /><sub><b>Kevyn Bruyere</b></sub></a><br /><a href="https://github.com/express-labs/pure-react-carousel/commits?author=kevynb" title="Code">💻</a></td>
     <td align="center"><a href="https://github.com/mariohoyos92"><img src="https://avatars1.githubusercontent.com/u/29843005?v=4" width="100px;" alt="Mario Hoyos"/><br /><sub><b>Mario Hoyos</b></sub></a><br /><a href="https://github.com/express-labs/pure-react-carousel/commits?author=mariohoyos92" title="Code">💻</a></td>
     <td align="center"><a href="http://www.nickrandall.com"><img src="https://avatars1.githubusercontent.com/u/1800460?v=4" width="100px;" alt="Nick Randall"/><br /><sub><b>Nick Randall</b></sub></a><br /><a href="#review-nicksrandall" title="Reviewed Pull Requests">👀</a></td>
+    <td align="center"><a href="https://github.com/CarsonF"><img src="https://avatars3.githubusercontent.com/u/932566?v=4" width="100px;" alt="Carson Full"/><br /><sub><b>Carson Full</b></sub></a><br /><a href="https://github.com/express-labs/pure-react-carousel/commits?author=CarsonF" title="Code">💻</a></td>
   </tr>
 </table>
 

src/Slider/Slider.jsx

@@ -613,8 +613,13 @@ const Slider = class Slider extends React.Component {
 
     const newTabIndex = tabIndex !== null ? tabIndex : 0;
 
-    // remove `dragStep` and `step` from Slider div, since it isn't a valid html attribute
-    const { dragStep, step, ...rest } = props;
+    // remove invalid div attributes
+    const {
+      dragStep,
+      step,
+      infinite,
+      ...rest
+    } = props;
 
     // filter out some tray props before passing them in.  We will process event handlers in the
     // trayProps object as callbacks to OUR event handlers.  Ref is needed by us. Style and

src/Slider/__tests__/Slider.test.jsx

@@ -1047,5 +1047,13 @@ describe('<Slider />', () => {
       instance.unlockScroll();
       expect(instance.scrollParent).toEqual(null);
     });
+
+    it('should not pass invalid props to div', () => {
+      const wrapper = shallow(<Slider {...props} />);
+      const divProps = wrapper.closest('div').props();
+      expect(divProps.dragStep).toBeUndefined();
+      expect(divProps.step).toBeUndefined();
+      expect(divProps.infinite).toBeUndefined();
+    });
   });
 });