Skip to content

Conversation

@andyb-elastic
Copy link
Contributor

Adds tests for unmapped fields, supported values source types, and
scripting. Moves some tests over from the integration test

In AggregatorTestCase, skips asserting on the bucket consumer when
collection did not occur, e.g. because of an unmapped field

Adds tests for unmapped fields, supported values source types, and
scripting. Moves some tests over from the integration test

In AggregatorTestCase, skips asserting on the bucket consumer when
collection did not occur, e.g. because of an unmapped field
@andyb-elastic andyb-elastic added >test Issues or PRs that are addressing/adding tests :Analytics/Aggregations Aggregations v8.0.0 labels Apr 21, 2020
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-analytics-geo (:Analytics/Aggregations)

@rjernst rjernst added the Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) label May 4, 2020
@arteam arteam added v8.1.0 and removed v8.0.0 labels Jan 12, 2022
@mark-vieira mark-vieira added v8.2.0 and removed v8.1.0 labels Feb 2, 2022
@elasticsearchmachine elasticsearchmachine changed the base branch from master to main July 22, 2022 23:13
@mark-vieira mark-vieira added v8.5.0 and removed v8.4.0 labels Jul 27, 2022
@elasticsearchmachine
Copy link
Collaborator

Pinging @elastic/es-analytics-geo (Team:Analytics)

nik9000 added a commit to nik9000/elasticsearch that referenced this pull request Jul 29, 2022
This ports some of the tests for `date_range` from a high overhead
`IntegTestCase` to a lower overhead `AggregatorTestCase`.
`AggregatorTestCase` is also lower level so it can find more fun things
like memory leaks. And it enables values source type testing for
`date_range`.

Inspired by elastic#55502 by [email protected]
@nik9000
Copy link
Member

nik9000 commented Jul 29, 2022

Replaced by #88958.

@nik9000 nik9000 closed this Jul 29, 2022
@nik9000 nik9000 removed the v8.5.0 label Jul 29, 2022
elasticsearchmachine pushed a commit that referenced this pull request Aug 1, 2022
This ports some of the tests for `date_range` from a high overhead
`IntegTestCase` to a lower overhead `AggregatorTestCase`.
`AggregatorTestCase` is also lower level so it can find more fun things
like memory leaks. And it enables values source type testing for
`date_range`.

Inspired by #55502 by [email protected]
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

:Analytics/Aggregations Aggregations Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) >test Issues or PRs that are addressing/adding tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

9 participants