Skip to content

Commit

Permalink
[Infra UI] Hosts flyout test fix (#161244)
Browse files Browse the repository at this point in the history
Closes #159368 

## Summary

I saw that the test was failing because of a missing environment set. I
guess that happened because the test runner doesn't include it like it
is included locally so I removed that check so the test shouldn't be
flaky.
  • Loading branch information
jennypavlova committed Jul 6, 2023
1 parent 569e873 commit 9d57196
Showing 1 changed file with 1 addition and 5 deletions.
6 changes: 1 addition & 5 deletions x-pack/test/functional/apps/infra/hosts_view.ts
Original file line number Diff line number Diff line change
Expand Up @@ -245,9 +245,7 @@ export default ({ getPageObjects, getService }: FtrProviderContext) => {
});
});

// FLAKY: https://github.com/elastic/kibana/issues/159368
// FLAKY: https://github.com/elastic/kibana/issues/159369
describe.skip('#Single host Flyout', () => {
describe('#Single host Flyout', () => {
before(async () => {
await setHostViewEnabled(true);
await pageObjects.common.navigateToApp(HOSTS_VIEW_PATH);
Expand Down Expand Up @@ -334,13 +332,11 @@ export default ({ getPageObjects, getService }: FtrProviderContext) => {

const query = decodeURIComponent(url.query ?? '');

const environment = 'environment=ENVIRONMENT_ALL';
const kuery = 'kuery=host.hostname:"Jennys-MBP.fritz.box"';
const rangeFrom = 'rangeFrom=2023-03-28T18:20:00.000Z';
const rangeTo = 'rangeTo=2023-03-28T18:21:00.000Z';

expect(url.pathname).to.eql('/app/apm/services');
expect(query).to.contain(environment);
expect(query).to.contain(kuery);
expect(query).to.contain(rangeFrom);
expect(query).to.contain(rangeTo);
Expand Down

0 comments on commit 9d57196

Please sign in to comment.