-
Notifications
You must be signed in to change notification settings - Fork 4
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
Fix empty reservation #12
Conversation
Signed-off-by: Sebastian Becker <[email protected]>
Signed-off-by: Sebastian Becker <[email protected]>
Signed-off-by: Sebastian Becker <[email protected]>
Codecov Report
@@ Coverage Diff @@
## master #12 +/- ##
============================================
+ Coverage 85.45% 85.47% +0.02%
- Complexity 335 336 +1
============================================
Files 61 61
Lines 1210 1212 +2
Branches 79 80 +1
============================================
+ Hits 1034 1036 +2
Misses 142 142
Partials 34 34
Continue to review full report at Codecov.
|
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.
LGTM
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.
Fix unused variable issue.
...ice/src/test/java/io/carbynestack/castor/service/download/CreateReservationSupplierTest.java
Outdated
Show resolved
Hide resolved
Signed-off-by: Sebastian Becker <[email protected]>
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.
LGTM
Resolves #11
Lazy loading is not addressed as implementation will be heavily adapted when #10 is addressed