Skip to content

JUnit5 assertThrows DataSourceFileResolverTest#108

Merged
garydgregory merged 2 commits intoapache:masterfrom
nhojpatrick:junit5-assertThrows-DataSourceFileResolverTest
Nov 27, 2022
Merged

JUnit5 assertThrows DataSourceFileResolverTest#108
garydgregory merged 2 commits intoapache:masterfrom
nhojpatrick:junit5-assertThrows-DataSourceFileResolverTest

Conversation

@nhojpatrick
Copy link
Contributor

No description provided.

@nhojpatrick nhojpatrick force-pushed the junit5-assertThrows-DataSourceFileResolverTest branch from 63cc108 to 3dd6f2c Compare November 11, 2022 22:36
@codecov-commenter
Copy link

codecov-commenter commented Nov 12, 2022

Codecov Report

Merging #108 (e74e428) into master (ef02398) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master     #108   +/-   ##
=========================================
  Coverage     65.62%   65.62%           
  Complexity      305      305           
=========================================
  Files            18       18           
  Lines          1053     1053           
  Branches        137      137           
=========================================
  Hits            691      691           
  Misses          283      283           
  Partials         79       79           

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@nhojpatrick nhojpatrick force-pushed the junit5-assertThrows-DataSourceFileResolverTest branch from 3dd6f2c to c808ab5 Compare November 21, 2022 22:00
Copy link
Member

@garydgregory garydgregory left a comment

Choose a reason for hiding this comment

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

Please inline the local variables that hold lambdas.

@nhojpatrick nhojpatrick force-pushed the junit5-assertThrows-DataSourceFileResolverTest branch from c808ab5 to e74e428 Compare November 26, 2022 20:03
@garydgregory garydgregory merged commit d9f9ce8 into apache:master Nov 27, 2022
garydgregory added a commit that referenced this pull request Nov 27, 2022
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.

3 participants