Merge pull request #10 from linode/tests/linode-reducers

Fix our usage of sinon
linode · May 24, 2016 · b67bd09 · b67bd09
2 parents 0c0abed + 0960519
commit b67bd09
Show file tree

Hide file tree

Showing 4 changed files with 297 additions and 262 deletions.
diff --git a/test/actions/api/linodes.spec.js b/test/actions/api/linodes.spec.js
@@ -1,4 +1,5 @@
 import sinon from 'sinon';
+import { expect } from 'chai';
 import {
   powerOnLinode,
   powerOffLinode,
@@ -13,20 +14,30 @@ describe("actions/linodes/power", sinon.test(() => {
     linode: { id: "foo", state: "booting" }
   };
 
+  let sandbox = null;
+
+  beforeEach(() => {
+    sandbox = sinon.sandbox.create();
+  });
+
+  afterEach(() => {
+    sandbox.restore();
+  });
+
   it('returns linode power boot status', async () => {
-    await mock_context(async ({
+    await mock_context(sandbox, async ({
         auth, dispatch, getState, fetchStub
       }) => {
       const f = powerOnLinode("foo");
 
       await f(dispatch, getState);
 
-      sinon.assert.calledWith(fetchStub,
-        auth.token, '/linodes/foo/boot', { method: "POST" });
-      sinon.assert.calledWith(dispatch, {
+      expect(fetchStub.calledWith(
+        auth.token, '/linodes/foo/boot', { method: "POST" })).to.be.true;
+      expect(dispatch.calledWith({
         type: UPDATE_LINODE,
         linode: { id: "foo", state: "booting" }
-      });
+      })).to.be.true;
     }, mock_booting_response);
   });
 
@@ -36,19 +47,19 @@ describe("actions/linodes/power", sinon.test(() => {
   };
 
   it('returns linode power shutdown status', async () => {
-    await mock_context(async ({
+    await mock_context(sandbox, async ({
         auth, dispatch, getState, fetchStub
       }) => {
       const f = powerOffLinode("foo");
 
       await f(dispatch, getState);
 
-      sinon.assert.calledWith(fetchStub,
-        auth.token, '/linodes/foo/shutdown', { method: "POST" });
-      sinon.assert.calledWith(dispatch, {
+      expect(fetchStub.calledWith(
+        auth.token, '/linodes/foo/shutdown', { method: "POST" })).to.be.true;
+      expect(dispatch.calledWith({
         type: UPDATE_LINODE,
         linode: { id: "foo", state: "shutting_down" }
-      });
+      })).to.be.true;
     }, mock_shutting_down_response);
   });
 
@@ -58,19 +69,19 @@ describe("actions/linodes/power", sinon.test(() => {
   };
 
   it('returns linode power reboot status', async () => {
-    await mock_context(async ({
+    await mock_context(sandbox, async ({
         auth, dispatch, getState, fetchStub
       }) => {
       const f = rebootLinode("foo");
 
       await f(dispatch, getState);
 
-      sinon.assert.calledWith(fetchStub,
-        auth.token, '/linodes/foo/reboot', { method: "POST" });
-      sinon.assert.calledWith(dispatch, {
+      expect(fetchStub.calledWith(
+        auth.token, '/linodes/foo/reboot', { method: "POST" })).to.be.true;
+      expect(dispatch.calledWith({
         type: UPDATE_LINODE,
         linode: { id: "foo", state: "rebooting" }
-      });
+      })).to.be.true;
     }, mock_rebooting_response);
   });
 }));