fix unit tests

mattermost · Oct 8, 2024 · c39b05f · c39b05f
1 parent a558975
commit c39b05f
Show file tree

Hide file tree

Showing 10 changed files with 24 additions and 0 deletions.
diff --git a/src/app/serverViewState.test.js b/src/app/serverViewState.test.js
@@ -15,6 +15,9 @@ import MainWindow from 'main/windows/mainWindow';
 import {ServerViewState} from './serverViewState';
 
 jest.mock('electron', () => ({
+    app: {
+        getPath: jest.fn(() => '/valid/downloads/path'),
+    },
     ipcMain: {
         on: jest.fn(),
         handle: jest.fn(),

diff --git a/src/common/config/index.test.js b/src/common/config/index.test.js
@@ -94,6 +94,12 @@ jest.mock('common/config/RegistryConfig', () => {
     return jest.fn();
 });
 
+jest.mock('electron', () => ({
+    app: {
+        getPath: jest.fn(() => '/valid/downloads/path'),
+    },
+}));
+
 describe('common/config', () => {
     it('should load buildConfig', () => {
         const config = new Config();

diff --git a/src/common/config/upgradePreferences.test.js b/src/common/config/upgradePreferences.test.js
@@ -18,6 +18,12 @@ jest.mock('common/views/View', () => ({
     }),
 }));
 
+jest.mock('electron', () => ({
+    app: {
+        getPath: jest.fn(() => '/valid/downloads/path'),
+    },
+}));
+
 describe('common/config/upgradePreferences', () => {
     describe('upgradeV0toV1', () => {
         it('should upgrade from v0', () => {

diff --git a/src/main/downloadsManager.test.js b/src/main/downloadsManager.test.js
@@ -28,6 +28,7 @@ jest.mock('electron', () => {
     return {
         app: {
             getAppPath: jest.fn(),
+            getPath: jest.fn(() => '/valid/downloads/path'),
         },
         BrowserView: jest.fn().mockImplementation(() => ({
             webContents: {

diff --git a/src/main/tray/tray.test.js b/src/main/tray/tray.test.js
@@ -30,6 +30,7 @@ jest.mock('electron', () => {
             getAppPath: () => '/path/to/app',
             isReady: jest.fn(),
             setPath: jest.fn(),
+            getPath: jest.fn(() => '/valid/downloads/path'),
         },
         ipcMain: {
             emit: jest.fn(),

diff --git a/src/main/views/MattermostBrowserView.test.js b/src/main/views/MattermostBrowserView.test.js
@@ -17,6 +17,7 @@ import MainWindow from '../windows/mainWindow';
 jest.mock('electron', () => ({
     app: {
         getVersion: () => '5.0.0',
+        getPath: jest.fn(() => '/valid/downloads/path'),
     },
     BrowserView: jest.fn().mockImplementation(() => ({
         webContents: {

diff --git a/src/main/views/downloadsDropdownMenuView.test.js b/src/main/views/downloadsDropdownMenuView.test.js
@@ -28,6 +28,7 @@ jest.mock('electron', () => {
     return {
         app: {
             getAppPath: () => '',
+            getPath: jest.fn(() => '/valid/downloads/path'),
         },
         BrowserView: jest.fn().mockImplementation(() => ({
             webContents: {

diff --git a/src/main/views/downloadsDropdownView.test.js b/src/main/views/downloadsDropdownView.test.js
@@ -36,6 +36,7 @@ jest.mock('electron', () => {
     return {
         app: {
             getAppPath: () => '',
+            getPath: jest.fn(() => '/valid/downloads/path'),
         },
         BrowserView: jest.fn().mockImplementation(() => ({
             webContents: {

diff --git a/src/main/views/serverDropdownView.test.js b/src/main/views/serverDropdownView.test.js
@@ -25,6 +25,9 @@ jest.mock('electron', () => ({
     ipcMain: {
         on: jest.fn(),
     },
+    app: {
+        getPath: jest.fn(() => '/valid/downloads/path'),
+    },
 }));
 jest.mock('main/windows/mainWindow', () => ({
     on: jest.fn(),

diff --git a/src/main/views/viewManager.test.js b/src/main/views/viewManager.test.js
@@ -18,6 +18,7 @@ import {ViewManager} from './viewManager';
 jest.mock('electron', () => ({
     app: {
         getAppPath: () => '/path/to/app',
+        getPath: jest.fn(() => '/valid/downloads/path'),
     },
     dialog: {
         showErrorBox: jest.fn(),