diff --git a/common/djangoapps/student/tests/test_bulk_email_settings.py b/common/djangoapps/student/tests/test_bulk_email_settings.py index 5965807897..a0e5a1b401 100644 --- a/common/djangoapps/student/tests/test_bulk_email_settings.py +++ b/common/djangoapps/student/tests/test_bulk_email_settings.py @@ -7,8 +7,8 @@ Course Auth is turned on. from django.test.utils import override_settings from django.conf import settings -from django.core.urlresolvers import reverse, NoReverseMatch -from unittest.case import SkipTest +from django.core.urlresolvers import reverse +import unittest from courseware.tests.tests import TEST_DATA_MONGO_MODULESTORE from student.tests.factories import UserFactory, CourseEnrollmentFactory @@ -23,6 +23,7 @@ from mock import patch @override_settings(MODULESTORE=TEST_DATA_MONGO_MODULESTORE) +@unittest.skipUnless(settings.ROOT_URLCONF == 'lms.urls', 'Test only valid in lms') class TestStudentDashboardEmailView(ModuleStoreTestCase): """ Check for email view displayed with flag @@ -35,11 +36,7 @@ class TestStudentDashboardEmailView(ModuleStoreTestCase): CourseEnrollmentFactory.create(user=student, course_id=self.course.id) self.client.login(username=student.username, password="test") - try: - # URL for dashboard - self.url = reverse('dashboard') - except NoReverseMatch: - raise SkipTest("Skip this test if url cannot be found (ie running from CMS tests)") + self.url = reverse('dashboard') # URL for email settings modal self.email_modal_link = ( ('