coverage for _new_state/1 and machine_state/0

StoneCypher · Jul 5, 2017 · e3212a8 · e3212a8
1 parent 09ed3cc
commit e3212a8
Show file tree

Hide file tree

Showing 6 changed files with 43 additions and 6 deletions.
diff --git a/dist/jssm.es5.cjs.js b/dist/jssm.es5.cjs.js
diff --git a/dist/jssm.es5.cjs.min.js b/dist/jssm.es5.cjs.min.js
diff --git a/docs/lib/index.html b/docs/lib/index.html
@@ -2,7 +2,7 @@
 <html>
 <head>
   <meta charset='utf-8' />
-  <title>jssm 2.9.14 | Documentation</title>
+  <title>jssm 2.9.16 | Documentation</title>
   <meta name='viewport' content='width=device-width,initial-scale=1'>
   <link href='assets/bass.css' type='text/css' rel='stylesheet' />
   <link href='assets/style.css' type='text/css' rel='stylesheet' />
@@ -14,7 +14,7 @@
       <div class='fixed xs-hide fix-3 overflow-auto max-height-100'>
         <div class='py1 px2'>
           <h3 class='mb0 no-anchor'>jssm</h3>
-          <div class='mb1'><code>2.9.14</code></div>
+          <div class='mb1'><code>2.9.16</code></div>
           <input
             placeholder='Filter'
             id='filter-input'

diff --git a/package.json b/package.json
@@ -1,6 +1,6 @@
 {
   "name": "jssm",
-  "version": "2.9.14",
+  "version": "2.9.16",
   "engines": {
     "node": ">=6.0.0"
   },

diff --git a/src/js/jssm.js b/src/js/jssm.js
@@ -148,9 +148,14 @@ class machine<mNT, mDT> {
   }
 
   _new_state(state_config : JssmGenericState<mNT>) : mNT { // whargarbl get that state_config any under control
-    if (this._states.has(state_config.name)) { throw new Error(`state ${(state_config.name:any)} already exists`); }
+
+    if (this._states.has(state_config.name)) {
+      throw new Error(`state ${(state_config.name:any)} already exists`);
+    }
+
     this._states.set(state_config.name, state_config);
     return state_config.name;
+
   }
 
 

diff --git a/src/js/tests/general.js b/src/js/tests/general.js
@@ -402,6 +402,21 @@ describe('reports on states', async it => {
 
 
 
+describe('returns states', async it => {
+
+  const machine = new jssm.machine({
+    initial_state: 'off',
+    transitions:[ { name:'turn_on', action:'power_on', from:'off', to:'red'} ]
+  });
+
+  it('that it has', t => t.is('object', typeof machine.machine_state() ) );
+
+});
+
+
+
+
+
 describe('reports on transitions', async it => {
 
   const machine = new jssm.machine({
@@ -549,4 +564,19 @@ describe('Illegal machines', async it => {
   }, Error));
 
 
+  it('internal state helper must not accept double states', t => t.throws(() => {
+
+    const machine = new jssm.machine({
+      initial_state: 'moot',
+      transitions:[
+        { name:'id1', from:'1', to:'2', action:'identical' }
+      ]
+    });
+
+    machine._new_state({from: '1', name:'id1', to:'2', complete:false});
+    machine._new_state({from: '1', name:'id1', to:'2', complete:false});
+
+  }, Error));
+
+
 });