From 65318286523f08b6fac6c1c2f1c11517dce433ff Mon Sep 17 00:00:00 2001 From: Matt Schile Date: Thu, 12 Dec 2024 19:16:02 -0700 Subject: [PATCH] test: fix default size system test (#30759) --- .../projects/screen-size/cypress/e2e/default_size.cy.js | 9 --------- 1 file changed, 9 deletions(-) diff --git a/system-tests/projects/screen-size/cypress/e2e/default_size.cy.js b/system-tests/projects/screen-size/cypress/e2e/default_size.cy.js index 3e6b8257d5c8..c3f0a6db0f34 100644 --- a/system-tests/projects/screen-size/cypress/e2e/default_size.cy.js +++ b/system-tests/projects/screen-size/cypress/e2e/default_size.cy.js @@ -2,7 +2,6 @@ describe('windowSize', () => { it('spawns with correct default size', () => { // assert the browser was spawned at 1280x720 and is full size // normally e2e tests spawn at fixed size, but this spec should be spawned without passing any width/height arguments in plugins file. - // TODO: look into fixing screen/available height and width if (Cypress.browser.name === 'chrome') { // NOTE: there is a bug in chrome headless=new where height is not spawned correctly // the issue is marked as fixed, but others are still running into it in Chrome 116 @@ -14,14 +13,6 @@ describe('windowSize', () => { innerWidth: 1280, innerHeight: 581, // chrome 128 decreased the size here from 633 to 581 }) - } else if (Cypress.browser.name === 'firefox') { - expect({ - innerWidth: top.window.innerWidth, - innerHeight: top.window.innerHeight, - }).deep.eq({ - innerWidth: 1280, - innerHeight: Cypress.env('CI') ? 676 : 677, // firefox 133 decreased the size here from 720 to 676/677 - }) } else { expect({ innerWidth: top.window.innerWidth,