Skip to content

Commit ba1193a

Browse files
keepConcentrationGoopher Maijenburg
keepConcentration
authored and
Goopher Maijenburg
committed
[OpenAPITools#21619] fix(python): update pyproject.toml license format with poetry1 fallback (OpenAPITools#21931)
* Add support for both string and object license formats in `pyproject.toml` for Python and Python-Pydantic generators. (OpenAPITools#21619) * Generate the samples (OpenAPITools#21698) * Generate the samples (OpenAPITools#21698) * Remove Python-Pydantic-v1 license format tests (OpenAPITools#21619) * Revert license format to string in Python-Pydantic-v1 samples (OpenAPITools#21619)
1 parent 9d69407 commit ba1193a

6 files changed

Lines changed: 113 additions & 3 deletions

File tree

‎modules/openapi-generator/src/main/resources/python/pyproject.mustache‎

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,12 @@ authors = [
1616
]
1717
{{/poetry1}}
1818
{{#licenseInfo}}
19+
{{#poetry1}}
1920
license = "{{{licenseInfo}}}"
21+
{{/poetry1}}
22+
{{^poetry1}}
23+
license = { text = "{{{licenseInfo}}}" }
24+
{{/poetry1}}
2025
{{/licenseInfo}}
2126
readme = "README.md"
2227
{{#poetry1}}

‎modules/openapi-generator/src/test/java/org/openapitools/codegen/python/PythonClientCodegenTest.java‎

Lines changed: 74 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -25,6 +25,7 @@
2525
import io.swagger.v3.parser.core.models.ParseOptions;
2626
import io.swagger.v3.parser.util.SchemaTypeUtil;
2727
import org.openapitools.codegen.*;
28+
import org.openapitools.codegen.config.CodegenConfigurator;
2829
import org.openapitools.codegen.languages.PythonClientCodegen;
2930
import org.openapitools.codegen.languages.features.CXFServerFeatures;
3031
import org.testng.Assert;
@@ -611,4 +612,77 @@ public void testInitFileImportsExports() throws IOException {
611612
assertFileContains(initFilePath, "from openapi_client.models.tag import Tag as Tag");
612613
assertFileContains(initFilePath, "from openapi_client.models.user import User as User");
613614
}
615+
616+
@Test(description = "Verify default license format uses object notation when poetry1 is false")
617+
public void testLicenseFormatInPyprojectToml() throws IOException {
618+
File output = Files.createTempDirectory("test").toFile().getCanonicalFile();
619+
output.deleteOnExit();
620+
621+
final CodegenConfigurator configurator = new CodegenConfigurator()
622+
.setGeneratorName("python")
623+
.setInputSpec("src/test/resources/bugs/issue_21619.yaml")
624+
.setOutputDir(output.getAbsolutePath())
625+
.addAdditionalProperty("licenseInfo", "MIT");
626+
627+
DefaultGenerator generator = new DefaultGenerator();
628+
List<File> files = generator.opts(configurator.toClientOptInput()).generate();
629+
files.forEach(File::deleteOnExit);
630+
631+
TestUtils.assertFileExists(Paths.get(output.getAbsolutePath(), "pyproject.toml"));
632+
// When poetry1=false (default), license should use object notation: { text = "MIT" }
633+
TestUtils.assertFileContains(Paths.get(output.getAbsolutePath(), "pyproject.toml"),
634+
"license = { text = \"MIT\" }");
635+
}
636+
637+
@Test(description = "Verify poetry1 mode uses string notation for license")
638+
public void testPoetry1LicenseFormat() throws IOException {
639+
File output = Files.createTempDirectory("test").toFile().getCanonicalFile();
640+
output.deleteOnExit();
641+
642+
final CodegenConfigurator configurator = new CodegenConfigurator()
643+
.setGeneratorName("python")
644+
.setInputSpec("src/test/resources/bugs/issue_21619.yaml")
645+
.setOutputDir(output.getAbsolutePath())
646+
.addAdditionalProperty("licenseInfo", "Apache-2.0")
647+
.addAdditionalProperty("poetry1", true); // Enable legacy poetry1 mode
648+
649+
DefaultGenerator generator = new DefaultGenerator();
650+
List<File> files = generator.opts(configurator.toClientOptInput()).generate();
651+
files.forEach(File::deleteOnExit);
652+
653+
Path pyprojectPath = Paths.get(output.getAbsolutePath(), "pyproject.toml");
654+
TestUtils.assertFileExists(pyprojectPath);
655+
656+
// In poetry1 mode, license should use simple string format: "Apache-2.0"
657+
TestUtils.assertFileContains(pyprojectPath, "license = \"Apache-2.0\"");
658+
659+
// Verify it does NOT use the new object format
660+
TestUtils.assertFileNotContains(pyprojectPath, "license = { text = \"Apache-2.0\" }");
661+
}
662+
663+
@Test(description = "Verify non-poetry1 mode uses object notation for license")
664+
public void testNonPoetry1LicenseFormat() throws IOException {
665+
File output = Files.createTempDirectory("test").toFile().getCanonicalFile();
666+
output.deleteOnExit();
667+
668+
final CodegenConfigurator configurator = new CodegenConfigurator()
669+
.setGeneratorName("python")
670+
.setInputSpec("src/test/resources/bugs/issue_21619.yaml")
671+
.setOutputDir(output.getAbsolutePath())
672+
.addAdditionalProperty("licenseInfo", "BSD-3-Clause")
673+
.addAdditionalProperty("poetry1", false); // Explicitly disable poetry1 mode
674+
675+
DefaultGenerator generator = new DefaultGenerator();
676+
List<File> files = generator.opts(configurator.toClientOptInput()).generate();
677+
files.forEach(File::deleteOnExit);
678+
679+
Path pyprojectPath = Paths.get(output.getAbsolutePath(), "pyproject.toml");
680+
TestUtils.assertFileExists(pyprojectPath);
681+
682+
// In non-poetry1 mode, license should use object format: { text = "BSD-3-Clause" }
683+
TestUtils.assertFileContains(pyprojectPath, "license = { text = \"BSD-3-Clause\" }");
684+
685+
// Verify it does NOT use the legacy string format
686+
TestUtils.assertFileNotContains(pyprojectPath, "license = \"BSD-3-Clause\"");
687+
}
614688
}

‎modules/openapi-generator/src/test/resources/bugs/issue_21619.yaml‎

Lines changed: 31 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,31 @@
1+
openapi: 3.0.0
2+
info:
3+
title: License Format Test API
4+
version: 1.0.0
5+
description: Simple API for testing pyproject.toml license format changes
6+
paths:
7+
/test:
8+
get:
9+
operationId: getTest
10+
summary: Simple test endpoint
11+
responses:
12+
'200':
13+
description: Success
14+
content:
15+
application/json:
16+
schema:
17+
type: object
18+
properties:
19+
message:
20+
type: string
21+
example: "Hello World"
22+
components:
23+
schemas:
24+
TestModel:
25+
type: object
26+
properties:
27+
id:
28+
type: integer
29+
format: int64
30+
name:
31+
type: string

‎samples/client/echo_api/python-disallowAdditionalPropertiesIfNotPresent/pyproject.toml‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@ description = "Echo Server API"
55
authors = [
66
{name = "OpenAPI Generator Community",email = "team@openapitools.org"},
77
]
8-
license = "Apache 2.0"
8+
license = { text = "Apache 2.0" }
99
readme = "README.md"
1010
keywords = ["OpenAPI", "OpenAPI-Generator", "Echo Server API"]
1111
requires-python = ">=3.9"

‎samples/client/echo_api/python/pyproject.toml‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@ description = "Echo Server API"
55
authors = [
66
{name = "OpenAPI Generator Community",email = "team@openapitools.org"},
77
]
8-
license = "Apache 2.0"
8+
license = { text = "Apache 2.0" }
99
readme = "README.md"
1010
keywords = ["OpenAPI", "OpenAPI-Generator", "Echo Server API"]
1111
requires-python = ">=3.9"

‎samples/openapi3/client/petstore/python/pyproject.toml‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@ description = "OpenAPI Petstore"
55
authors = [
66
{name = "OpenAPI Generator Community",email = "team@openapitools.org"},
77
]
8-
license = "Apache-2.0"
8+
license = { text = "Apache-2.0" }
99
readme = "README.md"
1010
keywords = ["OpenAPI", "OpenAPI-Generator", "OpenAPI Petstore"]
1111
requires-python = ">=3.9"

0 commit comments

Comments
 (0)