fixed default source folder and empty lines in templates

Author: AdityaMayukhSom

docs/generators/python-fastapi.md

@@ -32,7 +32,7 @@ These options may be applied as additional-properties (cli) or configOptions (pl
 |serverPort|TCP port to listen to in app.run| |8080|
 |sortModelPropertiesByRequiredFlag|Sort model properties to place required parameters before optional parameters.| |true|
 |sortParamsByRequiredFlag|Sort method arguments to place required parameters before optional parameters.| |true|
-|sourceFolder|directory for generated python source code| |generated-code/python-fastapi|
+|sourceFolder|directory for generated python source code| |src|
 
 ## IMPORT MAPPING
 

modules/openapi-generator/src/main/java/org/openapitools/codegen/languages/PythonFastAPIServerCodegen.java

@@ -155,7 +155,7 @@ public PythonFastAPIServerCodegen() {
 
         addOption(CodegenConstants.SOURCE_FOLDER,
                 "directory for generated python source code",
-                outputFolder);
+                DEFAULT_SOURCE_FOLDER);
 
         addOption(CodegenConstants.FASTAPI_IMPLEMENTATION_PACKAGE,
                 "python package name for the implementation code (convention: snake_case).",

modules/openapi-generator/src/main/resources/python-fastapi/api.mustache

@@ -5,11 +5,9 @@ import importlib
 import pkgutil
 
 from {{apiPackage}}.{{classFilename}}_{{baseSuffix}} import Base{{classname}}
-{{^isLibrary}}
-{{#fastapiImplementationPackage}}
+{{^isLibrary}}{{#fastapiImplementationPackage}}
 import {{fastapiImplementationPackage}}
-{{/fastapiImplementationPackage}}
-{{/isLibrary}}
+{{/fastapiImplementationPackage}}{{/isLibrary}}
 
 
 from fastapi import (  # noqa: F401

modules/openapi-generator/src/main/resources/python-fastapi/base_api.mustache

@@ -18,8 +18,8 @@ class Base{{classname}}{{#isLibrary}}(ABC){{/isLibrary}}:
         Base{{classname}}.subclasses = Base{{classname}}.subclasses + (cls,)
 
 {{#operations}}
-{{#operation}}
-    {{#isLibrary}}@abstractmethod{{/isLibrary}}
+{{#operation}}{{#isLibrary}}
+    @abstractmethod{{/isLibrary}}
     async def {{operationId}}(
         self,
         {{#allParams}}
@@ -30,7 +30,7 @@ class Base{{classname}}{{#isLibrary}}(ABC){{/isLibrary}}:
         ...{{/notes}}{{^notes}}...{{/notes}}
 {{^-last}}
 
-
 {{/-last}}
 {{/operation}}
 {{/operations}}
+

samples/server/petstore/python-fastapi/src/openapi_server/apis/fake_api.py

@@ -5,9 +5,11 @@
 import pkgutil
 
 from openapi_server.apis.fake_api_base import BaseFakeApi
+
 import openapi_server.impl
 
 
+
 from fastapi import (  # noqa: F401
     APIRouter,
     Body,

samples/server/petstore/python-fastapi/src/openapi_server/apis/fake_api_base.py

@@ -14,11 +14,12 @@ def __init_subclass__(cls, **kwargs):
         super().__init_subclass__(**kwargs)
         BaseFakeApi.subclasses = BaseFakeApi.subclasses + (cls,)
 
-    
+
     async def fake_query_param_default(
         self,
         has_default: Annotated[Optional[StrictStr], Field(description="has default value")],
         no_default: Annotated[Optional[StrictStr], Field(description="no default value")],
     ) -> None:
         """"""
         ...
+

samples/server/petstore/python-fastapi/src/openapi_server/apis/pet_api.py

@@ -5,9 +5,11 @@
 import pkgutil
 
 from openapi_server.apis.pet_api_base import BasePetApi
+
 import openapi_server.impl
 
 
+
 from fastapi import (  # noqa: F401
     APIRouter,
     Body,

samples/server/petstore/python-fastapi/src/openapi_server/apis/pet_api_base.py

@@ -16,7 +16,7 @@ def __init_subclass__(cls, **kwargs):
         super().__init_subclass__(**kwargs)
         BasePetApi.subclasses = BasePetApi.subclasses + (cls,)
 
-    
+
     async def add_pet(
         self,
         pet: Annotated[Pet, Field(description="Pet object that needs to be added to the store")],
@@ -25,7 +25,6 @@ async def add_pet(
         ...
 
 
-    
     async def delete_pet(
         self,
         petId: Annotated[StrictInt, Field(description="Pet id to delete")],
@@ -35,7 +34,6 @@ async def delete_pet(
         ...
 
 
-    
     async def find_pets_by_status(
         self,
         status: Annotated[List[StrictStr], Field(description="Status values that need to be considered for filter")],
@@ -44,7 +42,6 @@ async def find_pets_by_status(
         ...
 
 
-    
     async def find_pets_by_tags(
         self,
         tags: Annotated[List[StrictStr], Field(description="Tags to filter by")],
@@ -53,7 +50,6 @@ async def find_pets_by_tags(
         ...
 
 
-    
     async def get_pet_by_id(
         self,
         petId: Annotated[StrictInt, Field(description="ID of pet to return")],
@@ -62,7 +58,6 @@ async def get_pet_by_id(
         ...
 
 
-    
     async def update_pet(
         self,
         pet: Annotated[Pet, Field(description="Pet object that needs to be added to the store")],
@@ -71,7 +66,6 @@ async def update_pet(
         ...
 
 
-    
     async def update_pet_with_form(
         self,
         petId: Annotated[StrictInt, Field(description="ID of pet that needs to be updated")],
@@ -82,7 +76,6 @@ async def update_pet_with_form(
         ...
 
 
-    
     async def upload_file(
         self,
         petId: Annotated[StrictInt, Field(description="ID of pet to update")],
@@ -91,3 +84,4 @@ async def upload_file(
     ) -> ApiResponse:
         """"""
         ...
+

samples/server/petstore/python-fastapi/src/openapi_server/apis/store_api.py

@@ -5,9 +5,11 @@
 import pkgutil
 
 from openapi_server.apis.store_api_base import BaseStoreApi
+
 import openapi_server.impl
 
 
+
 from fastapi import (  # noqa: F401
     APIRouter,
     Body,

samples/server/petstore/python-fastapi/src/openapi_server/apis/store_api_base.py

@@ -15,7 +15,7 @@ def __init_subclass__(cls, **kwargs):
         super().__init_subclass__(**kwargs)
         BaseStoreApi.subclasses = BaseStoreApi.subclasses + (cls,)
 
-    
+
     async def delete_order(
         self,
         orderId: Annotated[StrictStr, Field(description="ID of the order that needs to be deleted")],
@@ -24,15 +24,13 @@ async def delete_order(
         ...
 
 
-    
     async def get_inventory(
         self,
     ) -> Dict[str, int]:
         """Returns a map of status codes to quantities"""
         ...
 
 
-    
     async def get_order_by_id(
         self,
         orderId: Annotated[int, Field(le=5, strict=True, ge=1, description="ID of pet that needs to be fetched")],
@@ -41,10 +39,10 @@ async def get_order_by_id(
         ...
 
 
-    
     async def place_order(
         self,
         order: Annotated[Order, Field(description="order placed for purchasing the pet")],
     ) -> Order:
         """"""
         ...
+