fix: fix marshmallow warning about using kwargs arguments to declare fields

Turns out this was a typo and MultiScanResultSchema did not validate that
scan_results was not empty, causing one test to fail. Updated the test.

Author: agateau-gg

pygitguardian/models.py

@@ -324,7 +324,7 @@ class MultiScanResultSchema(BaseSchema):
     scan_results = fields.List(
         fields.Nested(ScanResultSchema),
         required=True,
-        validates=validate.Length(min=1),
+        validate=validate.Length(min=1),
     )
 
     @post_load

tests/test_models.py

@@ -45,7 +45,27 @@ def test_document_model(self):
             (
                 MultiScanResultSchema,
                 MultiScanResult,
-                {"scan_results": [], "type": "hello"},
+                {
+                    "scan_results": [
+                        {
+                            "policy_break_count": 1,
+                            "policies": ["pol"],
+                            "policy_breaks": [
+                                {
+                                    "type": "break",
+                                    "policy": "mypol",
+                                    "matches": [
+                                        {
+                                            "match": "hello",
+                                            "type": "hello",
+                                        }
+                                    ],
+                                }
+                            ],
+                        }
+                    ],
+                    "type": "hello",
+                },
             ),
             (
                 PolicyBreakSchema,