-
Notifications
You must be signed in to change notification settings - Fork 0
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
Implemented RpmMetadata.Append
#405
Implemented RpmMetadata.Append
#405
Conversation
Codecov Report
@@ Coverage Diff @@
## master #405 +/- ##
============================================
+ Coverage 85.90% 86.00% +0.10%
Complexity 303 303
============================================
Files 55 55
Lines 1752 1772 +20
Branches 99 103 +4
============================================
+ Hits 1505 1524 +19
+ Misses 232 230 -2
- Partials 15 18 +3
Continue to review full report at Codecov.
|
|
||
@Test | ||
void appendsRecords() throws IOException { |
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.
@olenagerasimova thanks. Is it possible to add packages when primary.xml.example
and other.xml.example
are empty or absent? If it is so, I'd suggest to add such test case
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.
@genryxy now - it's not possible, there will be an invalid xml exception or smth like that. But that's a good question, I will discuss such scenario with our colleagues and make the changes accordingly.
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.
@olenagerasimova got it
@rultor release, tag= |
@olenagerasimova OK, I will release it now. Please check the progress here |
@olenagerasimova Oops, I failed. You can see the full log here (spent 9min)
|
@rultor release, tag= |
@olenagerasimova OK, I will release it now. Please check the progress here |
@olenagerasimova Oops, I failed. You can see the full log here (spent 10min)
|
@rultor release, tag= |
@olenagerasimova OK, I will release it now. Please check the progress here |
@olenagerasimova Done! FYI, the full log is here (took me 10min) |
Part of #388
Implemented
RpmMetadata.Append
and added test.