Skip to content

Commit

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

This will backport the following commits from `main` to `8.9`:
- [[Infra UI] Hosts flyout test fix
(#161244)](#161244)

<!--- Backport version: 8.9.7 -->

### Questions ?
Please refer to the [Backport tool
documentation](https://github.com/sqren/backport)

<!--BACKPORT
[{"author":{"name":"jennypavlova","email":"dzheni.pavlova@elastic.co"},"sourceCommit":{"committedDate":"2023-07-06T07:50:40Z","message":"[Infra
UI] Hosts flyout test fix (#161244)\n\nCloses #159368 \r\n\r\n##
Summary\r\n\r\nI saw that the test was failing because of a missing
environment set. I\r\nguess that happened because the test runner
doesn't include it like it\r\nis included locally so I removed that
check so the test shouldn't
be\r\nflaky.","sha":"9d57196a54ebe7816d2739ee5b85dcc76a71b253","branchLabelMapping":{"^v8.10.0$":"main","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["Team:Infra
Monitoring
UI","release_note:skip","backport:prev-minor","v8.10.0"],"number":161244,"url":"https://github.com/elastic/kibana/pull/161244","mergeCommit":{"message":"[Infra
UI] Hosts flyout test fix (#161244)\n\nCloses #159368 \r\n\r\n##
Summary\r\n\r\nI saw that the test was failing because of a missing
environment set. I\r\nguess that happened because the test runner
doesn't include it like it\r\nis included locally so I removed that
check so the test shouldn't
be\r\nflaky.","sha":"9d57196a54ebe7816d2739ee5b85dcc76a71b253"}},"sourceBranch":"main","suggestedTargetBranches":[],"targetPullRequestStates":[{"branch":"main","label":"v8.10.0","labelRegex":"^v8.10.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/161244","number":161244,"mergeCommit":{"message":"[Infra
UI] Hosts flyout test fix (#161244)\n\nCloses #159368 \r\n\r\n##
Summary\r\n\r\nI saw that the test was failing because of a missing
environment set. I\r\nguess that happened because the test runner
doesn't include it like it\r\nis included locally so I removed that
check so the test shouldn't
be\r\nflaky.","sha":"9d57196a54ebe7816d2739ee5b85dcc76a71b253"}}]}]
BACKPORT-->

Co-authored-by: jennypavlova <dzheni.pavlova@elastic.co>
  • Loading branch information
kibanamachine and jennypavlova authored Jul 12, 2023
1 parent 482b747 commit 8c14f31
Showing 1 changed file with 0 additions and 2 deletions.
2 changes: 0 additions & 2 deletions x-pack/test/functional/apps/infra/hosts_view.ts
Original file line number Diff line number Diff line change
Expand Up @@ -332,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 8c14f31

Please sign in to comment.