This test no longer follows modern workflows. Disabling. TE-736.

Also cleaning up some lint.

lms/djangoapps/courseware/features/change_enrollment.feature

@@ -10,14 +10,15 @@ And I am logged in
 And I visit the courses page
 When I register to audit the course
 And a "edx.course.enrollment.activated" server event is emitted
+# Skipping the rest in master. The testcase has not been run and no longer works.
 #And a "edx.course.enrollment.mode_changed" server events is emitted
-
-And I visit the dashboard
-And I click on Challenge Yourself
-And I choose an honor code upgrade
-Then I should be on the dashboard page
-Then 2 "edx.course.enrollment.mode_changed" server event is emitted
-
-# don't emit another mode_changed event upon unenrollment
-When I unenroll from the course numbered "6.002x"
-Then 2 "edx.course.enrollment.mode_changed" server events is emitted
+#
+#And I visit the dashboard
+#And I click on Challenge Yourself
+#And I choose an honor code upgrade
+#Then I should be on the dashboard page
+#Then 2 "edx.course.enrollment.mode_changed" server event is emitted
+#
+## don't emit another mode_changed event upon unenrollment
+#When I unenroll from the course numbered "6.002x"
+#Then 2 "edx.course.enrollment.mode_changed" server events is emitted

lms/djangoapps/courseware/features/lti.py

@@ -16,7 +16,8 @@ from student.tests.factories import UserFactory
 
 from common import visit_scenario_item
 
-TEST_COURSE_NAME="test_course_a"
+TEST_COURSE_NAME = "test_course_a"
+
 
 @step('I view the LTI and error is shown$')
 def lti_is_not_rendered(_step):
@@ -271,10 +272,12 @@ def check_lti_popup(parent_window):
     # For verification, iterate through the window titles and make sure that
     # both are there.
     tabs = []
+    expected_tabs = [u'LTI | Test Section | {0} Courseware | edX'.format(TEST_COURSE_NAME), u'TEST TITLE']
+
     for window in windows:
         world.browser.switch_to_window(window)
         tabs.append(world.browser.title)
-    assert_equal(tabs, [u'LTI | Test Section | test_course_a Courseware | edX', u'TEST TITLE'])
+    assert_equal(tabs, expected_tabs)   # pylint: disable=no-value-for-parameter
 
     # Now verify the contents of the LTI window (which is the 2nd window/tab)
     # Note: The LTI opens in a new browser window, but Selenium sticks with the