Merge pull request #811 from h3poteto/iss-209

refs #209 Fix Login spec to use ipc mock

package-lock.json

@@ -5671,6 +5671,12 @@
         "sumchecker": "2.0.2"
       }
     },
+    "electron-ipc-mock": {
+      "version": "0.0.3",
+      "resolved": "https://registry.npmjs.org/electron-ipc-mock/-/electron-ipc-mock-0.0.3.tgz",
+      "integrity": "sha1-7sEXxVEO7KfaaCkaikG/x4aNPgM=",
+      "dev": true
+    },
     "electron-is-accelerator": {
       "version": "0.1.2",
       "resolved": "https://registry.npmjs.org/electron-is-accelerator/-/electron-is-accelerator-0.1.2.tgz",

package.json

@@ -39,7 +39,7 @@
     "test": "npm run unit && npm run e2e",
     "unit": "karma start test/unit/karma.conf.js",
     "postinstall": "npm run lint:fix",
-    "spec:unit": "BABEL_ENV=test jest -u ./spec/unit/*/**.spec.js"
+    "spec:unit": "NODE_ENV=test jest -u ./spec/unit/*/**.spec.js"
   },
   "build": {
     "productName": "Whalebird",
@@ -165,6 +165,7 @@
     "electron-builder": "^20.38.2",
     "electron-debug": "^2.0.0",
     "electron-devtools-installer": "^2.2.4",
+    "electron-ipc-mock": "0.0.3",
     "electron-packager": "^12.2.0",
     "eslint": "^5.9.0",
     "eslint-config-standard": "^12.0.0",

spec/mock/electron.js

@@ -1,6 +1,9 @@
-export const ipcRenderer = {
-  send: jest.fn(),
-  on: jest.fn(),
-  once: jest.fn(),
-  removeAllListeners: jest.fn()
-}
+const { ipcRenderer, ipcMain } = require('electron-ipc-mock')()
+// export const ipcRenderer = {
+//   send: jest.fn(),
+//   on: jest.fn(),
+//   once: jest.fn(),
+//   removeAllListeners: jest.fn()
+// }
+module.exports.ipcRenderer = ipcRenderer
+module.exports.ipcMain = ipcMain

spec/unit/store/Login.spec.js

@@ -1,6 +1,6 @@
 import axios from 'axios'
 import Login from '@/store/Login'
-import { ipcRenderer } from '~/spec/mock/electron'
+import { ipcMain } from '~/spec/mock/electron'
 
 jest.mock('axios')
 
@@ -29,17 +29,39 @@ describe('Login', () => {
   })
 
   describe('actions', () => {
-    describe('fetchLogin', async () => {
+    describe('fetchLogin', () => {
+      describe('error', () => {
+        it('should return error', async () => {
+          ipcMain.once('get-auth-url', (event, instance) => {
+            event.sender.send('error-get-auth-url', new AuthError())
+          })
           const commitMock = jest.fn()
           await Login.actions.fetchLogin({ commit: commitMock }, 'pleroma.io')
-      expect(ipcRenderer.send).toHaveBeenCalledWith('get-auth-url', 'pleroma.io')
+            .catch((err) => {
+              expect(err instanceof AuthError).toEqual(true)
+            })
+        })
+      })
+      describe('success', () => {
+        it('should return url', async () => {
+          ipcMain.once('get-auth-url', (event, instance) => {
+            event.sender.send('response-get-auth-url', 'http://example.com/auth')
+          })
+          const commitMock = jest.fn()
+          const url = await Login.actions.fetchLogin({ commit: commitMock }, 'pleroma.io')
+          expect(url).toEqual('http://example.com/auth')
+        })
+      })
     })
     describe('pageBack', () => {
+      it('should reset instance', () => {
         const commitMock = jest.fn()
         Login.actions.pageBack({ commit: commitMock })
         expect(commitMock).toHaveBeenCalledWith('changeInstance', null)
       })
-    describe('confirmInstance', async () => {
+    })
+    describe('confirmInstance', () => {
+      it('should change instance', async () => {
         const resp = {
           data: 'test'
         }
@@ -53,4 +75,7 @@ describe('Login', () => {
         expect(commitMock).toHaveBeenCalledWith('changeInstance', 'pleroma.io')
       })
     })
+  })
 })
+
+class AuthError extends Error {}