Skip to content

Increase code coverage in Class2HTMLTestCase with new test input Java4Example#186

Merged
garydgregory merged 1 commit intoapache:masterfrom
nbauma109:codecov-codehtml
Apr 7, 2023
Merged

Increase code coverage in Class2HTMLTestCase with new test input Java4Example#186
garydgregory merged 1 commit intoapache:masterfrom
nbauma109:codecov-codehtml

Conversation

@nbauma109
Copy link
Contributor

No description provided.

@codecov-commenter
Copy link

Codecov Report

Merging #186 (c099a20) into master (8d04ad5) will increase coverage by 0.78%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master     #186      +/-   ##
============================================
+ Coverage     61.65%   62.43%   +0.78%     
- Complexity     3654     3688      +34     
============================================
  Files           363      363              
  Lines         15633    15633              
  Branches       1950     1950              
============================================
+ Hits           9638     9760     +122     
+ Misses         5121     5002     -119     
+ Partials        874      871       -3     
Impacted Files Coverage Δ
src/main/java/org/apache/bcel/util/Class2HTML.java 57.44% <0.00%> (+4.25%) ⬆️
src/main/java/org/apache/bcel/util/MethodHTML.java 69.23% <0.00%> (+12.30%) ⬆️
src/main/java/org/apache/bcel/util/CodeHTML.java 80.68% <0.00%> (+30.30%) ⬆️
.../main/java/org/apache/bcel/util/AttributeHTML.java 89.77% <0.00%> (+34.09%) ⬆️

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

@nbauma109
Copy link
Contributor Author

Hi @garydgregory
Please help moving forward with this PR
Thanks !

@garydgregory garydgregory merged commit 5794d43 into apache:master Apr 7, 2023
@garydgregory garydgregory changed the title updated Class2HTMLTestCase with new test input Java4Example Increase code coverage in Class2HTMLTestCase with new test input Java4Example Apr 7, 2023
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

Comments