test_api.py 8.2 KB
Newer Older
1 2 3
"""
Test for course API
"""
4
from hashlib import md5
5 6

from django.contrib.auth.models import AnonymousUser
7
from django.http import Http404
8
from opaque_keys.edx.keys import CourseKey
9
from rest_framework.exceptions import PermissionDenied
10 11
from rest_framework.request import Request
from rest_framework.test import APIRequestFactory
12

13
from openedx.core.djangoapps.content.course_overviews.models import CourseOverview
14
from xmodule.modulestore.tests.django_utils import SharedModuleStoreTestCase, ModuleStoreTestCase
15
from xmodule.modulestore.tests.factories import check_mongo_calls
16
from .mixins import CourseApiFactoryMixin
17
from ..api import course_detail, list_courses
18 19 20 21 22 23


class CourseApiTestMixin(CourseApiFactoryMixin):
    """
    Establish basic functionality for Course API tests
    """
24 25 26 27
    @classmethod
    def setUpClass(cls):
        super(CourseApiTestMixin, cls).setUpClass()
        cls.request_factory = APIRequestFactory()
28
        CourseOverview.get_all_courses()  # seed the CourseOverview table
29

30
    def verify_course(self, course, course_id=u'edX/toy/2012_Fall'):
31 32 33 34 35
        """
        Ensure that the returned course is the course we just created
        """
        self.assertEqual(course_id, str(course.id))

36 37 38 39 40 41 42 43 44 45

class CourseDetailTestMixin(CourseApiTestMixin):
    """
    Common functionality for course_detail tests
    """
    def _make_api_call(self, requesting_user, target_user, course_key):
        """
        Call the `course_detail` api endpoint to get information on the course
        identified by `course_key`.
        """
46
        request = Request(self.request_factory.get('/'))
47
        request.user = requesting_user
48 49
        with check_mongo_calls(0):
            return course_detail(request, target_user.username, course_key)
50 51 52 53 54 55 56 57 58 59 60 61 62 63 64


class TestGetCourseDetail(CourseDetailTestMixin, SharedModuleStoreTestCase):
    """
    Test course_detail api function
    """
    @classmethod
    def setUpClass(cls):
        super(TestGetCourseDetail, cls).setUpClass()
        cls.course = cls.create_course()
        cls.hidden_course = cls.create_course(course=u'hidden', visible_to_staff_only=True)
        cls.honor_user = cls.create_user('honor', is_staff=False)
        cls.staff_user = cls.create_user('staff', is_staff=True)

    def test_get_existing_course(self):
65 66
        course = self._make_api_call(self.honor_user, self.honor_user, self.course.id)
        self.verify_course(course)
67 68 69

    def test_get_nonexistent_course(self):
        course_key = CourseKey.from_string(u'edX/toy/nope')
70
        with self.assertRaises(Http404):
71
            self._make_api_call(self.honor_user, self.honor_user, course_key)
72 73

    def test_hidden_course_for_honor(self):
74
        with self.assertRaises(Http404):
75
            self._make_api_call(self.honor_user, self.honor_user, self.hidden_course.id)
76 77

    def test_hidden_course_for_staff(self):
78 79
        course = self._make_api_call(self.staff_user, self.staff_user, self.hidden_course.id)
        self.verify_course(course, course_id=u'edX/hidden/2012_Fall')
80 81

    def test_hidden_course_for_staff_as_honor(self):
82
        with self.assertRaises(Http404):
83
            self._make_api_call(self.staff_user, self.honor_user, self.hidden_course.id)
84 85 86 87 88 89


class CourseListTestMixin(CourseApiTestMixin):
    """
    Common behavior for list_courses tests
    """
90
    def _make_api_call(self, requesting_user, specified_user, org=None, filter_=None):
91 92 93 94
        """
        Call the list_courses api endpoint to get information about
        `specified_user` on behalf of `requesting_user`.
        """
95
        request = Request(self.request_factory.get('/'))
96
        request.user = requesting_user
97
        with check_mongo_calls(0):
98
            return list_courses(request, specified_user.username, org=org, filter_=filter_)
99

100 101 102 103 104 105 106
    def verify_courses(self, courses):
        """
        Verify that there is one course, and that it has the expected format.
        """
        self.assertEqual(len(courses), 1)
        self.verify_course(courses[0])

107 108 109 110 111

class TestGetCourseList(CourseListTestMixin, SharedModuleStoreTestCase):
    """
    Test the behavior of the `list_courses` api function.
    """
112

113 114 115
    @classmethod
    def setUpClass(cls):
        super(TestGetCourseList, cls).setUpClass()
116
        cls.course = cls.create_course()
117 118 119 120 121 122
        cls.staff_user = cls.create_user("staff", is_staff=True)
        cls.honor_user = cls.create_user("honor", is_staff=False)

    def test_as_staff(self):
        courses = self._make_api_call(self.staff_user, self.staff_user)
        self.assertEqual(len(courses), 1)
123
        self.verify_courses(courses)
124 125 126

    def test_for_honor_user_as_staff(self):
        courses = self._make_api_call(self.staff_user, self.honor_user)
127
        self.verify_courses(courses)
128 129 130

    def test_as_honor(self):
        courses = self._make_api_call(self.honor_user, self.honor_user)
131
        self.verify_courses(courses)
132 133 134 135 136 137 138 139

    def test_for_staff_user_as_honor(self):
        with self.assertRaises(PermissionDenied):
            self._make_api_call(self.honor_user, self.staff_user)

    def test_as_anonymous(self):
        anonuser = AnonymousUser()
        courses = self._make_api_call(anonuser, anonuser)
140
        self.verify_courses(courses)
141 142 143 144 145 146

    def test_for_honor_user_as_anonymous(self):
        anonuser = AnonymousUser()
        with self.assertRaises(PermissionDenied):
            self._make_api_call(anonuser, self.staff_user)

147
    @SharedModuleStoreTestCase.modifies_courseware
148 149 150 151 152
    def test_multiple_courses(self):
        self.create_course(course='second')
        courses = self._make_api_call(self.honor_user, self.honor_user)
        self.assertEqual(len(courses), 2)

153 154 155 156 157 158 159 160 161 162 163 164 165 166 167 168 169 170 171 172 173 174 175
    @SharedModuleStoreTestCase.modifies_courseware
    def test_filter_by_org(self):
        """Verify that courses are filtered by the provided org key."""
        # Create a second course to be filtered out of queries.
        alternate_course = self.create_course(
            org=md5(self.course.org).hexdigest()
        )

        self.assertNotEqual(alternate_course.org, self.course.org)

        # No filtering.
        unfiltered_courses = self._make_api_call(self.staff_user, self.staff_user)
        for org in [self.course.org, alternate_course.org]:
            self.assertTrue(
                any(course.org == org for course in unfiltered_courses)
            )

        # With filtering.
        filtered_courses = self._make_api_call(self.staff_user, self.staff_user, org=self.course.org)
        self.assertTrue(
            all(course.org == self.course.org for course in filtered_courses)
        )

176 177 178 179 180 181 182 183 184 185 186 187 188 189 190 191 192 193
    @SharedModuleStoreTestCase.modifies_courseware
    def test_filter(self):
        # Create a second course to be filtered out of queries.
        alternate_course = self.create_course(course='mobile', mobile_available=True)

        test_cases = [
            (None, [alternate_course, self.course]),
            (dict(mobile_available=True), [alternate_course]),
            (dict(mobile_available=False), [self.course]),
        ]
        for filter_, expected_courses in test_cases:
            filtered_courses = self._make_api_call(self.staff_user, self.staff_user, filter_=filter_)
            self.assertEquals(
                {course.id for course in filtered_courses},
                {course.id for course in expected_courses},
                "testing course_api.api.list_courses with filter_={}".format(filter_),
            )

194 195 196 197 198 199 200 201 202 203 204 205 206 207 208 209 210 211 212 213 214 215 216 217

class TestGetCourseListExtras(CourseListTestMixin, ModuleStoreTestCase):
    """
    Tests of course_list api function that require alternative configurations
    of created courses.
    """
    @classmethod
    def setUpClass(cls):
        super(TestGetCourseListExtras, cls).setUpClass()
        cls.staff_user = cls.create_user("staff", is_staff=True)
        cls.honor_user = cls.create_user("honor", is_staff=False)

    def test_no_courses(self):
        courses = self._make_api_call(self.honor_user, self.honor_user)
        self.assertEqual(len(courses), 0)

    def test_hidden_course_for_honor(self):
        self.create_course(visible_to_staff_only=True)
        courses = self._make_api_call(self.honor_user, self.honor_user)
        self.assertEqual(len(courses), 0)

    def test_hidden_course_for_staff(self):
        self.create_course(visible_to_staff_only=True)
        courses = self._make_api_call(self.staff_user, self.staff_user)
218
        self.verify_courses(courses)