Add unit tests for new LogAction functionality.

Author: wbobeirne

src/action/log.js

@@ -57,6 +57,9 @@ export function error(...args) {
  * @return {string}
  */
 export function getLogPath(network) {
+  if (!_FS) {
+    throw new Error('Cannot get log path with no FS in action/log.js');
+  }
   const lndDir = _FS.DocumentDirectoryPath;
   return `${lndDir}/logs/bitcoin/${network}/lnd.log`;
 }
@@ -68,7 +71,7 @@ export function getLogPath(network) {
 export function getLogs() {
   if (!_FS) {
     return Promise.reject(
-      'Cannot get logs when no filesystem is available in action/log.js'
+      new Error('Cannot get logs with no FS in action/log.js')
     );
   }
   return _FS.readFile(getLogPath(_store.network), 'utf8');
@@ -81,7 +84,7 @@ export function getLogs() {
 export async function shareLogs() {
   if (!_Share) {
     return Promise.reject(
-      'Cannot share logs with no Share library in action/log.js'
+      new Error('Cannot share logs with no Share in action/log.js')
     );
   }
   const logs = await getLogs();

test/unit/action/log.spec.js

@@ -10,6 +10,14 @@ describe('Action Logs Unit Tests', () => {
   let ipc;
   let ipcRenderer;
 
+  const RNFS = {
+    DocumentDirectoryPath: '/foo/bar',
+    readFile: () => Promise.resolve('logs'),
+  };
+  const Share = {
+    share: () => Promise.resolve(true),
+  };
+
   beforeEach(() => {
     sandbox = sinon.createSandbox({});
     ipcRenderer = {
@@ -115,5 +123,38 @@ describe('Action Logs Unit Tests', () => {
         expect(store.logs.length, 'to equal', 56);
       });
     });
+
+    describe('getLogPath()', () => {
+      it('should throw without a FS constructor argument', () => {
+        expect(() => log.getLogPath(), 'to throw');
+      });
+
+      it('should return a log path with FS constructor argument', () => {
+        new LogAction(store, ipc, false, RNFS);
+        expect(log.getLogPath(), 'to be a string');
+      });
+    });
+
+    describe('getLogs()', () => {
+      it('should throw without a FS constructor argument', async () => {
+        await expect(log.getLogs(), 'to be rejected');
+      });
+
+      it('should return logs with an FS constructor argument', async () => {
+        new LogAction(store, ipc, false, RNFS);
+        expect(await log.getLogs(), 'to be a string');
+      });
+    });
+
+    describe('shareLogs()', () => {
+      it('should throw without a Share or FS constructor argument', async () => {
+        await expect(log.shareLogs(), 'to be rejected');
+      });
+
+      it('should share the logs with a Share and FS constructor argument', async () => {
+        new LogAction(store, ipc, false, RNFS, Share);
+        expect(await log.shareLogs(), 'to be ok');
+      });
+    });
   });
 });