-
Notifications
You must be signed in to change notification settings - Fork 6
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
Add Unit Test #20
Add Unit Test #20
Conversation
Hi @egonw - I have added unit test case for below mentioned file.
Kindly review and provide feedback if any. |
Codecov ReportAll modified lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #20 +/- ##
============================================
+ Coverage 32.47% 33.23% +0.76%
- Complexity 3369 3456 +87
============================================
Files 280 280
Lines 31409 31409
Branches 6299 6299
============================================
+ Hits 10199 10439 +240
+ Misses 19513 19247 -266
- Partials 1697 1723 +26 ☔ View full report in Codecov by Sentry. |
src/test/java/org/xmlcml/cml/element/main/CMLDictionaryTest.java
Outdated
Show resolved
Hide resolved
src/test/java/org/xmlcml/cml/element/main/CMLIdentifierTest.java
Outdated
Show resolved
Hide resolved
Hi @egonw, Added the requested changes. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you for the updates
Merged into the 4.7 dev branch: #22 |
No description provided.