Skip to content

Commit

Permalink
Part of #1117: Update electron to v34 (#1123)
Browse files Browse the repository at this point in the history
* Part of #1117: Update electron from 32.2.8 to 33.3.1

Need to bump to node 20.18

* Part of #1117: Update electron from 33.3.1 to 34.0.0

* Update listener count for 'close' event on main window

* Calling BrowserWindow.close() rather than emit('close') on failing test

* Part of #1117: Update electron from 34.0.0 to 34.0.1
  • Loading branch information
tupaschoal authored Jan 25, 2025
1 parent 774787f commit 69efea2
Show file tree
Hide file tree
Showing 5 changed files with 793 additions and 118 deletions.
2 changes: 1 addition & 1 deletion DEVELOPMENT.md
Original file line number Diff line number Diff line change
Expand Up @@ -27,7 +27,7 @@ Working on your first Pull Request? You can learn how from this _free_ series [H

## Environment

- Install [Node.js](https://nodejs.org/en/): version 20.12 or higher
- Install [Node.js](https://nodejs.org/en/): version 20.18 or higher
- Install [git](https://git-scm.com/)

## Setup
Expand Down
2 changes: 1 addition & 1 deletion __tests__/__main__/main-window.mjs
Original file line number Diff line number Diff line change
Expand Up @@ -79,7 +79,7 @@ describe('main-window.mjs', () =>
assert.strictEqual(ipcMain.listenerCount('SWITCH_VIEW'), 1);
assert.strictEqual(ipcMain.listenerCount('RECEIVE_LEAVE_BY'), 1);
assert.strictEqual(mainWindow.listenerCount('minimize'), 2);
assert.strictEqual(mainWindow.listenerCount('close'), 1);
assert.strictEqual(mainWindow.listenerCount('close'), 2);
assert.strictEqual(loadFileSpy.calledOnce, true);
assert.strictEqual(showSpy.calledOnce, true);
assert.notStrictEqual(getLeaveByInterval(), null);
Expand Down
2 changes: 1 addition & 1 deletion __tests__/__main__/windows.mjs
Original file line number Diff line number Diff line change
Expand Up @@ -78,7 +78,7 @@ describe('Windows tests', () =>
show: false
});
Windows.openWaiverManagerWindow(mainWindow, true);
Windows.getWaiverWindow().emit('close');
Windows.getWaiverWindow().close();
assert.strictEqual(Windows.getWaiverWindow(), null);
});

Expand Down
Loading

0 comments on commit 69efea2

Please sign in to comment.