Commit ad14deba by Chris Dodge

remove unused is_feature_enabled method and update docs

parent 0653f84d
...@@ -15,13 +15,13 @@ In order to use edx-proctoring, you must obtain an account (and secret configura ...@@ -15,13 +15,13 @@ In order to use edx-proctoring, you must obtain an account (and secret configura
CONFIGURATION: CONFIGURATION:
You will need to turn on the ENABLE_PROCTORED_EXAMS in lms.env.json and cms.env.json FEATURES dictionary: You will need to turn on the ENABLE_SPECIAL_EXAMS in lms.env.json and cms.env.json FEATURES dictionary:
``` ```
: :
"FEATURES": { "FEATURES": {
: :
"ENABLE_PROCTORED_EXAMS": true, "ENABLE_SPECIAL_EXAMS": true,
: :
} }
``` ```
......
...@@ -49,14 +49,6 @@ from edx_proctoring.runtime import get_runtime_service ...@@ -49,14 +49,6 @@ from edx_proctoring.runtime import get_runtime_service
log = logging.getLogger(__name__) log = logging.getLogger(__name__)
def is_feature_enabled():
"""
Returns if this feature has been enabled in our FEATURE flags
"""
return hasattr(settings, 'FEATURES') and settings.FEATURES.get('ENABLE_PROCTORED_EXAMS', False)
def create_exam(course_id, content_id, exam_name, time_limit_mins, due_date=None, def create_exam(course_id, content_id, exam_name, time_limit_mins, due_date=None,
is_proctored=True, is_practice_exam=False, external_id=None, is_active=True): is_proctored=True, is_practice_exam=False, external_id=None, is_active=True):
""" """
......
...@@ -31,7 +31,6 @@ from edx_proctoring.api import ( ...@@ -31,7 +31,6 @@ from edx_proctoring.api import (
remove_exam_attempt, remove_exam_attempt,
get_all_exam_attempts, get_all_exam_attempts,
get_filtered_exam_attempts, get_filtered_exam_attempts,
is_feature_enabled,
mark_exam_attempt_timeout, mark_exam_attempt_timeout,
mark_exam_attempt_as_ready, mark_exam_attempt_as_ready,
update_attempt_status, update_attempt_status,
...@@ -310,18 +309,6 @@ class ProctoredExamApiTests(LoggedInTestCase): ...@@ -310,18 +309,6 @@ class ProctoredExamApiTests(LoggedInTestCase):
proctored_exam_id=self.proctored_exam_id, user_id=self.user_id, key=self.key, value=self.value proctored_exam_id=self.proctored_exam_id, user_id=self.user_id, key=self.key, value=self.value
) )
def test_feature_enabled(self):
"""
Checks the is_feature_enabled method
"""
self.assertFalse(is_feature_enabled())
with patch.dict('django.conf.settings.FEATURES', {'ENABLE_PROCTORED_EXAMS': False}):
self.assertFalse(is_feature_enabled())
with patch.dict('django.conf.settings.FEATURES', {'ENABLE_PROCTORED_EXAMS': True}):
self.assertTrue(is_feature_enabled())
def test_create_duplicate_exam(self): def test_create_duplicate_exam(self):
""" """
Test to create a proctored exam that has already exist in the Test to create a proctored exam that has already exist in the
......
...@@ -34,7 +34,7 @@ def load_requirements(*requirements_paths): ...@@ -34,7 +34,7 @@ def load_requirements(*requirements_paths):
setup( setup(
name='edx-proctoring', name='edx-proctoring',
version='0.10.13', version='0.10.14',
description='Proctoring subsystem for Open edX', description='Proctoring subsystem for Open edX',
long_description=open('README.md').read(), long_description=open('README.md').read(),
author='edX', author='edX',
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment