diff --git a/cms/djangoapps/contentstore/features/pages.py b/cms/djangoapps/contentstore/features/pages.py index 43f7bacfef..bd2defc69b 100644 --- a/cms/djangoapps/contentstore/features/pages.py +++ b/cms/djangoapps/contentstore/features/pages.py @@ -98,25 +98,25 @@ def _verify_page_names(first, second): @step(u'the built-in pages are in the default order$') def built_in_pages_in_default_order(step): - expected_pages = ['Courseware', 'Course Info', 'Discussion', 'Wiki', 'Progress'] + expected_pages = ['Courseware', 'Course Info', 'Wiki', 'Progress'] see_pages_in_expected_order(expected_pages) @step(u'the built-in pages are switched$') def built_in_pages_switched(step): - expected_pages = ['Courseware', 'Course Info', 'Wiki', 'Progress', 'Discussion'] + expected_pages = ['Courseware', 'Course Info', 'Progress', 'Wiki'] see_pages_in_expected_order(expected_pages) @step(u'the pages are in the default order$') def pages_in_default_order(step): - expected_pages = ['Courseware', 'Course Info', 'Discussion', 'Wiki', 'Progress', 'First', 'Empty'] + expected_pages = ['Courseware', 'Course Info', 'Wiki', 'Progress', 'First', 'Empty'] see_pages_in_expected_order(expected_pages) @step(u'the pages are switched$$') def pages_are_switched(step): - expected_pages = ['Courseware', 'Course Info', 'Wiki', 'Progress', 'First', 'Empty', 'Discussion'] + expected_pages = ['Courseware', 'Course Info', 'Progress', 'First', 'Empty', 'Wiki'] see_pages_in_expected_order(expected_pages) diff --git a/cms/envs/acceptance.py b/cms/envs/acceptance.py index fa59fc9ad1..164785510a 100644 --- a/cms/envs/acceptance.py +++ b/cms/envs/acceptance.py @@ -98,9 +98,12 @@ STATICFILES_FINDERS += ('pipeline.finders.PipelineFinder', ) # Use the auto_auth workflow for creating users and logging them in FEATURES['AUTOMATIC_AUTH_FOR_TESTING'] = True -# For consistency in user-experience, keep the value of this setting in sync with -# the one in lms/envs/acceptance.py -FEATURES['ENABLE_DISCUSSION_SERVICE'] = True +# Forums are disabled in test.py to speed up unit tests, but we do not have +# per-test control for lettuce acceptance tests. +# If you are writing an acceptance test that needs the discussion service enabled, +# do not write it in lettuce, but instead write it using bok-choy. +# DO NOT CHANGE THIS SETTING HERE. +FEATURES['ENABLE_DISCUSSION_SERVICE'] = False # HACK # Setting this flag to false causes imports to not load correctly in the lettuce python files diff --git a/lms/envs/acceptance.py b/lms/envs/acceptance.py index 3a8eff9edb..2496b74e1b 100644 --- a/lms/envs/acceptance.py +++ b/lms/envs/acceptance.py @@ -98,10 +98,11 @@ STATICFILES_FINDERS += ('pipeline.finders.PipelineFinder', ) BULK_EMAIL_DEFAULT_FROM_EMAIL = "test@test.org" # Forums are disabled in test.py to speed up unit tests, but we do not have -# per-test control for acceptance tests -# For consistency in user-experience, keep the value of this setting in sync with -# the one in cms/envs/acceptance.py -FEATURES['ENABLE_DISCUSSION_SERVICE'] = True +# per-test control for lettuce acceptance tests. +# If you are writing an acceptance test that needs the discussion service enabled, +# do not write it in lettuce, but instead write it using bok-choy. +# DO NOT CHANGE THIS SETTING HERE. +FEATURES['ENABLE_DISCUSSION_SERVICE'] = False # Use the auto_auth workflow for creating users and logging them in FEATURES['AUTOMATIC_AUTH_FOR_TESTING'] = True