-
Notifications
You must be signed in to change notification settings - Fork 3
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
Rating implementation #55
Conversation
A fixed field called rating is implemented, accepting values from 1 to 10, or None if not set.
@pboettch it looks ok to me is this ready for merge? |
No, let me finish the work on tscat_gui with this change, to see whether it fits all the needs. In the best case, no more changes necessary here. |
tests/test_event.py
Outdated
@@ -102,3 +102,30 @@ def test_constructor_with_dynamic_attribute_manual_access(self): | |||
self.assertEqual(e.field_str, "string-test") | |||
self.assertEqual(e.field_bool, True) | |||
self.assertEqual(e.field_dt, dt_val) | |||
|
|||
def test_event_rating(self): |
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.
Should be split in two data driven tests.
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #55 +/- ##
=========================================
Coverage 100.00% 100.00%
=========================================
Files 10 10
Lines 1674 1770 +96
=========================================
+ Hits 1674 1770 +96
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
e766dad
to
66e9ba7
Compare
A fixed field called rating is implemented, accepting values from 1 to 10, or None if not set.