Support multiple paths for test artifacts


#1

Right now in order to save all of our test reports, we need multiple entries:

      - type: test-results-store
        path: ~/maven-reactor/target/dependency-check-report.html

      - type: test-results-store
        path: ~/maven-reactor/sub-module1/target/failsafe-reports/

      - type: test-results-store
        path: ~/maven-reactor/sub-module1/target/surefire-reports/

      - type: test-results-store
        path: ~/maven-reactor/sub-module1/target/findbugs/

      - type: test-results-store
        path: ~/maven-reactor/sub-module2/target/failsafe-reports/

      - type: test-results-store
        path: ~/maven-reactor/sub-module2/target/surefire-reports/

      - type: test-results-store
        path: ~/maven-reactor/sub-module2/target/findbugs/

It would be much cleaner if we could just do something like:

      - type: test-results-store
        paths:
          - ~/maven-reactor/target/dependency-check-report.html
          - ~/maven-reactor/sub-module1/target/failsafe-reports/
          - etc.

#2

Similar with uploading artifacts, too. Great feedback - thank you! That would be a lot simpler. I opened a ticket internally for this.


#4

Also, supporting files (wildcards?) instead of just paths would be nice. Right now we’ve got a file ~/maven-reactor/target/dependency-check-report.html that should be part of our test artifacts, but we don’t want to add the entire target directory.


#5

@rohara any update on this issue? I am still getting error when I specify multiple paths under store_artifacts. Below code doesn’t work:

store_artifacts:
    paths:
      - tmp/capybara
      - test_results

#6

This was for store_test_results and I see the PR was merged so it should work, but not for store_artifacts (yet?).


#7