diff --git a/lms/djangoapps/learner_dashboard/tests/test_programs.py b/lms/djangoapps/learner_dashboard/tests/test_programs.py index ed1e1f9ae80497558f56035385b8beefd3b22ee2..5dd270a15fa301c259ad725844b27c2c2ef72b12 100644 --- a/lms/djangoapps/learner_dashboard/tests/test_programs.py +++ b/lms/djangoapps/learner_dashboard/tests/test_programs.py @@ -21,7 +21,6 @@ from openedx.core.djangoapps.catalog.tests.factories import ( ProgramFactory ) from openedx.core.djangoapps.catalog.tests.mixins import CatalogIntegrationMixin -from openedx.core.djangoapps.credentials import STUDENT_RECORDS_FLAG from openedx.core.djangoapps.programs.tests.mixins import ProgramsApiConfigMixin from openedx.core.djangoapps.waffle_utils.testutils import override_waffle_flag from openedx.core.djangolib.testing.utils import skip_unless_lms @@ -183,7 +182,6 @@ class TestProgramListing(ProgramsApiConfigMixin, SharedModuleStoreTestCase): @skip_unless_lms @mock.patch(PROGRAMS_MODULE + '.get_credit_pathways') @mock.patch(PROGRAMS_UTILS_MODULE + '.get_programs') -@override_waffle_flag(STUDENT_RECORDS_FLAG, active=True) class TestProgramDetails(ProgramsApiConfigMixin, CatalogIntegrationMixin, SharedModuleStoreTestCase): """Unit tests for the program details page.""" shard = 4 diff --git a/openedx/core/djangoapps/credentials/__init__.py b/openedx/core/djangoapps/credentials/__init__.py index 106a5a5257c31ebafe9ad17601ef7cb2719a6560..b7c6418ae687bef43012840c7cd7456e3f84afc3 100644 --- a/openedx/core/djangoapps/credentials/__init__.py +++ b/openedx/core/djangoapps/credentials/__init__.py @@ -4,10 +4,3 @@ edX Platform support for credentials. This package will be used as a wrapper for interacting with the credentials service. """ - -from openedx.core.djangoapps.waffle_utils import WaffleFlag, WaffleFlagNamespace - -WAFFLE_FLAG_NAMESPACE = WaffleFlagNamespace(name='credentials') - -# Waffle flag to enable the experimental Student Records feature -STUDENT_RECORDS_FLAG = WaffleFlag(WAFFLE_FLAG_NAMESPACE, 'student_records') diff --git a/openedx/core/djangoapps/credentials/models.py b/openedx/core/djangoapps/credentials/models.py index 5630013d51a423ce2f1cdac37628cde08a32db12..7fa094222c4bf5a2a75ec3447a55a0215913b316 100644 --- a/openedx/core/djangoapps/credentials/models.py +++ b/openedx/core/djangoapps/credentials/models.py @@ -11,8 +11,6 @@ from django.utils.translation import ugettext_lazy as _ from openedx.core.djangoapps.site_configuration import helpers -from . import STUDENT_RECORDS_FLAG - API_VERSION = 'v2' @@ -93,9 +91,6 @@ class CredentialsApiConfig(ConfigurationModel): """ Publicly-accessible Records URL root. """ - # Temporarily disable this feature while we work on it - if not STUDENT_RECORDS_FLAG.is_enabled(): - return None # Not every site wants the Learner Records feature, so we allow opting out. if not helpers.get_value('ENABLE_LEARNER_RECORDS', True): return None diff --git a/openedx/features/learner_profile/tests/views/test_learner_profile.py b/openedx/features/learner_profile/tests/views/test_learner_profile.py index 0499e53690950619a6968b7e4c7122ba5b3cd5b9..a9485b69d1d3ad51187618394157d10d5250756e 100644 --- a/openedx/features/learner_profile/tests/views/test_learner_profile.py +++ b/openedx/features/learner_profile/tests/views/test_learner_profile.py @@ -13,7 +13,6 @@ from django.urls import reverse from django.test.client import RequestFactory from lms.djangoapps.certificates.api import is_passing_status from opaque_keys.edx.locator import CourseLocator -from openedx.core.djangoapps.credentials import STUDENT_RECORDS_FLAG from openedx.core.djangoapps.waffle_utils.testutils import override_waffle_flag from openedx.features.learner_profile.views.learner_profile import learner_profile_context from student.tests.factories import CourseEnrollmentFactory, UserFactory @@ -23,7 +22,6 @@ from xmodule.modulestore.tests.factories import CourseFactory @ddt.ddt -@override_waffle_flag(STUDENT_RECORDS_FLAG, active=True) class LearnerProfileViewTest(UrlResetMixin, ModuleStoreTestCase): """ Tests for the student profile view. """