fix linux SIGTRAP related to power monitor (#1670)

* fix linux SIGTRAP related to power monitor * update tests --------- Co-authored-by: Jordan Muir <[email protected]>
floating · Sep 26, 2023 · a223705 · a223705
1 parent 42663d5
commit a223705
Show file tree

Hide file tree

Showing 4 changed files with 27 additions and 26 deletions.
diff --git a/main/chains/index.js b/main/chains/index.js
@@ -1,6 +1,6 @@
 // status = Network Mismatch, Not Connected, Connected, Standby, Syncing
 
-const { powerMonitor } = require('electron')
+const { app, powerMonitor } = require('electron')
 const EventEmitter = require('events')
 const { addHexPrefix } = require('@ethereumjs/util')
 const { Hardfork } = require('@ethereumjs/common')
@@ -491,19 +491,21 @@ class Chains extends EventEmitter {
       })
     }
 
-    powerMonitor.on('resume', () => {
-      const activeConnections = Object.keys(this.connections)
-        .map((type) => Object.keys(this.connections[type]).map((chainId) => `${type}:${chainId}`))
-        .flat()
+    app.on('ready', () => {
+      powerMonitor.on('resume', () => {
+        const activeConnections = Object.keys(this.connections)
+          .map((type) => Object.keys(this.connections[type]).map((chainId) => `${type}:${chainId}`))
+          .flat()
 
-      log.info('System resuming, resetting active connections', { chains: activeConnections })
+        log.info('System resuming, resetting active connections', { chains: activeConnections })
 
-      activeConnections.forEach((id) => {
-        const [type, chainId] = id.split(':')
-        removeConnection(chainId, type)
-      })
+        activeConnections.forEach((id) => {
+          const [type, chainId] = id.split(':')
+          removeConnection(chainId, type)
+        })
 
-      updateConnections()
+        updateConnections()
+      })
     })
 
     store.observer(updateConnections, 'chains:connections')

diff --git a/package-lock.json b/package-lock.json
diff --git a/package.json b/package.json
@@ -158,7 +158,7 @@
     "@types/zxcvbn": "4.4.1",
     "@typescript-eslint/eslint-plugin": "5.54.1",
     "@typescript-eslint/parser": "5.54.1",
-    "electron": "25.8.2",
+    "electron": "26.2.2",
     "electron-builder": "23.6.0",
     "electron-devtools-installer": "3.2.0",
     "electron-extension-installer": "1.2.0",

diff --git a/test/__mocks__/electron.js b/test/__mocks__/electron.js
@@ -4,14 +4,13 @@ const dataPaths = {
   userData: './test/.userData'
 }
 
-const app = {
-  getVersion: () => '0.1',
-  getName: () => 'frame-test-app',
-  getPath: (key) => dataPaths[key]
-}
-
-const setDataPath = (key, path) => (dataPaths[key] = path)
+const app = new EventEmitter()
+app.getVersion = () => '0.1'
+app.getName = () => 'frame-test-app'
+app.getPath = (key) => dataPaths[key]
 
 const powerMonitor = new EventEmitter()
 
+const setDataPath = (key, path) => (dataPaths[key] = path)
+
 export { app, setDataPath as _setDataPath, powerMonitor }