elastic/elasticsearch

[Test] Convert Pipeline agg integration tests into AggregatorTestCase

Open

#36,015 opened on Nov 28, 2018

View on GitHub
 (6 comments) (0 reactions) (0 assignees)Java (25,882 forks)batch import
:Analytics/Aggregations>non-issue>testTeam:Analyticshelp wanted

Repository metrics

Stars
 (76,700 stars)
PR merge metrics
 (Avg merge 2d) (1,000 merged PRs in 30d)

Description

Pipeline aggs were added before the AggregatorTestCase was created, so most of the pipelines are tested with a few unit tests and a lot of integration tests. We should be able to convert these over to AggregatorTestCase, which should be faster and generally easier to test against.

Contributor guide

Research direction
Look at existing AggregatorTestCase examples in the repository to understand the pattern, then identify pipeline aggregation tests that are integration tests and convert them to use AggregatorTestCase. Focus on one pipeline aggregation at a time.
Tech stack
java
Domain
backendtesting qa
Issue type
Test
DifficultyEstimated implementation difficulty for a new contributor, from 1 for very small changes to 5 for expert-level work.
3
Estimated timeA rough time range for an experienced contributor to investigate, implement, test, and prepare a pull request.
1-2 days
Activity statusHow available the issue appears right now: fresh, active, stale, blocked, or waiting on maintainer input.
Stale
ClarityHow clearly the issue explains the expected change, acceptance criteria, and next step.
Clear
Prerequisites
Java
Newbie friendlinessA 1-100 score estimating how approachable this issue is for first-time contributors.
75