Skip to content

Comments

[8.6] quick check of current URL to skip timepicker (#146462)#146954

Merged
kibanamachine merged 1 commit intoelastic:8.6from
kibanamachine:backport/8.6/pr-146462
Dec 4, 2022
Merged

[8.6] quick check of current URL to skip timepicker (#146462)#146954
kibanamachine merged 1 commit intoelastic:8.6from
kibanamachine:backport/8.6/pr-146462

Conversation

@kibanamachine
Copy link
Contributor

Backport

This will backport the following commits from main to 8.6:

Questions ?

Please refer to the Backport tool documentation

## Summary

We have some tests which set the default time but then also call
setAbsoluteTime in every test. This PR adds code in setAbsoluteTime to
quickly check if the desired start and end (from and to) times are
already set, and if so, just return. The end result should be less time
spent on functional tests. It could also reduce occasional flakiness
setting the time.

Co-authored-by: kibanamachine <42973632+kibanamachine@users.noreply.github.com>
Co-authored-by: Dzmitry Lemechko <dzmitry.lemechko@elastic.co>
(cherry picked from commit 1734b0e)
@kibanamachine kibanamachine added the backport This PR is a backport of another PR label Dec 4, 2022
@kibanamachine kibanamachine enabled auto-merge (squash) December 4, 2022 20:55
@kibanamachine kibanamachine merged commit ff7d73d into elastic:8.6 Dec 4, 2022
@kibana-ci
Copy link

💚 Build Succeeded

Metrics [docs]

Unknown metric groups

ESLint disabled in files

id before after diff
osquery 1 2 +1

ESLint disabled line counts

id before after diff
enterpriseSearch 19 21 +2
fleet 59 65 +6
osquery 108 113 +5
securitySolution 441 447 +6
total +19

Total ESLint disabled count

id before after diff
enterpriseSearch 20 22 +2
fleet 68 74 +6
osquery 109 115 +6
securitySolution 518 524 +6
total +20

To update your PR or re-run it, just comment with:
@elasticmachine merge upstream

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport This PR is a backport of another PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants