diff options
| -rw-r--r-- | yaksh/test_views.py | 154 | ||||
| -rw-r--r-- | yaksh/views.py | 4 |
2 files changed, 150 insertions, 8 deletions
diff --git a/yaksh/test_views.py b/yaksh/test_views.py index 1d1c3f3..2b7ad9b 100644 --- a/yaksh/test_views.py +++ b/yaksh/test_views.py @@ -139,6 +139,7 @@ class TestProfile(TestCase): self.assertEqual(response.status_code, 200) self.assertTemplateUsed(response, 'yaksh/editprofile.html') + class TestAddQuiz(TestCase): def setUp(self): self.client = Client() @@ -209,7 +210,7 @@ class TestAddQuiz(TestCase): self.pre_req_quiz.delete() self.course.delete() - def test_view_profile_denies_anonymous(self): + def test_add_quiz_denies_anonymous(self): """ If not logged in redirect to login page """ @@ -220,7 +221,7 @@ class TestAddQuiz(TestCase): redirect_destination = '/exam/login/?next=/exam/manage/addquiz/{0}/'.format(self.course.id) self.assertRedirects(response, redirect_destination) - def test_view_profile_denies_non_moderator(self): + def test_add_quiz_denies_non_moderator(self): """ If not moderator in redirect to login page """ @@ -342,6 +343,7 @@ class TestAddQuiz(TestCase): self.assertEqual(response.status_code, 302) self.assertRedirects(response, '/exam/manage/courses/') + class TestAddTeacher(TestCase): def setUp(self): self.client = Client() @@ -500,6 +502,7 @@ class TestAddTeacher(TestCase): self.assertIn(teacher_object, response.context['teachers_added']) self.assertIn(teacher_object, self.course.teachers.all()) + class TestRemoveTeacher(TestCase): def setUp(self): self.client = Client() @@ -758,6 +761,139 @@ class TestCourses(TestCase): self.assertNotIn(self.user2_course, response.context['courses']) +class TestAddCourse(TestCase): + def setUp(self): + self.client = Client() + + self.mod_group = Group.objects.create(name='moderator') + tzone = pytz.timezone('UTC') + + # Create Moderator with profile + self.user_plaintext_pass = 'demo' + self.user = User.objects.create_user( + username='demo_user', + password=self.user_plaintext_pass, + first_name='first_name', + last_name='last_name', + email='demo@test.com' + ) + + Profile.objects.create( + user=self.user, + roll_number=10, + institute='IIT', + department='Chemical', + position='Moderator', + timezone='UTC' + ) + + # Create Student + self.student_plaintext_pass = 'demo_student' + self.student = User.objects.create_user( + username='demo_student', + password=self.student_plaintext_pass, + first_name='student_first_name', + last_name='student_last_name', + email='demo_student@test.com' + ) + + # Add to moderator group + self.mod_group.user_set.add(self.user) + + self.course = Course.objects.create(name="Python Course", + enrollment="Enroll Request", creator=self.user) + + self.pre_req_quiz = Quiz.objects.create( + start_date_time=datetime(2014, 2, 1, 5, 8, 15, 0, tzone), + end_date_time=datetime(2015, 10, 9, 10, 8, 15, 0, tzone), + duration=30, active=True, + attempts_allowed=-1, time_between_attempts=0, + description='pre requisite quiz', pass_criteria=40, + language='Python', prerequisite=None, + course=self.course + ) + + self.quiz = Quiz.objects.create( + start_date_time=datetime(2014, 10, 9, 10, 8, 15, 0, tzone), + end_date_time=datetime(2015, 10, 9, 10, 8, 15, 0, tzone), + duration=30, active=True, + attempts_allowed=-1, time_between_attempts=0, + description='demo quiz', pass_criteria=40, + language='Python', prerequisite=self.pre_req_quiz, + course=self.course + ) + + def tearDown(self): + self.client.logout() + self.user.delete() + self.student.delete() + self.quiz.delete() + self.pre_req_quiz.delete() + self.course.delete() + + def test_add_course_denies_anonymous(self): + """ + If not logged in redirect to login page + """ + response = self.client.get(reverse('yaksh:add_course'), + follow=True + ) + redirect_destination = ('/exam/login/?next=/' + 'exam/manage/add_course/') + self.assertRedirects(response, redirect_destination) + + def test_add_course_denies_non_moderator(self): + """ + If not moderator in redirect to login page + """ + self.client.login( + username=self.student.username, + password=self.student_plaintext_pass + ) + course_id = self.course.id + response = self.client.get(reverse('yaksh:add_course'), + follow=True + ) + self.assertEqual(response.status_code, 404) + + def test_add_course_get(self): + """ + GET request to add course should display add course form + """ + self.client.login( + username=self.user.username, + password=self.user_plaintext_pass + ) + response = self.client.get(reverse('yaksh:add_course')) + self.assertEqual(response.status_code, 200) + self.assertTemplateUsed(response, 'yaksh/add_course.html') + self.assertIsNotNone(response.context['form']) + + def test_add_course_post_new_course(self): + """ + POST request to add course should add new courses if no course exists + """ + self.client.login( + username=self.user.username, + password=self.user_plaintext_pass + ) + + response = self.client.post(reverse('yaksh:add_course'), + data={'name': 'new_demo_course_1', + 'active': True, + 'enrollment': 'open' + } + ) + course_list = Course.objects.all().order_by('-id') + new_course = course_list[0] + self.assertEqual(new_course.name, 'new_demo_course_1') + self.assertEqual(new_course.enrollment, 'open') + self.assertEqual(new_course.active, True) + + self.assertEqual(response.status_code, 302) + self.assertRedirects(response, '/exam/manage/') + + class TestCourseDetail(TestCase): def setUp(self): self.client = Client() @@ -829,11 +965,13 @@ class TestCourseDetail(TestCase): """ If not logged in redirect to login page """ - response = self.client.get(reverse('yaksh:add_course'), + response = self.client.get(reverse('yaksh:course_detail', + kwargs={'course_id': self.user1_course.id} + ), follow=True ) - redirect_destination = ('/exam/login/?next=/exam' - '/manage/add_course/') + redirect_destination = ('/exam/login/?next=/exam/' + 'manage/course_detail/1/') self.assertRedirects(response, redirect_destination) def test_course_detail_denies_non_moderator(self): @@ -845,7 +983,9 @@ class TestCourseDetail(TestCase): password=self.student_plaintext_pass ) - response = self.client.get(reverse('yaksh:add_course'), + response = self.client.get(reverse('yaksh:course_detail', + kwargs={'course_id': self.user1_course.id} + ), follow=True ) self.assertEqual(response.status_code, 404) @@ -882,6 +1022,7 @@ class TestCourseDetail(TestCase): self.assertEqual(response.status_code, 200) self.assertTemplateUsed(response, 'yaksh/course_detail.html') + class TestEnrollRequest(TestCase): def setUp(self): self.client = Client() @@ -988,6 +1129,7 @@ class TestEnrollRequest(TestCase): ) self.assertRedirects(response, '/exam/manage/') + class TestViewAnswerPaper(TestCase): def setUp(self): self.client = Client() diff --git a/yaksh/views.py b/yaksh/views.py index 68e641e..dc6f2fa 100644 --- a/yaksh/views.py +++ b/yaksh/views.py @@ -299,7 +299,7 @@ def show_all_questionpapers(request, questionpaper_id=None): @login_required def prof_manage(request): """Take credentials of the user with professor/moderator -rights/permissions and log in.""" + rights/permissions and log in.""" user = request.user ci = RequestContext(request) if user.is_authenticated() and is_moderator(user): @@ -586,7 +586,7 @@ def add_course(request): new_course = form.save(commit=False) new_course.creator = user new_course.save() - return my_render_to_response('manage.html', {'course': new_course}) + return my_redirect('/exam/manage/') else: return my_render_to_response('yaksh/add_course.html', {'form': form}, |