Andal.LND/edx-platform
6
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

ECOM-528 added test cases

Browse Source
This commit is contained in:
arbisoft
2015-01-09 17:55:22 +05:00
parent fd76f3754c
commit 36626a2b8b
2 changed files with 61 additions and 99 deletions
Download Patch File Download Diff File Expand all files Collapse all files

156
common/djangoapps/student/tests/test_recent_enrollments.py
View File

@@ -127,43 +127,72 @@ class TestRecentEnrollments(ModuleStoreTestCase):
self.assertContains(response, "Thank you for enrolling in")
@ddt.data(
(['audit', 'honor'], 'honor', True),
(['professional'], 'honor', True),
(['verified'], 'honor', True),
(['professional', 'verified'], 'honor', True),
(['audit', 'honor', 'professional'], 'honor', True),
(['audit', 'honor', 'verified'], 'honor', True),
(['audit', 'honor', 'verified', 'professional'], 'honor', True),
(['audit'], 'honor', True),
(['honor'], 'honor', True),
#Register as an honor in any course modes with no payment option
([('audit', 0), ('honor', 0)], 'honor', True),
# ([('professional', 0)], 'honor', True), #This can not be the case as verified courses always has min_price
# ([('verified', 0)], 'honor', True), #This can not be the case as verified courses always has min_price
# ([('professional', 0), ('verified', 0)], 'honor', True), #This can not be the case as verified courses always has min_price
# ([('audit', 0), ('honor', 0), ('professional', 0)], 'honor', True), #This can not be the case as verified courses always has min_price
# ([('audit', 0), ('honor', 0), ('verified', 0)], 'honor', True), #This can not be the case as verified courses always has min_price
# ([('audit', 0), ('honor', 0), ('verified', 0), ('professional', 0)], 'honor', True), #This can not be the case as verified courses always has min_price
([('audit', 0)], 'honor', True),
([('honor', 0)], 'honor', True),
([], 'honor', True),
#Register as an honor in any course modes which has payment option
([('audit', 0), ('honor', 10)], 'honor', False),
([('professional', 20)], 'honor', True),
([('verified', 20)], 'honor', True),
([('professional', 20), ('verified', 20)], 'honor', True),
([('audit', 0), ('honor', 5), ('professional', 20)], 'honor', True),
([('audit', 0), ('honor', 10), ('verified', 20)], 'honor', True),
([('audit', 0), ('honor', 10), ('verified', 20), ('professional', 20)], 'honor', True),
([('audit', 10)], 'honor', False),
([('honor', 10)], 'honor', False),
([], 'honor', True),
(['audit', 'honor'], 'audit', True),
(['professional'], 'audit', True),
(['verified'], 'audit', True),
(['professional', 'verified'], 'audit', True),
(['audit', 'honor', 'professional'], 'audit', True),
(['audit', 'honor', 'verified'], 'audit', True),
(['audit', 'honor', 'verified', 'professional'], 'audit', True),
(['audit'], 'audit', True),
(['honor'], 'audit', True),
#Register as an audit in any course modes with no payment option
([('audit', 0), ('honor', 0)], 'audit', True),
# ([('professional', 0)], 'audit', True), #This can not be the case as verified courses always has min_price
# ([('verified', 0)], 'audit', True), #This can not be the case as verified courses always has min_price
# ([('professional', 0), ('verified', 0)], 'audit', True), #This can not be the case as verified courses always has min_price
# ([('audit', 0), ('honor', 0), ('professional', 0)], 'audit', True), #This can not be the case as verified courses always has min_price
# ([('audit', 0), ('honor', 0), ('verified', 0)], 'audit', True), #This can not be the case as verified courses always has min_price
# ([('audit', 0), ('honor', 0), ('verified', 0), ('professional', 0)], 'audit', True), #This can not be the case as verified courses always has min_price
([('audit', 0)], 'audit', True),
([('honor', 0)], 'audit', True),
([], 'audit', True),
#Register as an audit in any course modes which has payment option
([('audit', 0), ('honor', 10)], 'audit', False),
# ([('professional', 20)], 'audit', True), #This can not be the case as verified courses always has min_price
# ([('verified', 20)], 'audit', True), #This can not be the case as verified courses always has min_price
# ([('professional', 20), ('verified', 20)], 'audit', True), #This can not be the case as verified courses always has min_price
# ([('audit', 0), ('honor', 5), ('professional', 20)], 'audit', True), #This can not be the case as verified courses always has min_price
# ([('audit', 0), ('honor', 10), ('verified', 20)], 'audit', True), #This can not be the case as verified courses always has min_price
# ([('audit', 0), ('honor', 10), ('verified', 20), ('professional', 20)], 'audit', True), #This can not be the case as verified courses always has min_price
([('audit', 10)], 'audit', False),
([('honor', 10)], 'audit', False),
([], 'audit', True),
(['audit', 'honor'], 'verified', False),
(['professional'], 'verified', False),
(['verified'], 'verified', False),
(['professional', 'verified'], 'verified', False),
(['audit', 'honor', 'professional'], 'verified', False),
(['audit', 'honor', 'verified'], 'verified', False),
(['audit', 'honor', 'verified', 'professional'], 'verified', False),
(['audit'], 'verified', False),
(['honor'], 'verified', False),
([], 'verified', False)
#Register as a verified in any course modes which has payment option
([('audit', 0), ('honor', 10)], 'verified', False),
([('professional', 20)], 'verified', False),
([('verified', 20)], 'verified', False),
([('professional', 20), ('verified', 20)], 'verified', False),
([('audit', 0), ('honor', 5), ('professional', 20)], 'verified', False),
([('audit', 0), ('honor', 10), ('verified', 20)], 'verified', False),
([('audit', 0), ('honor', 10), ('verified', 20), ('professional', 20)], 'verified', False),
([('audit', 10)], 'verified', False),
([('honor', 10)], 'verified', False),
# ([], 'audit', True),
)
@ddt.unpack
def test_donate_button(self, course_modes, enrollment_mode, show_donate):
from nose.tools import set_trace;
set_trace()
# from nose.tools import set_trace;
# set_trace()
# Enable the enrollment success message
self._configure_message_timeout(10000)
@@ -171,8 +200,8 @@ class TestRecentEnrollments(ModuleStoreTestCase):
DonationConfiguration(enabled=True).save()
# Create the course mode(s)
for mode in course_modes:
CourseModeFactory(mode_slug=mode, course_id=self.course.id)
for mode, min_price in course_modes:
CourseModeFactory(mode_slug=mode, course_id=self.course.id, min_price=min_price)
self.enrollment.mode = enrollment_mode
self.enrollment.save()
@@ -186,69 +215,6 @@ class TestRecentEnrollments(ModuleStoreTestCase):
else:
self.assertNotContains(response, "donate-container")
# @ddt.data(
# (['audit', 'honor'], True),
# (['professional'], True),
# (['verified'], True),
# (['audit'], True),
# (['honor'], True),
# ([], True)
# )
# @ddt.unpack
# def test_donate_button_enrollment_audit(self, course_modes, show_donate):
# from nose.tools import set_trace; set_trace()
# # Enable the enrollment success message
# self._configure_message_timeout(10000)
#
# # Enable donations
# DonationConfiguration(enabled=True).save()
#
# # Create the course mode(s)
# for mode in course_modes:
# CourseModeFactory(mode_slug=mode, course_id=self.course.id)
# self.enrollment.mode = 'audit'
# self.enrollment.save()
# # Check that the donate button is or is not displayed
# self.client.login(username=self.student.username, password=self.PASSWORD)
# response = self.client.get(reverse("dashboard"))
#
# if show_donate:
# self.assertContains(response, "donate-container")
# else:
# self.assertNotContains(response, "donate-container")
#
# @ddt.data(
# (['audit', 'honor'], True),
# (['professional'], True),
# (['verified'], True),
# (['audit'], True),
# (['honor'], True),
# ([], True)
# )
# @ddt.unpack
# def test_donate_button_verified_courses(self, course_modes, show_donate):
# from nose.tools import set_trace; set_trace()
# # Enable the enrollment success message
# self._configure_message_timeout(10000)
#
# # Enable donations
# DonationConfiguration(enabled=True).save()
#
# # Create the course mode(s)
# for mode in course_modes:
# CourseModeFactory(mode_slug=mode, course_id=self.course.id)
#
# self.enrollment.mode = ''
#
# # Check that the donate button is or is not displayed
# self.client.login(username=self.student.username, password=self.PASSWORD)
# response = self.client.get(reverse("dashboard"))
#
# if show_donate:
# self.assertContains(response, "donate-container")
# else:
# self.assertNotContains(response, "donate-container")
def test_donate_button_honor_with_price(self):
# Enable the enrollment success message and donations
self._configure_message_timeout(10000)

4
common/djangoapps/student/views.py
View File

@@ -758,8 +758,6 @@ def _allow_donation(course_modes, course_id, enrollment):
True if the course is allowing donations.
"""
# from nose.tools import set_trace;
# set_trace()
donations_enabled = DonationConfiguration.current().enabled
is_verified_mode = CourseMode.has_verified_mode(course_modes[course_id])
has_payment_option = CourseMode.has_payment_options(course_id)
@@ -771,8 +769,6 @@ def _allow_donation(course_modes, course_id, enrollment):
else:
if enrollment.mode in ['audit', 'honor']:
return_val = True
# return_val = donations_enabled and ((not is_verified_mode and not has_payment_option) or (is_verified_mode and enrollment.mode in ['audit', 'honor']))
# return_val = donations_enabled and not is_verified_mode and not has_payment_option
return return_val