Merge pull request #1308 from bjones1/testing

Misc test cleanup

Author: bnmnetp

pytest.ini

@@ -0,0 +1,7 @@
+# ****************************
+# |docname| - Configure pytest
+# ****************************
+[pytest]
+# See `Registering marks <https://docs.pytest.org/en/stable/how-to/mark.html>`_; this mark is defined in `selenium_module_fixture`.
+markers =
+    exit_status_success: Determine if the runestone build exit code is checked by ``selenium_module_fixture``.

runestone/codelens/test/test_codelens.py

@@ -4,7 +4,15 @@
 def test_Next_Step(selenium_utils_get):
     driver = selenium_utils_get.driver
     assert "CodeLens" in driver.title
-    for tdiv in ["test_codelens_1", "test_codelens_2", "test_codelens_3", "test_codelens_4", "test_codelens_5", "test_codelens_6"]:
+    for tdiv in [
+        "test_codelens_1",
+        "test_codelens_2",
+        "test_codelens_3",
+        "test_codelens_4",
+        "test_codelens_5",
+        "test_codelens_6",
+    ]:
+        print(tdiv)
         selenium_utils_get.wait_until_ready(tdiv)
         clDiv = driver.find_element_by_id(tdiv)
         assert clDiv

runestone/common/test/test_error/test_add_js_error.py

@@ -6,7 +6,7 @@
 
 
 def test_1(selenium_module_fixture):
-    assert(
-        "Extension error:\nUnable to find this_file_does_not_exist.js in html_static_path." in
-        selenium_module_fixture.build_stderr_data,
+    assert (
+        "Extension error:\nUnable to find this_file_does_not_exist.js in html_static_path."
+        in selenium_module_fixture.build_stderr_data
     )

runestone/conftest.py

@@ -68,7 +68,7 @@ def run_webpack():
 # Provide access to the Selenium module fixture, for tests with specific needs.
 @pytest.fixture(scope="module")
 def selenium_module_fixture(request):
-    # Allow modules to specify the ``exit_status_success`` parameter passed to the ``ModuleFixture``` constructor by adding the statement ``pytestmark = pytest.mark.exit_status_success(False)``. (Since this is a module-scoped fixture, applying this mark to an individual test has no effect. Marking it True instead is equivalent to the unmarked, default value.) See the `example <https://docs.pytest.org/en/stable/fixture.html#using-markers-to-pass-data-to-fixtures>`_ (which applies only to function-scoped marks, not module-scoped marks), `marking whole classes or modules <https://docs.pytest.org/en/6.2.x/example/markers.html#marking-whole-classes-or-modules>`_, and the `API docs <https://docs.pytest.org/en/stable/reference.html#pytest.nodes.Node.get_closest_marker>`_.
+    # Allow modules to specify the ``exit_status_success`` parameter passed to the ``ModuleFixture`` constructor by adding the statement ``pytestmark = pytest.mark.exit_status_success(False)``. (Since this is a module-scoped fixture, applying this mark to an individual test has no effect. Marking it True instead is equivalent to the unmarked, default value.) See the `example <https://docs.pytest.org/en/stable/fixture.html#using-markers-to-pass-data-to-fixtures>`_ (which applies only to function-scoped marks, not module-scoped marks), `marking whole classes or modules <https://docs.pytest.org/en/6.2.x/example/markers.html#marking-whole-classes-or-modules>`_, and the `API docs <https://docs.pytest.org/en/stable/reference.html#pytest.nodes.Node.get_closest_marker>`_.
     exit_status_success_mark = request.node.get_closest_marker("exit_status_success")
     exit_status_success = True if exit_status_success_mark is None else exit_status_success_mark.args[0]