-
Notifications
You must be signed in to change notification settings - Fork 25.6k
Closed
Labels
:Analytics/AggregationsAggregationsAggregations>testIssues or PRs that are addressing/adding testsIssues or PRs that are addressing/adding testsTeam:AnalyticsMeta label for analytical engine team (ESQL/Aggs/Geo)Meta label for analytical engine team (ESQL/Aggs/Geo)
Description
The following aggregation tests all contain switch statements with branches that will never execute. The tests use a between(x, y) switch expression, but there are case values that are outside the randomised range. We need to fix these before #88739 can be merged. By "fix", I mean either remove the spurious cases, or if we actually want them, then extend the randomized range.
- InternalAutoDateHistogramTests
- InternalAvgTests
- InternalDerivativeTests
- InternalGeoCentroidTests
- InternalWeightedAvgTests
- SignificantLongTermsTests
- SignificantStringTermsTests
Metadata
Metadata
Assignees
Labels
:Analytics/AggregationsAggregationsAggregations>testIssues or PRs that are addressing/adding testsIssues or PRs that are addressing/adding testsTeam:AnalyticsMeta label for analytical engine team (ESQL/Aggs/Geo)Meta label for analytical engine team (ESQL/Aggs/Geo)