Seperate getNethash and requestToActivePeer to avoid mock problems in…

… tests

src/actions/peers.js

@@ -1,6 +1,6 @@
 import Lisk from 'lisk-js';
 import actionTypes from '../constants/actions';
-import { getNethash } from './../utils/api/peers';
+import { getNethash } from './../utils/api/nethash';
 
 const peerSet = (data, config) => ({
   data: Object.assign({

src/actions/peers.test.js

@@ -2,7 +2,7 @@ import { expect } from 'chai';
 import { spy, stub, match } from 'sinon';
 import actionTypes from '../constants/actions';
 import { activePeerSet, activePeerUpdate } from './peers';
-import * as peersApi from './../utils/api/peers';
+import * as nethashApi from './../utils/api/nethash';
 
 
 describe('actions: peers', () => {
@@ -29,7 +29,7 @@ describe('actions: peers', () => {
 
     beforeEach(() => {
       dispatch = spy();
-      getNetHash = stub(peersApi, 'getNethash');
+      getNetHash = stub(nethashApi, 'getNethash');
     });
 
     afterEach(() => {

src/utils/api/nethash.js

@@ -0,0 +1,6 @@
+import { requestToActivePeer } from './peers';
+
+/* eslint-disable  */
+export const getNethash = activePeer => (requestToActivePeer(activePeer, 'blocks/getNethash'));
+/* eslint-enable  */
+

src/utils/api/nethash.test.js

@@ -0,0 +1,29 @@
+import { expect } from 'chai';
+import { mock } from 'sinon';
+import * as peers from './peers';
+import { getNethash } from './nethash';
+
+
+describe('Utils: Nethash', () => {
+  let peersMock;
+  const activePeer = {};
+
+  beforeEach(() => {
+    peersMock = mock(peers);
+  });
+
+  afterEach(() => {
+    peersMock.restore();
+  });
+
+  it('should return the result from requestToActivePeer call', () => {
+    const mockedReturns = 'requestToActivePeer returns something';
+
+    peersMock.expects('requestToActivePeer')
+      .withArgs(activePeer, 'blocks/getNethash')
+      .returns(mockedReturns);
+
+    const returnedPromise = getNethash(activePeer);
+    expect(returnedPromise).to.equal(mockedReturns);
+  });
+});

src/utils/api/peers.js

@@ -1,6 +1,7 @@
 import { loadingStarted, loadingFinished } from '../../utils/loading';
 
-const requestToActivePeer = (activePeer, path, urlParams) =>
+/* eslint-disable  */
+export const requestToActivePeer = (activePeer, path, urlParams) =>
   new Promise((resolve, reject) => {
     loadingStarted(path);
     activePeer.sendRequest(path, urlParams, (data) => {
@@ -12,8 +13,4 @@ const requestToActivePeer = (activePeer, path, urlParams) =>
       loadingFinished(path);
     });
   });
-
-
-const getNethash = activePeer => (requestToActivePeer(activePeer, 'blocks/getNethash'));
-
-export { requestToActivePeer, getNethash };
+/* eslint-enable */