Skip to content

JUnit5 assertThrows OptionTest#140

Merged
garydgregory merged 2 commits into
apache:masterfrom
nhojpatrick:junit5-assertThrows-OptionTest
Nov 11, 2022
Merged

JUnit5 assertThrows OptionTest#140
garydgregory merged 2 commits into
apache:masterfrom
nhojpatrick:junit5-assertThrows-OptionTest

Conversation

@nhojpatrick

Copy link
Copy Markdown
Contributor

No description provided.

@nhojpatrick nhojpatrick force-pushed the junit5-assertThrows-OptionTest branch from 035d47c to f8d89e1 Compare October 21, 2022 19:35
@garydgregory

Copy link
Copy Markdown
Member

Hi @nhojpatrick
Please rebase on master and implement your FIXME items.

@garydgregory

Copy link
Copy Markdown
Member

FYI what really needs help with JUnit 5 is Commons VFS... ;-)

@codecov-commenter

codecov-commenter commented Oct 22, 2022

Copy link
Copy Markdown

Codecov Report

Merging #140 (0e37434) into master (c58d23d) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master     #140   +/-   ##
=========================================
  Coverage     93.20%   93.20%           
  Complexity      567      567           
=========================================
  Files            21       21           
  Lines          1206     1206           
  Branches        214      214           
=========================================
  Hits           1124     1124           
  Misses           46       46           
  Partials         36       36           

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

@nhojpatrick nhojpatrick force-pushed the junit5-assertThrows-OptionTest branch from f8d89e1 to 0e37434 Compare November 11, 2022 18:23
@nhojpatrick

Copy link
Copy Markdown
Contributor Author

@garydgregory rebased and updated for Java 8.

@garydgregory garydgregory merged commit 6932cec into apache:master Nov 11, 2022
@nhojpatrick nhojpatrick deleted the junit5-assertThrows-OptionTest branch November 11, 2022 22:30
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