Skip to content
Snippets Groups Projects

Refactor views to viewsets

Merged Jan Maximilian Michal requested to merge refactor-views-to-viewsets into master

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

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
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,)
  • I dont think those 3 things have been tested yet:

    This still needs tests for SubmissionTypeApiView and for the StudentListView and for the DELETE method on Tutors (which otherwise has to be removed again).

  • 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 Michal
  • added 1 commit

    • 29f365ad - Added more tests for out endpoints that test the viewsets

    Compare with previous version

  • mentioned in commit 0bcdeaba

  • Please register or sign in to reply
    Loading