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

Parallel tests execution #770

Merged
merged 2 commits into from
Dec 7, 2021
Merged

Parallel tests execution #770

merged 2 commits into from
Dec 7, 2021

Conversation

jponge
Copy link
Member

@jponge jponge commented Dec 1, 2021

Depends on #769, do not merge yet

Parallel builds (same JVM for the whole suite) work fine on my side but remain flaky in CI / constrained environments. Hence they must be activated with a Maven profile (-Pparallel-tests)

@codecov
Copy link

codecov bot commented Dec 1, 2021

Codecov Report

Merging #770 (c30df63) into main (a62cb00) will increase coverage by 0.11%.
The diff coverage is 100.00%.

❗ Current head c30df63 differs from pull request most recent head 54bd07c. Consider uploading reports for the commit 54bd07c to get more accurate results
Impacted file tree graph

@@             Coverage Diff              @@
##               main     #770      +/-   ##
============================================
+ Coverage     90.08%   90.20%   +0.11%     
- Complexity     3010     3023      +13     
============================================
  Files           377      377              
  Lines         11883    11885       +2     
  Branches       1497     1500       +3     
============================================
+ Hits          10705    10721      +16     
+ Misses          602      596       -6     
+ Partials        576      568       -8     
Impacted Files Coverage Δ
...ava/io/smallrye/mutiny/groups/MultiTimePeriod.java 100.00% <100.00%> (ø)
...utiny/operators/multi/MultiWindowOnDurationOp.java 65.71% <0.00%> (-4.29%) ⬇️
...tiny/operators/multi/MultiBufferWithTimeoutOp.java 74.78% <0.00%> (-0.85%) ⬇️
...smallrye/mutiny/helpers/test/AssertSubscriber.java 89.47% <0.00%> (-0.76%) ⬇️
...mallrye/mutiny/operators/multi/MultiFlatMapOp.java 86.76% <0.00%> (+0.92%) ⬆️
...allrye/mutiny/operators/uni/UniAndCombination.java 89.53% <0.00%> (+1.16%) ⬆️
...erators/multi/builders/BufferItemMultiEmitter.java 92.10% <0.00%> (+1.31%) ⬆️
...subscription/SwitchableSubscriptionSubscriber.java 99.18% <0.00%> (+1.63%) ⬆️
...java/io/smallrye/mutiny/helpers/Subscriptions.java 81.21% <0.00%> (+1.65%) ⬆️
...erators/multi/builders/SerializedMultiEmitter.java 83.54% <0.00%> (+2.53%) ⬆️
... and 2 more

@jponge jponge added the ci/cd/build CI/CD pipeline and build label Dec 2, 2021
@jponge jponge marked this pull request as ready for review December 7, 2021 08:59
@jponge jponge added this to the 1.3.0 milestone Dec 7, 2021
@jponge jponge changed the title Parallel builds Parallel tests execution Dec 7, 2021
@jponge jponge merged commit cb6b843 into main Dec 7, 2021
@jponge jponge deleted the internal/parallel-build branch December 7, 2021 15:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ci/cd/build CI/CD pipeline and build
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants