Fix easy to fix python issues

AIDASoft · Dec 18, 2023 · 727ad6c · 727ad6c
1 parent 2cf88ea
commit 727ad6c
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 10 deletions.
diff --git a/python/podio_gen/generator_utils.py b/python/podio_gen/generator_utils.py
@@ -120,6 +120,7 @@ def _to_json(self):
     """Return a string representation that can be parsed again"""
     return self.full_type
 
+
 class MemberVariable:
   """Simple class to hold information about a member variable"""
   def __init__(self, name, **kwargs):

diff --git a/python/podio_gen/test_ClassDefinitionValidator.py b/python/podio_gen/test_ClassDefinitionValidator.py
@@ -446,16 +446,16 @@ def test_datatype_uses_interface_type(self):
                               self.validate, make_dm({}, datatype, self.valid_interface), False)
 
   def test_interface_valid_upstream(self):
-   """Make sure that we can use interface definitions from upstream models"""
-   # Create an upstream datamodel that contains the interface type
-   upstream_dm = make_dm({}, self.valid_datatype, self.valid_interface)
-
-   # Make a downstream model datatype that uses the interface from the upstream
-   # but doesn't bring along its own interface definitions
-   datatype = {'DownstreamType': deepcopy(self.valid_datatype['DataType'])}
-   datatype['DownstreamType']['OneToOneRelations'] = [MemberVariable(type='InterfaceType', name='interfaceRelation')]
-   self._assert_no_exception(DefinitionError, '{} should allow to use interface types from an upstream datamodel',
-                             self.validate, make_dm({}, datatype), upstream_dm)
+    """Make sure that we can use interface definitions from upstream models"""
+    # Create an upstream datamodel that contains the interface type
+    upstream_dm = make_dm({}, self.valid_datatype, self.valid_interface)
+
+    # Make a downstream model datatype that uses the interface from the upstream
+    # but doesn't bring along its own interface definitions
+    datatype = {'DownstreamType': deepcopy(self.valid_datatype['DataType'])}
+    datatype['DownstreamType']['OneToOneRelations'] = [MemberVariable(type='InterfaceType', name='interfaceRelation')]
+    self._assert_no_exception(DefinitionError, '{} should allow to use interface types from an upstream datamodel',
+                              self.validate, make_dm({}, datatype), upstream_dm)
 
 
 if __name__ == '__main__':