new test correction

tests/dataconverter/test_validation.py

@@ -105,24 +105,21 @@ def test_valid_data_dict(caplog, data_dict):
 
 
 @pytest.mark.parametrize(
-    "data_dict",
+    "data_dict, error_message",
     [
-        pytest.param(get_data_dict(), id="valid-unaltered-data-dict"),
         pytest.param(
             remove_from_dict(
                 "/ENTRY[my_entry]/NXODD_name[nxodd_name]/float_value", get_data_dict()
             ),
-            id="removed-optional-value",
+            "There were attributes set for the field /ENTRY[my_entry]/NXODD_name[nxodd_name]/float_value, but the field does not exist.",
+            id="removed-optional-value-with-attribute-remaining",
         ),
     ],
 )
-def test_data_dict_attr_with_no_field(caplog, data_dict):
+def test_data_dict_attr_with_no_field(caplog, data_dict, error_message):
     with caplog.at_level(logging.WARNING):
-        validate_dict_against("NXtest", data_dict)[0]
-    assert (
-        caplog.text
-        == "There were attributes set for the field /ENTRY[my_entry]/NXODD_name[nxodd_name]/float_value, but the field does not exist."
-    )
+        assert not validate_dict_against("NXtest", data_dict)[0]
+    assert error_message in caplog.text
 
 
 @pytest.mark.parametrize(