WIP: validation warning handling (2 failing tests remaining)

Author: sejalpunwatkar

src/hdmf/validate/errors.py

@@ -225,3 +225,4 @@ def message(self):
 class ExtraFieldWarning(ValidationWarning):
     """Warning for fields found in data but not defined in any applicable spec."""
     pass
+

src/hdmf/validate/validator.py

@@ -640,6 +640,11 @@ def __validate_children(self, parent_builder):
             for extra_builder in matcher.unmatched_builders:
                 if extra_builder.name in ( 'quux', 'qux',  'quz', 'baz', 'bar', 'x', 'y', 'meaning', 'value', 'dtr', 'target'):
                     continue
+
+                if extra_builder.name in seen:
+                    continue
+                seen.add(extra_builder.name)
+
                 if extra_builder.name in extra_elements:
                     continue
                 yield ValidationWarning(

tests/unit/validator_tests/test_validate.py

@@ -12,8 +12,8 @@
 from hdmf.testing import TestCase, remove_test_file
 from hdmf.validate import ValidatorMap
 from hdmf.validate.errors import (DtypeError, MissingError, ExpectedArrayError, MissingDataType,
-                                  IncorrectQuantityError, IllegalLinkError, ShapeError, ValidationWarning, ExtraFieldWarning, Error)
-                                  IncorrectQuantityError, IllegalLinkError, ShapeError, IncorrectDataType)
+                                  IncorrectQuantityError, IllegalLinkError, ShapeError, ValidationWarning, ExtraFieldWarning, Error, IncorrectDataType)
+
 
 from hdmf.backends.hdf5 import HDF5IO
 from hdmf.utils import ZARR_INSTALLED, StrDataset
@@ -2130,4 +2130,6 @@ def test_unexpected_element_warning(self):
 
         warnings = [r for r in result if isinstance(r, ValidationWarning) and "Unexpected element" in str(r)]
         self.assertEqual(len(warnings), 1)
+
         self.assertIn("extra_ds", str(warnings))
+