default errors from ledger.get_errors() to be unique (#8)

* unqify errors with legdger.get_errors()

* upgrade version

* opt out of unique errrors

Author: acmisiti

.gitignore

@@ -23,6 +23,7 @@ var/
 *.egg-info/
 .installed.cfg
 *.egg
+.venv/
 
 # PyInstaller
 #  Usually these files are written by a python script from a template

pylidator/error_ledger.py

@@ -94,8 +94,13 @@ def get_descriptive_error_messages(self):
     def get_warning_messages(self):
         return list(unique_everseen([e["message"] for e in self._errors if e["level"] == e.WARN]))
 
-    def get_errors(self):
-        return [e for e in self._errors if e["level"] == e.ERROR]
+    def get_errors(self, unique=True):
+        errors = [e for e in self._errors if e["level"] == e.ERROR]
+        if unique:
+            return ensure_unique_error_list(errors)
+        else:
+            return errors
+
 
     def get_warnings(self):
         return [e for e in self._errors if e["level"] == e.WARN]
@@ -146,3 +151,17 @@ def unique_everseen(iterable, key=None):
             if k not in seen:
                 seen_add(k)
                 yield element
+
+def ensure_unique_error_list(dict_list):
+    # Use a set to track unique frozensets of dictionary items
+    seen = set()
+    unique_dicts = []
+    
+    for d in dict_list:
+        # Convert dictionary items to a frozenset
+        items = frozenset(d.items())
+        if items not in seen:
+            seen.add(items)
+            unique_dicts.append(d)
+    
+    return unique_dicts

pylidator/tests/pylidator_test.py

@@ -136,3 +136,27 @@ def test_validator_requesting_unavailable_context_throws(self):
                 providers=_providers,
                 extra_context={"not_constants_service": cs},
             )
+
+    def test_validator_returns_array_of_strings_results_in_errors_that_are_unique(self):
+        data = TestObj(returns=["error one", "error one"])
+        ret = pylidator.validate(data, {pylidator.ERROR: [validate_parent]}, providers=_providers)
+        self.assertEqual(
+            [
+                {'validation_type': None, 'message': 'error one', 'level': 'ERROR'},
+                {'validation_type': None, 'message': 'error one', 'level': 'ERROR'}
+            ],
+            ret.get_full_results(),
+        )
+        self.assertEqual(
+            [
+                {'validation_type': None, 'message': 'error one', 'level': 'ERROR'},
+            ],
+            ret.get_errors(),
+        )
+        self.assertEqual(
+            [
+                {'validation_type': None, 'message': 'error one', 'level': 'ERROR'},
+                {'validation_type': None, 'message': 'error one', 'level': 'ERROR'}
+            ],
+            ret.get_errors(unique=False),
+        )

setup.py

@@ -10,7 +10,7 @@
     # Versions should comply with PEP440.  For a discussion on single-sourcing
     # the version across setup.py and the project code, see
     # https://packaging.python.org/en/latest/single_source_version.html
-    version="1.1.5",
+    version="1.1.6",
     description="A practical system for organizing validation rules.",
     long_description=long_description,
     long_description_content_type="text/markdown",