epugh commented on code in PR #5190:
URL: https://github.com/apache/solr/pull/5190#discussion_r4553270645
##########
solr/packaging/build.gradle:
##########
@@ -338,17 +338,20 @@ def configureBatsTask(BatsTask task, String outputName,
boolean usePicocli) {
task.environment TEST_OUTPUT_DIR: integrationTestOutput
task.environment TEST_FAILURE_DIR: solrTestFailuresDir
task.environment BATS_LIB_PREFIX: batsLibsDir
- if (usePicocli) {
- task.environment SOLR_PICOCLI: 'true'
+ if (!usePicocli) {
+ task.environment SOLR_PICOCLI: 'false'
}
}
+// TODO: Remove this when cleaning up commons-cli
task integrationTests(type: BatsTask) {
+ description = 'Runs the BATS suites for the CLI powered by commons-cli.'
configureBatsTask(it, 'test-output', false)
}
+// The same suites run against the picocli CLI.
task integrationTestsPicocli(type: BatsTask) {
- description = 'Runs the BATS suites for the migrated commands against the
experimental picocli CLI.'
+ description = 'Runs the BATS suites for the CLI powered by PicoCLI.'
Review Comment:
part of me thinks that anything mentions PicoCLI like this shouldn't be in
this PR because int he future we won't need it. But I guess we can strip out
the picocli words when we rip out commons-cli at that point. So this line
would just be `Runs the BATS suites for the CLI`!
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
