Skip to content

Commit 82337a5

Browse files
test: remove edxnotes related tests in course_settings tests
1 parent 26ed661 commit 82337a5

1 file changed

Lines changed: 0 additions & 75 deletions

File tree

cms/djangoapps/contentstore/tests/test_course_settings.py

Lines changed: 0 additions & 75 deletions
Original file line numberDiff line numberDiff line change
@@ -1230,80 +1230,6 @@ def test_update_from_json_filtered_off(self):
12301230
)
12311231
self.assertNotIn('giturl', test_model)
12321232

1233-
@patch.dict(settings.FEATURES, {'ENABLE_EDXNOTES': True})
1234-
def test_edxnotes_present(self):
1235-
"""
1236-
If feature flag ENABLE_EDXNOTES is on, show the setting as a non-deprecated Advanced Setting.
1237-
"""
1238-
test_model = CourseMetadata.fetch(self.fullcourse)
1239-
self.assertIn('edxnotes', test_model)
1240-
1241-
@patch.dict(settings.FEATURES, {'ENABLE_EDXNOTES': False})
1242-
def test_edxnotes_not_present(self):
1243-
"""
1244-
If feature flag ENABLE_EDXNOTES is off, don't show the setting at all on the Advanced Settings page.
1245-
"""
1246-
test_model = CourseMetadata.fetch(self.fullcourse)
1247-
self.assertNotIn('edxnotes', test_model)
1248-
1249-
@patch.dict(settings.FEATURES, {'ENABLE_EDXNOTES': False})
1250-
def test_validate_update_filtered_edxnotes_off(self):
1251-
"""
1252-
If feature flag is off, then edxnotes must be filtered.
1253-
"""
1254-
# pylint: disable=unused-variable
1255-
is_valid, errors, test_model = CourseMetadata.validate_and_update_from_json(
1256-
self.course,
1257-
{
1258-
"edxnotes": {"value": "true"},
1259-
},
1260-
user=self.user
1261-
)
1262-
self.assertNotIn('edxnotes', test_model)
1263-
1264-
@patch.dict(settings.FEATURES, {'ENABLE_EDXNOTES': True})
1265-
def test_validate_update_filtered_edxnotes_on(self):
1266-
"""
1267-
If feature flag is on, then edxnotes must not be filtered.
1268-
"""
1269-
# pylint: disable=unused-variable
1270-
is_valid, errors, test_model = CourseMetadata.validate_and_update_from_json(
1271-
self.course,
1272-
{
1273-
"edxnotes": {"value": "true"},
1274-
},
1275-
user=self.user
1276-
)
1277-
self.assertIn('edxnotes', test_model)
1278-
1279-
@patch.dict(settings.FEATURES, {'ENABLE_EDXNOTES': True})
1280-
def test_update_from_json_filtered_edxnotes_on(self):
1281-
"""
1282-
If feature flag is on, then edxnotes must be updated.
1283-
"""
1284-
test_model = CourseMetadata.update_from_json(
1285-
self.course,
1286-
{
1287-
"edxnotes": {"value": "true"},
1288-
},
1289-
user=self.user
1290-
)
1291-
self.assertIn('edxnotes', test_model)
1292-
1293-
@patch.dict(settings.FEATURES, {'ENABLE_EDXNOTES': False})
1294-
def test_update_from_json_filtered_edxnotes_off(self):
1295-
"""
1296-
If feature flag is off, then edxnotes must not be updated.
1297-
"""
1298-
test_model = CourseMetadata.update_from_json(
1299-
self.course,
1300-
{
1301-
"edxnotes": {"value": "true"},
1302-
},
1303-
user=self.user
1304-
)
1305-
self.assertNotIn('edxnotes', test_model)
1306-
13071233
@patch.dict(settings.FEATURES, {'ENABLE_OTHER_COURSE_SETTINGS': True})
13081234
def test_othercoursesettings_present(self):
13091235
"""
@@ -1466,7 +1392,6 @@ def test_http_update_from_json(self):
14661392
self.assertIn('advertised_start', test_model, 'Missing revised advertised_start metadata field')
14671393
self.assertEqual(test_model['advertised_start']['value'], 'start B', "advertised_start not expected value")
14681394

1469-
@patch.dict(settings.FEATURES, {'ENABLE_EDXNOTES': True})
14701395
@patch('xmodule.util.xmodule_django.get_current_request')
14711396
def test_post_settings_with_staff_not_enrolled(self, mock_request):
14721397
"""

0 commit comments

Comments
 (0)