Skip to content

Conversation

@lukecwik
Copy link
Member

Be sure to do all of the following to help us incorporate your contribution
quickly and easily:

  • Make sure the PR title is formatted like:
    [BEAM-<Jira issue #>] Description of pull request
  • Make sure tests pass via mvn clean verify. (Even better, enable
    Travis-CI on your fork and ensure the whole test matrix passes).
  • Replace "<Jira issue #>" in the title with the actual Jira issue
    number, if there is one.
  • If this contribution is large, please file an Apache
    Individual Contributor License Agreement.

Added surefire.dependencyToScan and surefire.testPipelineOptions configuration options

surefire.dependencyToScan adds an additional dependency used to find tests.
This allows for java-sdk-all to be added to all runners for running
the @category(RunnableOnService.class) tests.

surefire.testPipelineOptions configures the pipeline options used while
constructing instances of a TestPipeline. A json list of arguments
is expected, for example:
[ "--runner=MyRunner", "--myRunnerOption=myRunnerValue" ]

…java

Added surefire.dependencyToScan and surefire.testPipelineOptions configuration options

surefire.dependencyToScan adds an additional dependency used to find tests.
This allows for java-sdk-all to be added to all runners for running
the @category(RunnableOnService.class) tests.

surefire.testPipelineOptions configures the pipeline options used while
constructing instances of a TestPipeline. A json list of arguments
is expected, for example:
[ "--runner=MyRunner", "--myRunnerOption=myRunnerValue" ]
@lukecwik
Copy link
Member Author

R: @davorbonaci

Please review and after merge update Jenkins to use:
mvn -B -e -pl runners/google-cloud-dataflow-java -am -P DataflowPipelineTests clean verify -Ddataflow.testPipelineOptions=...

Where -Ddataflow.testPipelineOptions replaces -DdataflowOptions

Currently this change will run all the java-sdk-all tests while building that module and then the @category(RunnableOnService.class) tests found in both java-sdk-all and google-cloud-dataflow-java using the Dataflow runner.

@lukecwik
Copy link
Member Author

Will revisit how this is being done and potentially come up with new proposal.

@lukecwik lukecwik closed this Apr 12, 2016
swegner pushed a commit to swegner/beam that referenced this pull request Apr 22, 2016
iemejia pushed a commit to iemejia/beam that referenced this pull request Jan 12, 2018
damccorm pushed a commit that referenced this pull request Dec 9, 2022
…#23140)

* Updated go_tests Workflow (#170)

Co-authored-by: Elias Segundo <elias.segundo@luisrazo.local>

* Added master changes in go_tests to avoid merge conflicts

* Re adding the fetch-depth: 2 in checkout step in Go Test workflow

* Switching trigger to pull_request (#262)

* Switching trigger to pull_request

* Removing ref from checkout

Co-authored-by: Elias Segundo Antonio <eliassegundo.segundo@gmail.com>
Co-authored-by: Elias Segundo <elias.segundo@luisrazo.local>
Co-authored-by: elink22 <103056145+elink22@users.noreply.github.com>
lostluck pushed a commit to lostluck/beam that referenced this pull request Dec 22, 2022
…apache#23140)

* Updated go_tests Workflow (apache#170)

Co-authored-by: Elias Segundo <elias.segundo@luisrazo.local>

* Added master changes in go_tests to avoid merge conflicts

* Re adding the fetch-depth: 2 in checkout step in Go Test workflow

* Switching trigger to pull_request (apache#262)

* Switching trigger to pull_request

* Removing ref from checkout

Co-authored-by: Elias Segundo Antonio <eliassegundo.segundo@gmail.com>
Co-authored-by: Elias Segundo <elias.segundo@luisrazo.local>
Co-authored-by: elink22 <103056145+elink22@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant