Skip to content
test_generic.py 18.7 KiB
Newer Older
Dong Zhuang's avatar
Dong Zhuang committed
__copyright__ = "Copyright (C) 2017 Zesheng Wang, Andreas Kloeckner, Zhuang Dong"

__license__ = """
Permission is hereby granted, free of charge, to any person obtaining a copy
of this software and associated documentation files (the "Software"), to deal
in the Software without restriction, including without limitation the rights
to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
copies of the Software, and to permit persons to whom the Software is
furnished to do so, subject to the following conditions:

The above copyright notice and this permission notice shall be included in
all copies or substantial portions of the Software.

THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN
THE SOFTWARE.
"""

Josh Asplund's avatar
Josh Asplund committed
import pytest
Andreas Klöckner's avatar
Andreas Klöckner committed
from django.test import Client, TestCase
from django.urls import NoReverseMatch, reverse
zwang180's avatar
zwang180 committed

Dong Zhuang's avatar
Dong Zhuang committed
from course.models import (
    FlowRuleException,
    FlowSession,
    GradingOpportunity,
    Participation,
Dong Zhuang's avatar
Dong Zhuang committed
)
from tests.base_test_mixins import SingleCoursePageTestMixin, classmethod_with_client
Dong Zhuang's avatar
Dong Zhuang committed

Dong Zhuang's avatar
Dong Zhuang committed
class GradeGenericTestMixin(SingleCoursePageTestMixin):
    def setUpTestData(cls):
        super().setUpTestData()
        client = Client()
        cls.flow_session_ids = []
        cls.do_quiz(client, cls.student_participation)

    # Use specified user to take a quiz
    @classmethod_with_client
    def do_quiz(cls, client, participation):  # noqa: N805
        client.force_login(participation.user)
        cls.start_flow(client, cls.flow_id)
        cls.end_flow(client)
        cls.flow_session_ids.append(
            int(cls.default_flow_params["flow_session_id"]))
    # Separate the test here
    def test_grading_opportunity(self):
        # Should only have one grading opportunity object
Dong Zhuang's avatar
Dong Zhuang committed
        self.assertEqual(GradingOpportunity.objects.all().count(), 1)

    def test_view_my_grade(self):
        resp = self.client.get(reverse("relate-view_participant_grades",
Dong Zhuang's avatar
Dong Zhuang committed
                                  args=[self.course.identifier]))
        self.assertEqual(resp.status_code, 200)

    def test_view_participant_grades(self):
Dong Zhuang's avatar
Dong Zhuang committed
        params = {"course_identifier": self.course.identifier,
                  "participation_id": self.instructor_participation.user.id}
        resp = self.client.get(reverse("relate-view_participant_grades",
                                                    kwargs=params))
        self.assertEqual(resp.status_code, 200)

    def test_view_participant_list(self):
        resp = self.client.get(reverse("relate-view_participant_list",
Dong Zhuang's avatar
Dong Zhuang committed
                                  args=[self.course.identifier]))
        self.assertEqual(resp.status_code, 200)

    def test_view_grading_opportunity_list(self):
        resp = self.client.get(reverse("relate-view_grading_opportunity_list",
Dong Zhuang's avatar
Dong Zhuang committed
                                  args=[self.course.identifier]))
        self.assertEqual(resp.status_code, 200)

    def test_view_gradebook(self):
        resp = self.client.get(reverse("relate-view_gradebook",
Dong Zhuang's avatar
Dong Zhuang committed
                                  args=[self.course.identifier]))
        self.assertEqual(resp.status_code, 200)

    def test_view_grades_by_opportunity(self):
        # Check attributes
Dong Zhuang's avatar
Dong Zhuang committed
        self.assertEqual(GradingOpportunity.objects.all().count(), 1)
        opportunity = GradingOpportunity.objects.first()
        self.assertEqual(self.course, opportunity.course)
        self.assertEqual(self.flow_id, opportunity.flow_id)
Dong Zhuang's avatar
Dong Zhuang committed
        params = {"course_identifier": self.course.identifier,
                  "opp_id": opportunity.id}
        resp = self.client.get(reverse("relate-view_grades_by_opportunity",
Dong Zhuang's avatar
Dong Zhuang committed
                                  kwargs=params))
        self.assertEqual(resp.status_code, 200)

    def test_view_participant_grade_by_opportunity(self):
        # Check attributes
Dong Zhuang's avatar
Dong Zhuang committed
        self.assertEqual(GradingOpportunity.objects.all().count(), 1)
        opportunity = GradingOpportunity.objects.first()
        self.assertEqual(self.course, opportunity.course)
        self.assertEqual(self.flow_id, opportunity.flow_id)
Dong Zhuang's avatar
Dong Zhuang committed
        params = {"course_identifier": self.course.identifier,
                  "opportunity_id": opportunity.id,
                  "participation_id": self.student_participation.id}
        resp = self.client.get(reverse("relate-view_single_grade", kwargs=params))
        self.assertEqual(resp.status_code, 200)

    def test_view_reopen_session(self):
        # Check attributes
zwang180's avatar
zwang180 committed
        self.assertEqual(len(GradingOpportunity.objects.all()), 1)
        opportunity = GradingOpportunity.objects.all()[0]
        self.assertEqual(self.course, opportunity.course)
        self.assertEqual(self.flow_id, opportunity.flow_id)
zwang180's avatar
zwang180 committed
        all_session = FlowSession.objects.all()
        # Check flow numbers
        self.assertEqual(len(all_session), len(self.flow_session_ids))
        # Check each flow session
zwang180's avatar
zwang180 committed
        for session in all_session:
            self.check_reopen_session(session.id, opportunity.id)
        # Check flow numbers again
Dong Zhuang's avatar
Dong Zhuang committed
        self.assertEqual(FlowSession.objects.all().count(),
                         len(self.flow_session_ids))
    # Just show the grading interfaces of the unanswered pages
    # the answered pages are tested in tests.teat_pages.test_generic
    def test_view_grade_flow_page(self):
Dong Zhuang's avatar
Dong Zhuang committed
        params = {"course_identifier": self.course.identifier,
                  "flow_session_id": self.flow_session_ids[0]}

        page_count = FlowSession.objects.get(id=self.flow_session_ids[0]).page_count
        for i in range(page_count):
            resp = self.client.get(
                self.get_page_grading_url_by_ordinal(page_ordinal=i, **params))
            self.assertEqual(resp.status_code, 200)

        # test PageOrdinalOutOfRange
        resp = self.client.get(
            self.get_page_grading_url_by_ordinal(page_ordinal=page_count+1,
                                                 **params))
        self.assertEqual(resp.status_code, 404)

    def test_view_grader_statistics(self):
Dong Zhuang's avatar
Dong Zhuang committed
        params = {"course_identifier": self.course.identifier,
        resp = self.client.get(reverse("relate-show_grader_statistics",
                                            kwargs=params))
        self.assertEqual(resp.status_code, 200)

zwang180's avatar
zwang180 committed
    def test_view_grant_exception_new_session(self):
zwang180's avatar
zwang180 committed
        all_session = FlowSession.objects.all()
        # Check number of flow sessions and ids
        self.assertEqual(all_session.count(), len(self.flow_session_ids))
zwang180's avatar
zwang180 committed
        for session in all_session:
            # Perform all checking before moving to stage three
            params = self.check_stage_one_and_two(session.participation)
            self.assertTrue(session.id in self.flow_session_ids)
            self.check_grant_new_exception(params)
Dong Zhuang's avatar
Dong Zhuang committed
        self.assertEqual(FlowSession.objects.all().count(),
                         2 * self.n_quiz_takers)

    def test_view_grant_exception_exist_session(self):
        # Store numbers to reuse
        session_nums = len(self.flow_session_ids)
zwang180's avatar
zwang180 committed

        all_session = FlowSession.objects.all()
        # Check session numbers
zwang180's avatar
zwang180 committed
        self.assertEqual(len(all_session), session_nums)

        # Check for each existing session
zwang180's avatar
zwang180 committed
        for session in all_session:
            # Perform all checking before moving to stage three
            params = self.check_stage_one_and_two(session.participation)
            self.check_grant_exist_exception(session.id, params)

        # Should have two exception rules now
        # One for access and one for grading
        self.assertEqual(len(FlowRuleException.objects.all()), 2 * session_nums)
    # Helper method for testing grant exceptions for new session
    def check_grant_new_exception(self, params):
zwang180's avatar
zwang180 committed
        # Grant a new one
        data = {"access_rules_tag_for_new_session": ["<<<NONE>>>"],
                    "create_session": ["Create session"]}
        resp = self.client.post(reverse("relate-grant_exception_stage_2",
Dong Zhuang's avatar
Dong Zhuang committed
                                                kwargs=params), data)
zwang180's avatar
zwang180 committed
        self.assertEqual(resp.status_code, 200)

    # Helper method for testing grant exceptions for existing one
    def check_grant_exist_exception(self, session_id, parameters):
        params = parameters.copy()
        flow_session = FlowSession.objects.filter(id=session_id)[0]
        self.assertTrue(flow_session.id in self.flow_session_ids)
zwang180's avatar
zwang180 committed

        # Grant an existing one
        data = {"session": [str(flow_session.id)], "next": ["Next \xbb"]}
        resp = self.client.post(reverse("relate-grant_exception_stage_2",
Dong Zhuang's avatar
Dong Zhuang committed
                                                kwargs=params), data)
zwang180's avatar
zwang180 committed
        self.assertEqual(resp.status_code, 302)

        # Prepare parameters
Dong Zhuang's avatar
Dong Zhuang committed
        params["session_id"] = data["session"][0]
zwang180's avatar
zwang180 committed
        # Check redirect
        self.assertEqual(resp.url, reverse("relate-grant_exception_stage_3",
                                                                kwargs=params))

        # Check stage three page
        resp = self.client.get(reverse("relate-grant_exception_stage_3",
zwang180's avatar
zwang180 committed
                                                                kwargs=params))
        self.assertEqual(resp.status_code, 200)

        # Create a new exception rule
        data = {"comment": ["test-rule"], "save": ["Save"], "view": ["on"],
                "see_answer_after_submission": ["on"],
                "create_grading_exception": ["on"],
                "create_access_exception": ["on"],
                "access_expires": [""], "due": [""],
                "bonus_points": ["0.0"], "max_points": [""],
                "credit_percent": ["100.0"], "max_points_enforced_cap": [""],
                "generates_grade": ["on"], "see_correctness": ["on"]}
        resp = self.client.post(reverse("relate-grant_exception_stage_3",
Dong Zhuang's avatar
Dong Zhuang committed
                                                kwargs=params), data)
zwang180's avatar
zwang180 committed
        self.assertEqual(resp.status_code, 302)

        # Check redirect
        self.assertEqual(resp.url, reverse("relate-grant_exception",
Dong Zhuang's avatar
Dong Zhuang committed
                                        args=[self.course.identifier]))
    # Helper method for testing reopen session
    def check_reopen_session(self, session_id, opportunity_id):
        flow_session = FlowSession.objects.filter(id=session_id)[0]
        self.assertEqual(flow_session.in_progress, False)

        # Check reopen session form
Dong Zhuang's avatar
Dong Zhuang committed
        params = {"course_identifier": self.course.identifier,
                    "opportunity_id": opportunity_id,
                    "flow_session_id": session_id}
        resp = self.client.get(reverse("relate-view_reopen_session",
                                                    kwargs=params))
        self.assertEqual(resp.status_code, 200)

        # Reopen session
        data = {"set_access_rules_tag": ["<<<NONE>>>"],
                "comment": ["test-reopen"],
                "unsubmit_pages": ["on"],
                "reopen": ["Reopen"]}
        resp = self.client.post(
Dong Zhuang's avatar
Dong Zhuang committed
            reverse("relate-view_reopen_session", kwargs=params), data)

        flow_session = FlowSession.objects.filter(id=session_id)[0]
        self.assertEqual(flow_session.in_progress, True)
zwang180's avatar
zwang180 committed

    # Helper method for testing grant exception view
    def check_stage_one_and_two(self, participation):
zwang180's avatar
zwang180 committed
        # Check stage one page
        resp = self.client.get(
Dong Zhuang's avatar
Dong Zhuang committed
            reverse("relate-grant_exception", args=[self.course.identifier]))
zwang180's avatar
zwang180 committed
        self.assertEqual(resp.status_code, 200)

        # Move to stage two
Dong Zhuang's avatar
Dong Zhuang committed
        self.assertEqual(Participation.objects.all().count(), self.n_participations)
Dong Zhuang's avatar
Dong Zhuang committed
        data = {"next": ["Next \xbb"],
                "participation": [str(participation.id)],
                "flow_id": [self.flow_id]}
        resp = self.client.post(reverse("relate-grant_exception",
Dong Zhuang's avatar
Dong Zhuang committed
                                   args=[self.course.identifier]), data)
zwang180's avatar
zwang180 committed
        self.assertEqual(resp.status_code, 302)

        # Prepare parameters
Dong Zhuang's avatar
Dong Zhuang committed
        params = data.copy()
zwang180's avatar
zwang180 committed
        params["participation_id"] = params["participation"][0]
Dong Zhuang's avatar
Dong Zhuang committed
        params["course_identifier"] = self.course.identifier
zwang180's avatar
zwang180 committed
        params["flow_id"] = params["flow_id"][0]
        del params["next"]
        del params["participation"]
        # Check redirect
Dong Zhuang's avatar
Dong Zhuang committed
        self.assertEqual(resp.url,
                         reverse(
                             "relate-grant_exception_stage_2", kwargs=params))
zwang180's avatar
zwang180 committed

        # Check stage two page
        resp = self.client.get(
Dong Zhuang's avatar
Dong Zhuang committed
            reverse("relate-grant_exception_stage_2", kwargs=params))
zwang180's avatar
zwang180 committed
        self.assertEqual(resp.status_code, 200)

        # Return params to reuse
        return params
Josh Asplund's avatar
Josh Asplund committed
@pytest.mark.slow
Dong Zhuang's avatar
Dong Zhuang committed
class GradeTwoQuizTakerTest(GradeGenericTestMixin, TestCase):
Dong Zhuang's avatar
Dong Zhuang committed

    force_login_student_for_each_test = False

Dong Zhuang's avatar
Dong Zhuang committed
    @classmethod
    def setUpTestData(cls): # noqa
        super().setUpTestData()
        client = Client()

        cls.do_quiz(client, cls.instructor_participation)
Dong Zhuang's avatar
Dong Zhuang committed
        cls.n_quiz_takers = 2
        cls.n_participations = 3

    def setUp(self):
        super().setUp()
        # Make sure the instructor is logged in after all quizzes finished
        self.client.force_login(self.instructor_participation.user)
Josh Asplund's avatar
Josh Asplund committed
@pytest.mark.slow
Dong Zhuang's avatar
Dong Zhuang committed
class GradeThreeQuizTakerTest(GradeGenericTestMixin, TestCase):
Dong Zhuang's avatar
Dong Zhuang committed

    force_login_student_for_each_test = False

Dong Zhuang's avatar
Dong Zhuang committed
    @classmethod
    def setUpTestData(cls): # noqa
        super().setUpTestData()
        client = Client()
        cls.do_quiz(client, cls.ta_participation)
        cls.do_quiz(client, cls.instructor_participation)
Dong Zhuang's avatar
Dong Zhuang committed
        cls.n_quiz_takers = 3
        cls.n_participations = 3

    def setUp(self):
        super().setUp()
        # Make sure the instructor is logged in after all quizzes finished
        self.client.force_login(self.instructor_participation.user)
Josh Asplund's avatar
Josh Asplund committed
@pytest.mark.slow
class GradePermissionsTests(SingleCoursePageTestMixin, TestCase):
    @classmethod
    def setUpTestData(cls):
        super().setUpTestData()

        client = Client()
        client.force_login(cls.student_participation.user)

        cls.start_flow(client, flow_id=cls.flow_id)
        cls.end_flow(client)

    def view_grades_permission(self, user, status_codes):
        try:
            participation = Participation.objects.get(user=user)
        except Participation.DoesNotExist:
            participation = self.student_participation

        urlname_views = ([
            ("relate-view_gradebook",
             {"course_identifier": self.course.identifier}),
            ("relate-view_grades_by_opportunity",
             {"course_identifier": self.course.identifier, "opp_id": 1}),
            ("relate-view_grading_opportunity_list",
             {"course_identifier": self.course.identifier}),
            ("relate-view_participant_grades",
             {"course_identifier": self.course.identifier,
              "participation_id": participation.pk}),
            ("relate-view_participant_list",
             {"course_identifier": self.course.identifier}),
            ("relate-view_reopen_session",
             {"course_identifier": self.course.identifier, "flow_session_id": 1,
              "opportunity_id": 1}),
            ("relate-view_single_grade",
             {"course_identifier": self.course.identifier,
              "participation_id": participation.pk, "opportunity_id": 1}),
            ("relate-export_gradebook_csv",
             {"course_identifier": self.course.identifier}),
            ("relate-import_grades",
             {"course_identifier": self.course.identifier}),
            ("relate-download_all_submissions",
             {"course_identifier": self.course.identifier,
              "flow_id": self.flow_id}),
            ("relate-edit_grading_opportunity",
             {"course_identifier": self.course.identifier, "opportunity_id": 1})]
        )
        with self.temporarily_switch_to_user(user):
            for (urlname, kwargs) in urlname_views:
                try:
                    url = reverse(urlname, kwargs=kwargs)
                except NoReverseMatch:
                    self.fail(
                        "Reversal of url named '%s' failed with "
                        "NoReverseMatch" % urlname)
                with self.subTest(user=user, urlname=urlname, method="GET"):
                    resp = self.client.get(url)
                    self.assertEqual(
                        resp.status_code,
                        status_codes.get(
                            urlname + "_get",
                            status_codes.get(
                                urlname,
                                status_codes.get("default_status_code")
                            )))

                with self.subTest(user=user, urlname=urlname, method="POST"):
                    postdata = {}
                    resp = self.client.post(url, data=postdata)
                    self.assertEqual(
                        resp.status_code,
                        status_codes.get(
                            urlname + "_post",
                            status_codes.get(
                                urlname,
                                status_codes.get("default_status_code")
                            )))

    def test_view_grades_instructor(self):
        status_codes = {"default_status_code": 200,

                        # no action_defined
                        "relate-view_single_grade_post": 400,
                        "relate-view_grades_by_opportunity_post": 400}
        self.view_grades_permission(self.instructor_participation.user,
                                    status_codes)

    def test_view_grades_ta(self):
        status_codes = {"default_status_code": 200,
                        "relate-edit_grading_opportunity": 403,
                        "relate-import_grades": 403,
                        "relate-export_gradebook_csv": 403,

                        # no action_defined
                        "relate-view_single_grade_post": 400,
                        "relate-view_grades_by_opportunity": 200}
        self.view_grades_permission(self.ta_participation.user,
                                    status_codes)

    def test_view_grades_student(self):
        status_codes = {"default_status_code": 403,
                        "relate-view_participant_grades": 200,

                        # no action_defined
                        "relate-view_single_grade_post": 400,
                        "relate-view_single_grade": 200}
        self.view_grades_permission(self.student_participation.user,
                                    status_codes)

    def test_view_grades_anonymous(self):
        status_codes = {"default_status_code": 403}
        self.view_grades_permission(None, status_codes)