Refactor views to viewsets
This still needs tests for SubmissionTypeApiView and for the StudentListView and for the DELETE method on Tutors (which otherwise has to be removed again).
Doing this is actually super helpful in providing a fairly simple interface for object creation.
Edited by Jan Maximilian Michal
Merge request reports
Activity
Filter activity
added 1 commit
- 7cc854f0 - Fixed viewset permissions and added logging to permission base
I believe all tests that were required are written? @robinwilliam.hundt
28 24 29 25 class ExamApiViewSet(viewsets.ReadOnlyModelViewSet): 30 26 """ Gets a list of an individual exam by Id if provided """ 31 permissions_classes = (IsReviewer,) Was this the typo?
I looked it over at least 5 times but didn't see it Edited by robinwilliam.hundt
Hm I shouldn’t look at this while using the phone.
Ok than those tests will follow.
Edited by Jan Maximilian Michal- SubmissionTypeApiView
- StudentListView
- DELETE method on Tutors (which otherwise has to be removed again)
Edited by Jan Maximilian Michaladded 1 commit
- 29f365ad - Added more tests for out endpoints that test the viewsets
mentioned in commit 0bcdeaba
Please register or sign in to reply