-
Notifications
You must be signed in to change notification settings - Fork 421
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 tests #118
add tests #118
Conversation
Codecov Report
@@ Coverage Diff @@
## main #118 +/- ##
==========================================
+ Coverage 47.21% 49.14% +1.92%
==========================================
Files 67 65 -2
Lines 1974 1921 -53
==========================================
+ Hits 932 944 +12
+ Misses 1042 977 -65
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.
Amazing progress. Looking forward to reading more amazing tests! Left just minor adjustments.
Thanks for these, @htahir1. I'll read up on what |
@hamzamaiot I'm done with all the refactoring / changes as you requested. AFAICS this is good to go, pending your approval / final review. The only thing that is hanging is the comment / question / problem above. |
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.
Alright, almost there! Left some final comments, but all minor!
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.
My comment should be here.
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.
This should clear the final pieces up.
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.
Just a few last minor things. Also a general thought as we just went over it: Should these tests cover all artifact stores? Or how do we handle different file system tests? Open to suggestions
Types of changes