Skip to content
This repository was archived by the owner on Dec 25, 2024. It is now read-only.

Commit d678771

Browse files
authored
v2 improve endpoint params code sample (#146)
* Adds type hints for path/query/cookie/header params in endpoint code samples * Samples regen * Simplifies makefiles
1 parent 016bd0d commit d678771

31 files changed

Lines changed: 73 additions & 67 deletions

File tree

modules/openapi-json-schema-generator/src/main/resources/python/paths/path/verb/_helper_operation_doc_example.hbs

Lines changed: 10 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -10,6 +10,9 @@ from {{packageName}}.{{apiPackage}}.tags import {{moduleName}}
1010
{{/if}}
1111
{{/each}}
1212
{{/if}}
13+
{{#or pathParams queryParams headerParams cookieParams}}
14+
from {{packageName}}.paths.{{path.snakeCase}}.{{jsonPathPiece.original}} import operation
15+
{{/or}}
1316
{{#each imports}}
1417
{{{.}}}
1518
{{/each}}
@@ -31,7 +34,7 @@ with {{{packageName}}}.ApiClient(used_configuration) as api_client:
3134

3235
# example passing only required values which don't have defaults set
3336
{{#if pathParams}}
34-
path_params = {
37+
path_params: operation.RequestPathParameters.Params = {
3538
{{#each pathParams}}
3639
{{#if refInfo}}
3740
{{#if getDeepestRef.required}}
@@ -46,7 +49,7 @@ with {{{packageName}}}.ApiClient(used_configuration) as api_client:
4649
}
4750
{{/if}}
4851
{{#if queryParams}}
49-
query_params = {
52+
query_params: operation.RequestQueryParameters.Params = {
5053
{{#each queryParams}}
5154
{{#if refInfo}}
5255
{{#if getDeepestRef.required}}
@@ -61,7 +64,7 @@ with {{{packageName}}}.ApiClient(used_configuration) as api_client:
6164
}
6265
{{/if}}
6366
{{#if cookieParams}}
64-
cookie_params = {
67+
cookie_params: operation.RequestCookieParameters.Params = {
6568
{{#each cookieParams}}
6669
{{#if refInfo}}
6770
{{#if getDeepestRef.required}}
@@ -76,7 +79,7 @@ with {{{packageName}}}.ApiClient(used_configuration) as api_client:
7679
}
7780
{{/if}}
7881
{{#if headerParams}}
79-
header_params = {
82+
header_params: operation.RequestHeaderParameters.Params = {
8083
{{#each headerParams}}
8184
{{#if refInfo}}
8285
{{#if getDeepestRef.required}}
@@ -169,7 +172,7 @@ with {{{packageName}}}.ApiClient(used_configuration) as api_client:
169172
}
170173
{{/if}}
171174
{{#if queryParams}}
172-
query_params = {
175+
query_params: operation.RequestQueryParameters.Params = {
173176
{{#each queryParams}}
174177
{{#if refInfo}}
175178
'{{getDeepestRef.name}}': {{{getDeepestRef.example}}},
@@ -180,7 +183,7 @@ with {{{packageName}}}.ApiClient(used_configuration) as api_client:
180183
}
181184
{{/if}}
182185
{{#if cookieParams}}
183-
cookie_params = {
186+
cookie_params: operation.RequestCookieParameters.Params = {
184187
{{#each cookieParams}}
185188
{{#if refInfo}}
186189
'{{getDeepestRef.name}}': {{{getDeepestRef.example}}},
@@ -191,7 +194,7 @@ with {{{packageName}}}.ApiClient(used_configuration) as api_client:
191194
}
192195
{{/if}}
193196
{{#if headerParams}}
194-
header_params = {
197+
header_params: operation.RequestHeaderParameters.Params = {
195198
{{#each headerParams}}
196199
{{#if refInfo}}
197200
'{{getDeepestRef.name}}': {{{getDeepestRef.example}}},

samples/openapi3/client/3_0_3_unit_test/python/Makefile

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,10 +1,7 @@
1-
REQUIREMENTS_FILE=dev-requirements.txt
2-
REQUIREMENTS_OUT=dev-requirements.txt.log
31
SETUP_OUT=*.egg-info
42
VENV=venv
53

64
clean:
7-
rm -rf $(REQUIREMENTS_OUT)
85
rm -rf $(SETUP_OUT)
96
rm -rf $(VENV)
107
rm -rf .tox

samples/openapi3/client/features/nonCompliantUseDiscriminatorIfCompositionFails/python/Makefile

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,10 +1,7 @@
1-
REQUIREMENTS_FILE=dev-requirements.txt
2-
REQUIREMENTS_OUT=dev-requirements.txt.log
31
SETUP_OUT=*.egg-info
42
VENV=venv
53

64
clean:
7-
rm -rf $(REQUIREMENTS_OUT)
85
rm -rf $(SETUP_OUT)
96
rm -rf $(VENV)
107
rm -rf .tox

samples/openapi3/client/features/nonCompliantUseDiscriminatorIfCompositionFails/python/docs/apis/tags/DefaultApi.md

Lines changed: 0 additions & 10 deletions
This file was deleted.

samples/openapi3/client/features/security/python/Makefile

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,10 +1,7 @@
1-
REQUIREMENTS_FILE=dev-requirements.txt
2-
REQUIREMENTS_OUT=dev-requirements.txt.log
31
SETUP_OUT=*.egg-info
42
VENV=venv
53

64
clean:
7-
rm -rf $(REQUIREMENTS_OUT)
85
rm -rf $(SETUP_OUT)
96
rm -rf $(VENV)
107
rm -rf .tox

samples/openapi3/client/petstore/python/Makefile

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,10 +1,7 @@
1-
REQUIREMENTS_FILE=dev-requirements.txt
2-
REQUIREMENTS_OUT=dev-requirements.txt.log
31
SETUP_OUT=*.egg-info
42
VENV=venv
53

64
clean:
7-
rm -rf $(REQUIREMENTS_OUT)
85
rm -rf $(SETUP_OUT)
96
rm -rf $(VENV)
107
rm -rf .tox

samples/openapi3/client/petstore/python/docs/paths/fake/delete.md

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -169,6 +169,7 @@ server_index | Class | Description
169169
import petstore_api
170170
from petstore_api.configurations import api_configuration
171171
from petstore_api.apis.tags import fake_api
172+
from petstore_api.paths.fake.delete import operation
172173
from pprint import pprint
173174
# security_index 0
174175
from petstore_api.components.security_schemes import security_scheme_bearer_test
@@ -189,11 +190,11 @@ with petstore_api.ApiClient(used_configuration) as api_client:
189190
api_instance = fake_api.FakeApi(api_client)
190191

191192
# example passing only required values which don't have defaults set
192-
query_params = {
193+
query_params: operation.RequestQueryParameters.Params = {
193194
'required_string_group': "required_string_group_example",
194195
'required_int64_group': 1,
195196
}
196-
header_params = {
197+
header_params: operation.RequestHeaderParameters.Params = {
197198
'required_boolean_group': "true",
198199
}
199200
try:
@@ -207,13 +208,13 @@ with petstore_api.ApiClient(used_configuration) as api_client:
207208
print("Exception when calling FakeApi->group_parameters: %s\n" % e)
208209

209210
# example passing only optional values
210-
query_params = {
211+
query_params: operation.RequestQueryParameters.Params = {
211212
'required_string_group': "required_string_group_example",
212213
'required_int64_group': 1,
213214
'string_group': "string_group_example",
214215
'int64_group': 1,
215216
}
216-
header_params = {
217+
header_params: operation.RequestHeaderParameters.Params = {
217218
'required_boolean_group': "true",
218219
'boolean_group': "true",
219220
}

samples/openapi3/client/petstore/python/docs/paths/fake/get.md

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -229,6 +229,7 @@ server_index | Class | Description
229229
import petstore_api
230230
from petstore_api.configurations import api_configuration
231231
from petstore_api.apis.tags import fake_api
232+
from petstore_api.paths.fake.get import operation
232233
from pprint import pprint
233234
used_configuration = api_configuration.ApiConfiguration(
234235
)
@@ -238,15 +239,15 @@ with petstore_api.ApiClient(used_configuration) as api_client:
238239
api_instance = fake_api.FakeApi(api_client)
239240

240241
# example passing only optional values
241-
query_params = {
242+
query_params: operation.RequestQueryParameters.Params = {
242243
'enum_query_string_array': [
243244
"$"
244245
],
245246
'enum_query_string': "-efg",
246247
'enum_query_integer': 1,
247248
'enum_query_double': 1.1,
248249
}
249-
header_params = {
250+
header_params: operation.RequestHeaderParameters.Params = {
250251
'enum_header_string_array': [
251252
"$"
252253
],

samples/openapi3/client/petstore/python/docs/paths/fake_body_with_query_params/put.md

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -91,6 +91,7 @@ server_index | Class | Description
9191
import petstore_api
9292
from petstore_api.configurations import api_configuration
9393
from petstore_api.apis.tags import fake_api
94+
from petstore_api.paths.fake_body_with_query_params.put import operation
9495
from pprint import pprint
9596
used_configuration = api_configuration.ApiConfiguration(
9697
)
@@ -100,7 +101,7 @@ with petstore_api.ApiClient(used_configuration) as api_client:
100101
api_instance = fake_api.FakeApi(api_client)
101102

102103
# example passing only required values which don't have defaults set
103-
query_params = {
104+
query_params: operation.RequestQueryParameters.Params = {
104105
'query': "query_example",
105106
}
106107
body = user.User(

samples/openapi3/client/petstore/python/docs/paths/fake_case_sensitive_params/put.md

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -96,6 +96,7 @@ server_index | Class | Description
9696
import petstore_api
9797
from petstore_api.configurations import api_configuration
9898
from petstore_api.apis.tags import fake_api
99+
from petstore_api.paths.fake_case_sensitive_params.put import operation
99100
from pprint import pprint
100101
used_configuration = api_configuration.ApiConfiguration(
101102
)
@@ -105,7 +106,7 @@ with petstore_api.ApiClient(used_configuration) as api_client:
105106
api_instance = fake_api.FakeApi(api_client)
106107

107108
# example passing only required values which don't have defaults set
108-
query_params = {
109+
query_params: operation.RequestQueryParameters.Params = {
109110
'someVar': "someVar_example",
110111
'SomeVar': "SomeVar_example",
111112
'some_var': "some_var_example",

0 commit comments

Comments
 (0)