Call setCurrentTime() after a new video has loaded. (#95)

* Call `setCurrentTime()` after a new video has loaded.

Fixes #68

Apparently doing `setCurrentTime()` immediately is fine for the initial
load, but not when starting a new video.

* lint fix

* test: return Promises from spies

src/index.js

@@ -88,10 +88,14 @@ class Vimeo extends React.Component {
         case 'video':
           if (value) {
             const { start } = this.props;
-            player.loadVideo(value);
+            const loaded = player.loadVideo(value);
             // Set the start time only when loading a new video.
+            // It seems like this has to be done after the video has loaded, else it just starts at
+            // the beginning!
             if (typeof start === 'number') {
-              player.setCurrentTime(start);
+              loaded.then(() => {
+                player.setCurrentTime(start);
+              });
             }
           } else {
             player.unload();

test/util/createVimeo.js

@@ -4,20 +4,22 @@ import proxyquire from 'proxyquire';
 export default function createVimeo({ shouldFail = false } = {}) {
   let isPaused = true;
 
+  const createPromiseSpy = () => createSpy().andCall(() => Promise.resolve());
+
   const playerMock = {
     on: createSpy(),
     ready() {
       return shouldFail
         ? Promise.reject(new Error('artificial failure'))
         : Promise.resolve();
     },
-    setVolume: createSpy(),
-    setCurrentTime: createSpy(),
-    setAutopause: createSpy(),
-    setColor: createSpy(),
-    setLoop: createSpy(),
-    loadVideo: createSpy(),
-    unload: createSpy(),
+    setVolume: createPromiseSpy(),
+    setCurrentTime: createPromiseSpy(),
+    setAutopause: createPromiseSpy(),
+    setColor: createPromiseSpy(),
+    setLoop: createPromiseSpy(),
+    loadVideo: createPromiseSpy(),
+    unload: createPromiseSpy(),
     play: createSpy().andCall(() => {
       isPaused = false;
     }),