Merge pull request #139 from gpbl/attributes-spread

Spread attributes first (fix #137)

src/DayPicker.js

@@ -213,22 +213,25 @@ export default class DayPicker extends Component {
 
   handleKeyDown(e) {
     e.persist();
+    const { canChangeMonth, onKeyDown } = this.props;
 
-    if (!this.props.canChangeMonth && this.props.onKeyDown) {
-      this.props.onKeyDown(e);
+    if (!canChangeMonth && onKeyDown) {
+      onKeyDown(e);
       return;
     }
 
-    if (this.props.canChangeMonth) {
-      const callback = this.props.onKeyDown ? () => this.props.onKeyDown(e) : null;
-
+    if (canChangeMonth) {
       switch (e.keyCode) {
       case keys.LEFT:
-        this.showPreviousMonth(callback);
+        this.showPreviousMonth(onKeyDown);
         break;
       case keys.RIGHT:
-        this.showNextMonth(callback);
+        this.showNextMonth(onKeyDown);
         break;
+      default:
+        if (onKeyDown) {
+          onKeyDown(e);
+        }
       }
     }
   }
@@ -469,11 +472,12 @@ export default class DayPicker extends Component {
     }
 
     return (
-      <div className={ className }
+      <div
+        {...attributes}
+        className={ className }
         role="widget"
         tabIndex={ canChangeMonth && attributes.tabIndex }
-        onKeyDown={ e => this.handleKeyDown(e) }
-        {...attributes}>
+        onKeyDown={ e => this.handleKeyDown(e) }>
         { canChangeMonth && this.renderNavBar() }
         { months }
       </div>

test/DayPicker.js

@@ -98,6 +98,7 @@ describe("DayPicker", () => {
 
     const dayPicker = shallowRenderer.getRenderOutput();
     expect(dayPicker.props.className).to.contain("custom-class");
+    expect(dayPicker.props.className).to.contain("DayPicker");
   });
 
   // RENDERING
@@ -924,7 +925,7 @@ describe("DayPicker", () => {
       const dayPicker = TestUtils.renderIntoDocument(<DayPicker onKeyDown={spy} />);
       const node = ReactDOM.findDOMNode(dayPicker);
       TestUtils.Simulate.keyDown(node);
-      expect(spy.calledOnce).to.be.true;
+      expect(spy).to.be.calledOnce;
     });
 
     it("should handle keydown event when cannot change month", () => {
@@ -934,7 +935,7 @@ describe("DayPicker", () => {
       );
       const node = ReactDOM.findDOMNode(dayPicker);
       TestUtils.Simulate.keyDown(node);
-      expect(spy.calledOnce).to.be.true;
+      expect(spy).to.be.calledOnce;
     });
 
   });