fix(play/stop): prevent scenario from messing up when calling play/st…

…op successively

dist/theater.js

@@ -187,6 +187,10 @@ return /******/ (function(modules) { // webpackBootstrap
 	  }
 
 	  function play() {
+	    if (props.status === 'stopping') {
+	      props.status = 'playing';
+	    }
+
 	    if (props.status === 'ready') {
 	      props.status = 'playing';
 	      playNextScene();
@@ -206,11 +210,16 @@ return /******/ (function(modules) { // webpackBootstrap
 	  }
 
 	  function stop() {
-	    props.status = 'ready';
+	    props.status = 'stopping';
 	    return this;
 	  }
 
 	  function playNextScene() {
+	    if (props.status === 'stopping') {
+	      props.status = 'ready';
+	      return this;
+	    }
+
 	    if (props.status !== 'playing') return this;
 
 	    var currentScene = props.scenario[props.currentScene];

src/theaterJS.js

@@ -98,6 +98,10 @@ function theaterJS (options = {}) {
   }
 
   function play () {
+    if (props.status === 'stopping') {
+      props.status = 'playing'
+    }
+
     if (props.status === 'ready') {
       props.status = 'playing'
       playNextScene()
@@ -118,11 +122,16 @@ function theaterJS (options = {}) {
   }
 
   function stop () {
-    props.status = 'ready'
+    props.status = 'stopping'
     return this
   }
 
   function playNextScene () {
+    if (props.status === 'stopping') {
+      props.status = 'ready'
+      return this
+    }
+
     if (props.status !== 'playing') return this
 
     let currentScene = props.scenario[props.currentScene]

test/TheaterJS.spec.js

@@ -206,14 +206,55 @@ describe('theaterJS', function () {
   })
 
   describe('has a stop method that', function () {
-    it('sets the status to ready', function () {
-      theater = theaterJS({ autoplay: false })
+    beforeEach(function () {
+      jasmine.clock().install()
 
-      theater.addActor('vader').addScene('vader:Hello!').play()
+      theater = theaterJS()
+      theater.addActor('vader').addScene('vader:Hello!', 'vader:How u doing?')
+    })
+
+    afterEach(function () {
+      jasmine.clock().uninstall()
+    })
+
+    it('should stop the scenario', function () {
       expect(theater.status).toBe('playing')
 
       theater.stop()
-      expect(theater.status).toBe('ready')
+
+      expect(theater.status).toBe('stopping')
+
+      jasmine.clock().tick(LONG_TIME)
+      expect(theater.getCurrentActor().displayValue).toBe('Hello!')
+    })
+
+    it('should be resume-able', function () {
+      theater.stop()
+
+      jasmine.clock().tick(LONG_TIME)
+      expect(theater.getCurrentActor().displayValue).toBe('Hello!')
+
+      theater.play()
+
+      jasmine.clock().tick(LONG_TIME)
+      expect(theater.getCurrentActor().displayValue).toBe('How u doing?')
+    })
+
+    it('shouldn\'t be conflicting if called several times alternatively', function () {
+      theater.stop()
+      jasmine.clock().tick(200)
+      theater.play()
+      jasmine.clock().tick(200)
+      theater.stop()
+      jasmine.clock().tick(200)
+      theater.play()
+      jasmine.clock().tick(200)
+      theater.stop()
+      jasmine.clock().tick(200)
+      theater.play()
+
+      jasmine.clock().tick(LONG_TIME)
+      expect(theater.getCurrentActor().displayValue).toBe('How u doing?')
     })
   })