Skip to content

Commit

Permalink
[8.x] [Inventory] Fix the link to discover test (#201197) (#201252)
Browse files Browse the repository at this point in the history
# Backport

This will backport the following commits from `main` to `8.x`:
- [[Inventory] Fix the link to discover test
(#201197)](#201197)

<!--- Backport version: 9.4.3 -->

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

<!--BACKPORT
[{"author":{"name":"jennypavlova","email":"dzheni.pavlova@elastic.co"},"sourceCommit":{"committedDate":"2024-11-21T17:14:43Z","message":"[Inventory]
Fix the link to discover test (#201197)\n\nCloses #201189\r\n\r\n##
Summary\r\n\r\nAfter this fix was added
in\r\nhttps://github.com//pull/200984 the test started
failing\r\nas it was verifying the previous kuery value - it was missing
the `\" \"`\r\nso after this bug was fixed the test should be updated as
well\r\n(basically changing `container.id:foo` with
`container.id:\"foo\"`) and\r\nthis PR updates the test.\r\n\r\nI
checked locally and the test is passing
now.","sha":"74cf5d45784b9f633decae428c61ff96281dcc4b","branchLabelMapping":{"^v9.0.0$":"main","^v8.18.0$":"8.x","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:skip","v9.0.0","backport:prev-minor","ci:project-deploy-observability","Team:obs-ux-infra_services"],"title":"[Inventory]
Fix the link to discover
test","number":201197,"url":"https://github.com/elastic/kibana/pull/201197","mergeCommit":{"message":"[Inventory]
Fix the link to discover test (#201197)\n\nCloses #201189\r\n\r\n##
Summary\r\n\r\nAfter this fix was added
in\r\nhttps://github.com//pull/200984 the test started
failing\r\nas it was verifying the previous kuery value - it was missing
the `\" \"`\r\nso after this bug was fixed the test should be updated as
well\r\n(basically changing `container.id:foo` with
`container.id:\"foo\"`) and\r\nthis PR updates the test.\r\n\r\nI
checked locally and the test is passing
now.","sha":"74cf5d45784b9f633decae428c61ff96281dcc4b"}},"sourceBranch":"main","suggestedTargetBranches":[],"targetPullRequestStates":[{"branch":"main","label":"v9.0.0","branchLabelMappingKey":"^v9.0.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/201197","number":201197,"mergeCommit":{"message":"[Inventory]
Fix the link to discover test (#201197)\n\nCloses #201189\r\n\r\n##
Summary\r\n\r\nAfter this fix was added
in\r\nhttps://github.com//pull/200984 the test started
failing\r\nas it was verifying the previous kuery value - it was missing
the `\" \"`\r\nso after this bug was fixed the test should be updated as
well\r\n(basically changing `container.id:foo` with
`container.id:\"foo\"`) and\r\nthis PR updates the test.\r\n\r\nI
checked locally and the test is passing
now.","sha":"74cf5d45784b9f633decae428c61ff96281dcc4b"}}]}] BACKPORT-->

Co-authored-by: jennypavlova <dzheni.pavlova@elastic.co>
  • Loading branch information
kibanamachine and jennypavlova authored Nov 21, 2024
1 parent 21ec910 commit b342de5
Showing 1 changed file with 1 addition and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -225,7 +225,7 @@ describe('Home page', () => {
cy.getByTestSubj('inventoryEntityActionOpenInDiscover').click();
cy.url().should(
'include',
"query:'container.id:%20foo%20AND%20entity.definition_id%20:%20builtin*"
"query:'container.id:%20%22foo%22%20AND%20entity.definition_id%20:%20builtin*"
);
});
});
Expand Down

0 comments on commit b342de5

Please sign in to comment.