fix: only check updates, don't use built in notify

davidwinter · Jun 10, 2021 · 2030c16 · 2030c16
1 parent 1e54361
commit 2030c16
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 5 deletions.
diff --git a/auto-update-stub.js b/auto-update-stub.js
@@ -19,7 +19,7 @@ class AutoUpdateStub extends EventEmitter {
 		}
 	}
 
-	checkForUpdatesAndNotify() {
+	checkForUpdates() {
 		if (this.downloadAvailable) {
 			this.emit('update-downloaded');
 		} else {

diff --git a/index.js b/index.js
@@ -44,11 +44,11 @@ class ElectronAutoUpdate {
 			this.logger.info(`Will check again in ${this.checkFrequency} milliseconds`);
 
 			setTimeout(() => {
-				this.autoUpdater.checkForUpdatesAndNotify();
+				this.autoUpdater.checkForUpdates();
 			}, this.checkFrequency);
 		});
 
-		this.autoUpdater.checkForUpdatesAndNotify();
+		this.autoUpdater.checkForUpdates();
 	}
 
 	async updateDownloaded() {
@@ -68,7 +68,7 @@ class ElectronAutoUpdate {
 			this.logger.info('Will remind later to install update');
 
 			setTimeout(() => {
-				this.autoUpdater.checkForUpdatesAndNotify();
+				this.autoUpdater.checkForUpdates();
 			}, this.checkFrequency);
 		}
 	}

diff --git a/test.js b/test.js
@@ -63,7 +63,7 @@ test('checks for updates at the frequency specified', t => {
 		checkFrequency: 5,
 		electronUpdater: {
 			autoUpdater: {
-				checkForUpdatesAndNotify: checkSpy,
+				checkForUpdates: checkSpy,
 				on: (eventName, fn) => {
 					if (eventName === 'update-not-available') {
 						fn();