#9 converted impress-vendor to module in order to pass multiple tests

Author: abbr

src/components/show/impress-vendor.js

@@ -21,7 +21,7 @@
 
 // You are one of those who like to know how things work inside?
 // Let me show you the cogs that make impress.js run...
-;(function(document, window) {
+export function Impress(document, window) {
   'use strict'
 
   // HELPER FUNCTIONS
@@ -687,7 +687,7 @@
 
   // Flag that can be used in JS to check if browser have passed the support test
   impress.supported = impressSupported
-})(document, window)
+}
 
 // NAVIGATION EVENTS
 
@@ -697,7 +697,7 @@
 //
 // In future I think about moving it to make them optional, move to separate files
 // and treat more like a 'plugins'.
-;(function(document, window) {
+export function ImpressHandleEvents() {
   'use strict'
 
   // Throttling function calls, by Remy Sharp
@@ -768,27 +768,22 @@
             return
           }
 
-          if (
-            event.keyCode === 9 ||
-            (event.keyCode >= 32 && event.keyCode <= 34) ||
-            (event.keyCode >= 37 && event.keyCode <= 40)
-          ) {
-            switch (event.keyCode) {
-              case 33: // Page up
-              case 37: // Left
-              case 38: // Up
-                api.prev()
-                break
-              case 9: // Tab
-              case 32: // Space
-              case 34: // Page down
-              case 39: // Right
-              case 40: // Down
-                api.next()
-                break
-            }
-
-            event.preventDefault()
+          switch (event.key) {
+            case 'PageUp':
+            case 'ArrowLeft':
+            case 'ArrowUp':
+              api.prev()
+              event.preventDefault()
+              break
+            case 'Tab':
+            case 'Space':
+            case 'PageDown':
+            case 'ArrowRight':
+            case 'ArrowDown':
+              api.next()
+              event.preventDefault()
+              break
+            default:
           }
         },
         false
@@ -885,7 +880,7 @@
     },
     false
   )
-})(document, window)
+}
 
 // THAT'S ALL FOLKS!
 //

src/components/show/impress.js

@@ -1,14 +1,14 @@
 'use strict'
 import React from 'react'
 import ReactDOM from 'react-dom'
-import './impress-vendor.js'
+import { Impress, ImpressHandleEvents } from './impress-vendor.js'
 import './impress.less'
 import DeckStore from 'stores/deck'
 import AutoScale from 'components/mixins/autoScale'
 import _ from 'lodash'
 import Global from './global'
 var DisplayableComponent = require('components/widgets/displayableComponent')
-
+ImpressHandleEvents(document, window)
 let Presentation = class extends AutoScale.autoScaleMixin(React.Component) {
   constructor(props) {
     super(props)
@@ -23,6 +23,7 @@ let Presentation = class extends AutoScale.autoScaleMixin(React.Component) {
   }
 
   componentDidMount() {
+    Impress(document, window)
     super.componentDidMount && super.componentDidMount()
     if ('ontouchstart' in document.documentElement) {
       document.querySelector('.hint').innerHTML =

test/components/show/impressTest.js

@@ -4,10 +4,31 @@ describe('show > impress', () => {
   beforeEach(() => {
     appWrapper = getAppWrapper('Impress')
   })
+  afterEach(() => {
+    $('body').removeAttr('style')
+    $('body').removeAttr('class')
+  })
 
   it('should have 5 initial views', () => {
     expect(appWrapper.find('#impress').props().children.length).to.equal(5)
     expect($('body').attr('class')).to.contain('impress-on-step-1')
-    expect($('#step-1').attr('class').split(' ')).to.include('active')
+    expect(
+      $('#step-1')
+        .attr('class')
+        .split(' ')
+    ).to.include('active')
+  })
+
+  it('should move to 2nd step when press right key', () => {
+    let event = new KeyboardEvent('keyup', {
+      key: 'ArrowRight'
+    })
+    document.dispatchEvent(event)
+    expect($('body').attr('class')).to.contain('impress-on-step-2')
+    expect(
+      $('#step-2')
+        .attr('class')
+        .split(' ')
+    ).to.include('active')
   })
 })