Skip to content

Commit

Permalink
[ci] skip tests that fail on chrome 128+ (elastic#192830)
Browse files Browse the repository at this point in the history
## Summary
Currently `google-chrome-stable` is pinned to `v127.x.x` as with
`v128.x.x` we get a few FTR breakages (some of them on visual
inaccuracies, some other).

We'd like to unpin chrome, and move on to 128, and start fixing these
test failures. So we're skipping the failures temporarily, bumping
chrome to 128, then allow for unskipping and fixing these.
  • Loading branch information
delanni authored Sep 16, 2024
1 parent 0a385f3 commit f458714
Show file tree
Hide file tree
Showing 4 changed files with 9 additions and 5 deletions.
6 changes: 4 additions & 2 deletions test/functional/apps/dashboard/group5/embed_mode.ts
Original file line number Diff line number Diff line change
Expand Up @@ -57,7 +57,8 @@ export default function ({
await browser.setWindowSize(1300, 900);
});

describe('default URL params', () => {
// Fails in with chrome 128+ https://github.com/elastic/kibana/issues/163207
describe.skip('default URL params', () => {
it('hides the chrome', async () => {
const globalNavShown = await globalNav.exists();
expect(globalNavShown).to.be(true);
Expand Down Expand Up @@ -91,7 +92,8 @@ export default function ({
});
});

describe('non-default URL params', () => {
// Fails in with chrome 128+ https://github.com/elastic/kibana/issues/163207
describe.skip('non-default URL params', () => {
it('shows or hides elements based on URL params', async () => {
const currentUrl = await browser.getCurrentUrl();
const newUrl = [currentUrl].concat(urlParamExtensions).join('&');
Expand Down
2 changes: 1 addition & 1 deletion test/functional/apps/kibana_overview/_analytics.ts
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,7 @@ export default function ({ getService, getPageObjects }: FtrProviderContext) {
const kibanaServer = getService('kibanaServer');
const PageObjects = getPageObjects(['common', 'header']);

// Failing: See https://github.com/elastic/kibana/issues/192509
// Fails in chrome 128+: See https://github.com/elastic/kibana/issues/192509
describe.skip('overview page - Analytics apps', function describeIndexTests() {
before(async () => {
await esArchiver.load('test/functional/fixtures/es_archiver/logstash_functional');
Expand Down
3 changes: 2 additions & 1 deletion x-pack/test/functional/apps/lens/group6/workspace_size.ts
Original file line number Diff line number Diff line change
Expand Up @@ -268,7 +268,8 @@ export default function ({ getService, getPageObjects }: FtrProviderContext) {
await assertWorkspaceDimensions('600px', '375px');
});

it('gauge size (absolute pixels) - major arc', async () => {
// Fails in chrome 128+
it.skip('gauge size (absolute pixels) - major arc', async () => {
await lens.openVisualOptions();
await lens.setGaugeShape('Major arc');
await assertWorkspaceDimensions('600px', '430px');
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -108,7 +108,8 @@ export default function ({ getPageObjects, getService }) {
expect(hits).to.equal('2');
});

it('should apply layer query to fit to bounds', async () => {
// Fails in chrome 128+: https://github.com/elastic/kibana/issues/175378
it.skip('should apply layer query to fit to bounds', async () => {
// Set view to other side of world so no matching results
await maps.setView(-15, -100, 6);
await maps.clickFitToBounds('logstash');
Expand Down

0 comments on commit f458714

Please sign in to comment.