You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
We recently received a PR (#398) where we were setting a quiz group's course ID to self.id rather than self.course_id. This should have been caught by our tests, but wasn't since we set the id of most of our test objects to 1. It's entirely possible there are other similar issues hidden because we reuse ID numbers so frequently.
Convert our tests to use unique ID numbers between object types. (e.g. 5 for courses, 27 for quizzes, etc.)
This will likely be a fair bit of work, since we reuse the fixtures in many places.
The text was updated successfully, but these errors were encountered:
We recently received a PR (#398) where we were setting a quiz group's course ID to
self.id
rather thanself.course_id
. This should have been caught by our tests, but wasn't since we set theid
of most of our test objects to1
. It's entirely possible there are other similar issues hidden because we reuse ID numbers so frequently.Convert our tests to use unique ID numbers between object types. (e.g. 5 for courses, 27 for quizzes, etc.)
This will likely be a fair bit of work, since we reuse the fixtures in many places.
The text was updated successfully, but these errors were encountered: