New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[CCR] Add qa test library #34611
[CCR] Add qa test library #34611
Conversation
test code duplication.
Pinging @elastic/es-distributed |
@@ -5,6 +5,7 @@ apply plugin: 'elasticsearch.standalone-test' | |||
dependencies { | |||
testCompile project(path: xpackModule('core'), configuration: 'testArtifacts') | |||
testCompile project(path: xpackModule('ccr'), configuration: 'runtime') | |||
testCompile project(':x-pack:plugin:ccr:qa:test-common') |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
SQL has this and it sticks it in qa
.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Well, it is in x-pack/qa/sql
now, but I'm in the process of moving it to x-pack/plugin/sql/qa
.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
That is a much better idea than adding another gradle module. So I did this: 633afa7
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
and here is the second half of the change: 772a4f8
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, left one comment, no need for another round.
x-pack/plugin/ccr/qa/build.gradle
Outdated
@@ -1,5 +1,14 @@ | |||
import org.elasticsearch.gradle.test.RestIntegTestTask | |||
|
|||
apply plugin: 'elasticsearch.build' | |||
|
|||
archivesBaseName = 'ccr-qa-test-common' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Let us remove this.
* Introduced test qa lib that all CCR qa modules depend on to avoid test code duplication.
* Introduced test qa lib that all CCR qa modules depend on to avoid test code duplication.
Introduced test qa lib that all CCR qa modules depend on in order to avoid test code duplication.
All qa multi node tests now extend from a new base class that adds several helper methods.