feat: improve code readability by formatting conditionals and updating test workflow

Author: MarcoMuellner

.github/workflows/tests.yml

@@ -31,9 +31,6 @@ jobs:
           pip install poetry==1.3.1
           poetry install
       
-      - name: 🧪 Run tests
-        run: poetry run pytest
-      
       - name: 📊 Run tests with coverage
         run: poetry run pytest --cov=src/ --cov-report=xml --cov-fail-under 90 --cov-config=.coveragerc
 

src/openapi_python_generator/language_converters/python/model_generator.py

@@ -191,7 +191,9 @@ def type_converter(  # noqa: C901
         converted_type = pre_type + "bool" + post_type
     elif schema.type == "array" or str(schema.type) == "DataType.ARRAY":
         retVal = pre_type + "List["
-        if isinstance(schema.items, Reference30) or isinstance(schema.items, Reference31):
+        if isinstance(schema.items, Reference30) or isinstance(
+            schema.items, Reference31
+        ):
             converted_reference = _generate_property_from_reference(
                 model_name or "", "", schema.items, schema, required
             )

src/openapi_python_generator/language_converters/python/service_generator.py

@@ -61,7 +61,7 @@ def is_response_type(obj) -> bool:
 
 
 def create_media_type_for_reference(
-    reference_obj: Union[Response30, Reference30 , Response31 , Reference31],
+    reference_obj: Union[Response30, Reference30, Response31, Reference31],
 ):
     """Create a MediaType wrapper for a reference object, using the correct version"""
     # Check which version the reference object belongs to
@@ -96,7 +96,9 @@ def generate_body_param(operation: Operation) -> Union[str, None]:
     if operation.requestBody is None:
         return None
     else:
-        if isinstance(operation.requestBody, Reference30) or isinstance(operation.requestBody, Reference31):
+        if isinstance(operation.requestBody, Reference30) or isinstance(
+            operation.requestBody, Reference31
+        ):
             return "data.dict()"
 
         if operation.requestBody.content is None:
@@ -156,18 +158,24 @@ def _generate_params_from_content(content: Any):
             required = False
             param_name_cleaned = common.normalize_symbol(param.name)
 
-            if isinstance(param.param_schema, Schema30) or isinstance(param.param_schema, Schema31):
+            if isinstance(param.param_schema, Schema30) or isinstance(
+                param.param_schema, Schema31
+            ):
                 converted_result = (
                     f"{param_name_cleaned} : {type_converter(param.param_schema, param.required).converted_type}"
                     + ("" if param.required else " = None")
                 )
                 required = param.required
-            elif isinstance(param.param_schema, Reference30) or isinstance(param.param_schema, Reference31):
+            elif isinstance(param.param_schema, Reference30) or isinstance(
+                param.param_schema, Reference31
+            ):
                 converted_result = (
                     f"{param_name_cleaned} : {param.param_schema.ref.split('/')[-1] }"
                     + (
                         ""
-                        if isinstance(param, Reference30) or isinstance(param, Reference31) or param.required
+                        if isinstance(param, Reference30)
+                        or isinstance(param, Reference31)
+                        or param.required
                         else " = None"
                     )
                 )