From 65081e89392aa61c929af8df816390245823235f Mon Sep 17 00:00:00 2001 From: "trop[bot]" <37223003+trop[bot]@users.noreply.github.com> Date: Mon, 28 Apr 2025 16:42:36 -0500 Subject: [PATCH] fix: run `visibleOnAllWorkspaces` tests on the right platforms (#46834) * test: add platform test on visibleOnAllWorkspaces tests visibleOnAllWorkspaces is not supported on Windows Co-authored-by: Charles Kerr * test: do not skip visibleOnAllWorkspaces tests on Windows That feature is supported on Linux, so move the test from the "window states (excluding Linux)" section into the "window states" section. Co-authored-by: Charles Kerr * fix: nested it() calls in visibleOnAllWorkspaces specs Co-authored-by: Charles Kerr * chore: make the process.platform test simpler Co-authored-by: Charles Kerr * chore: disable visibleOnAllWorkspaces test on Linux --------- Co-authored-by: trop[bot] <37223003+trop[bot]@users.noreply.github.com> Co-authored-by: Charles Kerr --- spec/api-browser-window-spec.ts | 41 +++++++++++++++++---------------- 1 file changed, 21 insertions(+), 20 deletions(-) diff --git a/spec/api-browser-window-spec.ts b/spec/api-browser-window-spec.ts index c421f3cb094f..163be971dab9 100755 --- a/spec/api-browser-window-spec.ts +++ b/spec/api-browser-window-spec.ts @@ -5407,6 +5407,27 @@ describe('BrowserWindow module', () => { expect(w.webContents.isLoadingMainFrame()).to.be.true('isLoadingMainFrame'); }); }); + + // FIXME: enable this test on Linux as well. + ifdescribe(process.platform === 'darwin')('visibleOnAllWorkspaces state', () => { + describe('with properties', () => { + it('can be changed', () => { + const w = new BrowserWindow({ show: false }); + expect(w.visibleOnAllWorkspaces).to.be.false(); + w.visibleOnAllWorkspaces = true; + expect(w.visibleOnAllWorkspaces).to.be.true(); + }); + }); + + describe('with functions', () => { + it('can be changed', () => { + const w = new BrowserWindow({ show: false }); + expect(w.isVisibleOnAllWorkspaces()).to.be.false(); + w.setVisibleOnAllWorkspaces(true); + expect(w.isVisibleOnAllWorkspaces()).to.be.true(); + }); + }); + }); }); ifdescribe(process.platform !== 'linux')('window states (excluding Linux)', () => { @@ -5447,26 +5468,6 @@ describe('BrowserWindow module', () => { }); }); - describe('visibleOnAllWorkspaces state', () => { - it('with properties', () => { - it('can be changed', () => { - const w = new BrowserWindow({ show: false }); - expect(w.visibleOnAllWorkspaces).to.be.false(); - w.visibleOnAllWorkspaces = true; - expect(w.visibleOnAllWorkspaces).to.be.true(); - }); - }); - - it('with functions', () => { - it('can be changed', () => { - const w = new BrowserWindow({ show: false }); - expect(w.isVisibleOnAllWorkspaces()).to.be.false(); - w.setVisibleOnAllWorkspaces(true); - expect(w.isVisibleOnAllWorkspaces()).to.be.true(); - }); - }); - }); - ifdescribe(process.platform === 'darwin')('documentEdited state', () => { it('with properties', () => { it('can be changed', () => {