-
Notifications
You must be signed in to change notification settings - Fork 133
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
Refactor Datumaro format code and test code #824
Refactor Datumaro format code and test code #824
Conversation
Signed-off-by: Kim, Vinnam <vinnam.kim@intel.com>
Signed-off-by: Kim, Vinnam <vinnam.kim@intel.com>
Signed-off-by: Kim, Vinnam <vinnam.kim@intel.com>
Signed-off-by: Kim, Vinnam <vinnam.kim@intel.com>
Signed-off-by: Kim, Vinnam <vinnam.kim@intel.com>
Signed-off-by: Kim, Vinnam <vinnam.kim@intel.com>
Codecov ReportBase: 78.27% // Head: 78.33% // Increases project coverage by
Additional details and impacted files@@ Coverage Diff @@
## develop #824 +/- ##
===========================================
+ Coverage 78.27% 78.33% +0.06%
===========================================
Files 184 189 +5
Lines 23614 23681 +67
Branches 4892 4895 +3
===========================================
+ Hits 18484 18551 +67
Misses 4032 4032
Partials 1098 1098
Flags with carried forward coverage won't be shown. Click here to find out more.
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. ☔ View full report at Codecov. |
Summary
unittest
topytest
in Datumaro unit test code. Please see the reason why we move on as follows.Currently, Datumaro uses old-fashioned unittest framework and its unit test codes has been time-worn so much. There have been super many duplicated and never re-use codes in our tests. In addition, if we keep using unittest, it is expected that the spaghetti with no taste will continue to grow its size bigger and bigger. This epic aims to migrate our testing framework from unittest to pytest. Appropriate code refactoring and use of pytest fixtures can slim down our testing code.
How to test
This PR's test refactoring will cover.
Checklist
develop
branchLicense
Feel free to contact the maintainers if that's a concern.