Skip to content

Commit

Permalink
test: re-enable power monitor tests on arm64 (#36023)
Browse files Browse the repository at this point in the history
test: re-enable powermonitor on arm64

Co-authored-by: John Kleinschmidt <jkleinsc@electronjs.org>

Co-authored-by: trop[bot] <37223003+trop[bot]@users.noreply.github.com>
Co-authored-by: John Kleinschmidt <jkleinsc@electronjs.org>
  • Loading branch information
trop[bot] and jkleinsc committed Oct 15, 2022
1 parent c0371ab commit 589199f
Showing 1 changed file with 1 addition and 2 deletions.
3 changes: 1 addition & 2 deletions spec-main/api-power-monitor-spec.ts
Original file line number Diff line number Diff line change
Expand Up @@ -14,8 +14,7 @@ import { promisify } from 'util';
describe('powerMonitor', () => {
let logindMock: any, dbusMockPowerMonitor: any, getCalls: any, emitSignal: any, reset: any;

// TODO(deepak1556): Enable on arm64 after upgrade, it crashes at the moment.
ifdescribe(process.platform === 'linux' && process.arch !== 'arm64' && process.env.DBUS_SYSTEM_BUS_ADDRESS != null)('when powerMonitor module is loaded with dbus mock', () => {
ifdescribe(process.platform === 'linux' && process.env.DBUS_SYSTEM_BUS_ADDRESS != null)('when powerMonitor module is loaded with dbus mock', () => {
before(async () => {
const systemBus = dbus.systemBus();
const loginService = systemBus.getService('org.freedesktop.login1');
Expand Down

0 comments on commit 589199f

Please sign in to comment.