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

Fix incomplete AssertJ API usage #531

Merged
merged 1 commit into from
Apr 27, 2021
Merged

Conversation

jponge
Copy link
Member

@jponge jponge commented Apr 27, 2021

No description provided.

@jponge jponge added this to the 0.16.0 milestone Apr 27, 2021
@jponge jponge added the bug Something isn't working label Apr 27, 2021
@jponge jponge self-assigned this Apr 27, 2021
@jponge jponge requested a review from cescoffier April 27, 2021 13:39
@codecov
Copy link

codecov bot commented Apr 27, 2021

Codecov Report

Merging #531 (eb55d26) into main (425d20d) will increase coverage by 0.34%.
The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff              @@
##               main     #531      +/-   ##
============================================
+ Coverage     89.88%   90.23%   +0.34%     
- Complexity     3006     3016      +10     
============================================
  Files           382      382              
  Lines         11621    11621              
  Branches       1453     1455       +2     
============================================
+ Hits          10446    10486      +40     
+ Misses          597      578      -19     
+ Partials        578      557      -21     
Impacted Files Coverage Δ Complexity Δ
...perators/multi/processors/SerializedProcessor.java 92.07% <0.00%> (-1.00%) 26.00% <0.00%> (ø%)
...mallrye/mutiny/operators/multi/MultiFlatMapOp.java 85.04% <0.00%> (+0.31%) 3.00% <0.00%> (ø%)
...smallrye/mutiny/operators/multi/MultiEmitOnOp.java 90.47% <0.00%> (+0.95%) 2.00% <0.00%> (ø%)
...java/io/smallrye/mutiny/helpers/Subscriptions.java 81.76% <0.00%> (+1.10%) 50.00% <0.00%> (+1.00%)
...erators/multi/builders/SerializedMultiEmitter.java 83.54% <0.00%> (+1.26%) 27.00% <0.00%> (+1.00%)
...tiny/operators/multi/MultiBufferWithTimeoutOp.java 74.57% <0.00%> (+1.69%) 4.00% <0.00%> (ø%)
...subscription/SwitchableSubscriptionSubscriber.java 97.54% <0.00%> (+2.45%) 46.00% <0.00%> (+1.00%)
...io/smallrye/mutiny/operators/uni/UniMemoizeOp.java 95.04% <0.00%> (+3.96%) 28.00% <0.00%> (+2.00%)
.../io/smallrye/mutiny/helpers/queues/DrainUtils.java 80.00% <0.00%> (+4.44%) 12.00% <0.00%> (+1.00%)
...y/operators/multi/builders/IterableBasedMulti.java 88.29% <0.00%> (+6.38%) 4.00% <0.00%> (ø%)
... and 2 more

@jponge jponge force-pushed the fix/incomplete-assertj-usage branch from 3e5eaf4 to eb55d26 Compare April 27, 2021 14:13
@cescoffier cescoffier merged commit f3d1abd into main Apr 27, 2021
@cescoffier cescoffier deleted the fix/incomplete-assertj-usage branch April 27, 2021 15:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants