Skip to content
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

Add Azure test slicing for parallel test running. #2231

Open
wants to merge 61 commits into
base: master
Choose a base branch
from

Conversation

tadgh
Copy link
Collaborator

@tadgh tadgh commented Dec 10, 2020

Playing with Azure pipelines to see if we can parallelize the test process across modules.

@jamesagnew
Copy link
Collaborator

Whaaaaaaaaat? This is so cool

@hapifhir hapifhir deleted a comment from codecov bot Jan 5, 2021
inputs:
goals: 'clean install'
# These are Maven CLI options (and show up in the build logs) - "-nsu"=Don't update snapshots. We can remove this when Maven OSS is more healthy
options: "-P ALLMODULES,JACOCO,ERRORPRONE,CI -e -B -Dmaven.repo.local=$(MAVEN_CACHE_FOLDER) -Dmaven.test.skip"
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Minor nitpick: If you change "maven.test.skip" to "skipTests", the test code will be compiled but not executed. This might be useful so we can see if there are any compilation problems early, but without taking the hit of actually running the tests.

Copy link
Collaborator

@KevinDougan-SmileCDR KevinDougan-SmileCDR left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approve pending review of minor changes.

@codecov
Copy link

codecov bot commented Jan 5, 2021

Codecov Report

Merging #2231 (d195636) into master (368e9a9) will decrease coverage by 42.62%.
The diff coverage is n/a.

Impacted file tree graph

@@              Coverage Diff              @@
##             master    #2231       +/-   ##
=============================================
- Coverage     81.43%   38.81%   -42.63%     
+ Complexity    17263     1428    -15835     
=============================================
  Files          1181      211      -970     
  Lines         63218    10309    -52909     
  Branches       9837     1779     -8058     
=============================================
- Hits          51482     4001    -47481     
+ Misses         7890     5655     -2235     
+ Partials       3846      653     -3193     
Impacted Files Coverage Δ Complexity Δ
...pi/src/main/java/ca/uhn/fhir/jpa/api/dao/IDao.java 0.00% <0.00%> (-100.00%) 0.00% <0.00%> (-1.00%)
...va/ca/uhn/fhir/rest/server/method/AtParameter.java 0.00% <0.00%> (-100.00%) 0.00% <0.00%> (-1.00%)
...ca/uhn/fhir/rest/server/method/SinceParameter.java 0.00% <0.00%> (-100.00%) 0.00% <0.00%> (-1.00%)
...java/ca/uhn/fhir/jpa/api/model/DeleteConflict.java 0.00% <0.00%> (-100.00%) 0.00% <0.00%> (-5.00%)
...java/ca/uhn/fhir/jpa/api/model/ExpungeOptions.java 0.00% <0.00%> (-100.00%) 0.00% <0.00%> (-10.00%)
...java/ca/uhn/fhir/jpa/api/model/ExpungeOutcome.java 0.00% <0.00%> (-100.00%) 0.00% <0.00%> (-3.00%)
...va/ca/uhn/fhir/jpa/api/model/DaoMethodOutcome.java 0.00% <0.00%> (-100.00%) 0.00% <0.00%> (-8.00%)
...hn/fhir/rest/server/method/PatchTypeParameter.java 0.00% <0.00%> (-100.00%) 0.00% <0.00%> (-5.00%)
...hn/fhir/rest/server/method/SinceOrAtParameter.java 0.00% <0.00%> (-100.00%) 0.00% <0.00%> (-3.00%)
...n/fhir/rest/server/provider/BaseLastNProvider.java 0.00% <0.00%> (-100.00%) 0.00% <0.00%> (-2.00%)
... and 1119 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 368e9a9...6bcaea7. Read the comment docs.

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.

None yet

3 participants