Updating docstring for a test and removing a test that will unlikely be used in the future.

Author: kmacdonald-stsci

jwst/regtest/test_infrastructure.py

@@ -85,18 +85,6 @@ def test_diff_astropy_tables_columns(diff_astropy_tables, two_tables):
         assert diff_astropy_tables(path1, path2)
 
 
-@pytest.mark.xfail(reason="table meta comparison currently deactivated")
-def test_diff_astropy_tables_meta(diff_astropy_tables, two_tables):
-    path1, path2 = two_tables
-
-    t1 = Table.read(path1)
-    t1.meta = {"name": "not the first table"}
-    t1.write(path1, overwrite=True, format="ascii.ecsv")
-
-    with pytest.raises(AssertionError, match="Metadata does not match"):
-        assert diff_astropy_tables(path1, path2)
-
-
 def test_diff_astropy_tables_allclose(diff_astropy_tables, two_tables):
     path1, path2 = two_tables
 

jwst/tests/test_helpers.py

@@ -17,11 +17,10 @@ def test_warning_inside_capture_logging():
 
 def test_unexpected_warning_inside_capture_logging():
     """
-    This should fail (see xfail) since the warning doesn't match.
-    We're using an xfail here because we're testing that pytest.warns
-    fails to catch a warnings.
+    This test catches the exception raised due to warnings not matching.
+    We're testing that pytest.warns fails to catch a warnings.
     """
-    with pytest.raises(UserWarning):
+    with pytest.raises(UserWarning, match="not the warning"):
         with _help_pytest_warns(), pytest.warns(UserWarning, match="expected"):
             warnings.warn("not the warning you're looking for", UserWarning)