-
-
Notifications
You must be signed in to change notification settings - Fork 867
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
[WIP] Testable parts #7888
[WIP] Testable parts #7888
Conversation
- Default = False - Determines whether a particular part can have tests defined against it
✅ Deploy Preview for inventree-web-pui-preview canceled.
|
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #7888 +/- ##
==========================================
+ Coverage 83.47% 83.74% +0.26%
==========================================
Files 1128 1130 +2
Lines 50276 50298 +22
Branches 1734 1735 +1
==========================================
+ Hits 41969 42122 +153
+ Misses 7867 7716 -151
- Partials 440 460 +20
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
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; This separation makes a lot of sense
- limit_choices_to should be "testable", not "trackable" - ref: inventree#7888
* Fix restriction for PartTestTemplate - limit_choices_to should be "testable", not "trackable" - ref: #7888 * Add migration file * Fix validation check * Fix API filter * Fix for test fixture * Fix another test * Fixture data * Fix comment * More fixes * More fixes * Moar fix plz thx
testable
testable
fromtrackable
testable
is the current value oftrackable
TODO