test: properly mock electron

jely2002 · Feb 2, 2022 · cf7109c · cf7109c
1 parent c75a13f
commit cf7109c
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 8 deletions.
diff --git a/.github/workflows/continuous-integration.yaml b/.github/workflows/continuous-integration.yaml
@@ -25,12 +25,7 @@ jobs:
  node-version: '12'
  - name: Install npm modules
  run: npm install
- - name: Check if electron is installed
- run: cat node_modules/electron/path.txt
  - name: Run Jest
- run: |
- npm uninstall electron
- npm install electron
- npm run test
+ run: npm run test
  - name: Upload coverage to codecov
  uses: codecov/codecov-action@v1
diff --git a/tests/Settings.test.js b/tests/Settings.test.js
@@ -1,7 +1,17 @@
 const fs = require('fs').promises;
 const os = require('os');
+const { globalShortcut, clipboard } = require('electron');
 
-jest.mock('electron');
+jest.mock('electron', () => ({
+ clipboard: {
+ readText: jest.fn()
+ },
+ globalShortcut: {
+ unregisterAll: jest.fn(),
+ isRegistered: jest.fn(),
+ register: jest.fn(),
+ }
+}));
 
 const Settings = require('../modules/persistence/Settings');
 const env = {version: '2.0.0-test1', app: {getPath: jest.fn().mockReturnValue('test/path')}};
@@ -44,7 +54,6 @@ const defaultSettings = {
 describe('Load settings from file', () => {
  beforeEach(() => {
  jest.clearAllMocks();
- jest.mock('electron');
  fs.writeFile = jest.fn().mockResolvedValue('');
  console.log = jest.fn().mockImplementation(() => {
  });