chore: gen cts sync tests

Author: shortcuts

generators/src/main/java/com/algolia/codegen/AlgoliaPythonGenerator.java

@@ -135,9 +135,9 @@ public OperationsMap postProcessOperationsWithModels(OperationsMap objs, List<Mo
 
     List<OperationsMap> modes = new ArrayList<>();
     modes.add(operations);
-    OperationsMap operationsAsync = (OperationsMap) operations.clone();
-    operationsAsync.put("isAsync", true);
-    modes.add(operationsAsync);
+    OperationsMap operationsSync = (OperationsMap) operations.clone();
+    operationsSync.put("isSync", true);
+    modes.add(operationsSync);
 
     additionalProperties.put("modes", modes);
 

generators/src/main/java/com/algolia/codegen/cts/tests/TestsClient.java

@@ -17,11 +17,13 @@
 public class TestsClient extends TestsGenerator {
 
   private final boolean withBenchmark;
+  private final boolean withSyncTests;
   private final String testType;
 
   public TestsClient(String language, String client, boolean withBenchmark) {
     super(language, client);
     this.withBenchmark = withBenchmark;
+    this.withSyncTests = language.equals("python");
     this.testType = withBenchmark ? "benchmark" : "client";
   }
 
@@ -208,5 +210,20 @@ public void run(Map<String, CodegenModel> models, Map<String, CodegenOperation>
       blocks.add(testObj);
     }
     bundle.put(withBenchmark ? "blocksBenchmark" : "blocksClient", blocks);
+    if (this.withSyncTests) {
+      List<Object> modes = new ArrayList<>();
+
+      Map<String, Object> sync = new HashMap<>();
+      sync.put("isSync", true);
+      sync.put(withBenchmark ? "blocksBenchmarkSync" : "blocksClientSync", blocks);
+
+      Map<String, Object> async = new HashMap<>();
+      async.put(withBenchmark ? "blocksBenchmark" : "blocksClient", blocks);
+
+      modes.add(sync);
+      modes.add(async);
+
+      bundle.put("modes", modes);
+    }
   }
 }

generators/src/main/java/com/algolia/codegen/cts/tests/TestsRequest.java

@@ -15,11 +15,13 @@
 public class TestsRequest extends TestsGenerator {
 
   private final boolean withSnippets;
+  private final boolean withSyncTests;
   private List<SupportingFile> supportingFiles;
 
   public TestsRequest(String language, String client, boolean withSnippets) {
     super(language, client);
     this.withSnippets = withSnippets;
+    this.withSyncTests = language.equals("python");
   }
 
   protected Map<String, Request[]> loadRequestCTS() throws Exception {
@@ -256,5 +258,34 @@ public void run(Map<String, CodegenModel> models, Map<String, CodegenOperation>
     } else if (supportingFiles != null) {
       supportingFiles.removeIf(f -> f.getTemplateFile().equals("tests/e2e/e2e.mustache"));
     }
+
+    if (this.withSyncTests) {
+      List<Object> modes = new ArrayList<>();
+
+      if (!blocksE2E.isEmpty()) {
+        Map<String, Object> sync = new HashMap<>();
+        sync.put("isSync", true);
+        sync.put("blocksE2ESync", blocksE2E);
+
+        Map<String, Object> async = new HashMap<>();
+        sync.put("isSync", true);
+        sync.put("blocksE2E", blocksE2E);
+
+        modes.add(sync);
+        modes.add(async);
+      }
+
+      Map<String, Object> sync = new HashMap<>();
+      sync.put("isSync", true);
+      sync.put("blocksRequestsSync", blocks);
+
+      Map<String, Object> async = new HashMap<>();
+      async.put("blocksRequests", blocks);
+
+      modes.add(sync);
+      modes.add(async);
+
+      bundle.put("modes", modes);
+    }
   }
 }

templates/python/api.mustache

@@ -17,8 +17,8 @@ from algoliasearch.{{packageName}}.models.{{#lambda.snakecase}}{{{.}}}{{/lambda.
 
 {{#modes}}
 {{#operations}}
-class {{classname}}{{^isAsync}}Sync{{/isAsync}}:
-    """The Algolia '{{classname}}{{^isAsync}}Sync{{/isAsync}}' class.
+class {{classname}}{{#isSync}}Sync{{/isSync}}:
+    """The Algolia '{{classname}}{{#isSync}}Sync{{/isSync}}' class.
 
     Args:
     app_id (str): The Algolia application ID to retrieve information from.
@@ -29,10 +29,10 @@ class {{classname}}{{^isAsync}}Sync{{/isAsync}}:
     The initialized API client.
 
     Example:
-    _client = {{classname}}{{^isAsync}}Sync{{/isAsync}}("YOUR_ALGOLIA_APP_ID", "YOUR_ALGOLIA_API_KEY"{{#hasRegionalHost}}, region="{{#allowedRegions}}'{{.}}'{{^-last}} or {{/-last}}{{/allowedRegions}}"{{/hasRegionalHost}})
-    _client_with_named_args = {{classname}}{{^isAsync}}Sync{{/isAsync}}(app_id="YOUR_ALGOLIA_APP_ID", api_key="YOUR_ALGOLIA_API_KEY"{{#hasRegionalHost}}, region="{{#allowedRegions}}'{{.}}'{{^-last}} or {{/-last}}{{/allowedRegions}}"{{/hasRegionalHost}})
+    _client = {{classname}}{{#isSync}}Sync{{/isSync}}("YOUR_ALGOLIA_APP_ID", "YOUR_ALGOLIA_API_KEY"{{#hasRegionalHost}}, region="{{#allowedRegions}}'{{.}}'{{^-last}} or {{/-last}}{{/allowedRegions}}"{{/hasRegionalHost}})
+    _client_with_named_args = {{classname}}{{#isSync}}Sync{{/isSync}}(app_id="YOUR_ALGOLIA_APP_ID", api_key="YOUR_ALGOLIA_API_KEY"{{#hasRegionalHost}}, region="{{#allowedRegions}}'{{.}}'{{^-last}} or {{/-last}}{{/allowedRegions}}"{{/hasRegionalHost}})
 
-    See `{{classname}}{{^isAsync}}Sync{{/isAsync}}.create_with_config` for advanced configuration.
+    See `{{classname}}{{#isSync}}Sync{{/isSync}}.create_with_config` for advanced configuration.
     """
 
     _transporter: Transporter
@@ -63,34 +63,34 @@ class {{classname}}{{^isAsync}}Sync{{/isAsync}}:
         The initialized API client.
 
         Example:
-        _client_with_custom_config = {{classname}}{{^isAsync}}Sync{{/isAsync}}.create_with_config(config={{#lambda.pascalcase}}{{client}}Config{{/lambda.pascalcase}}(...))
-        _client_with_custom_config_and_transporter = {{classname}}{{^isAsync}}Sync{{/isAsync}}.create_with_config(config={{#lambda.pascalcase}}{{client}}Config{{/lambda.pascalcase}}(...), transporter=Transporter(...))
+        _client_with_custom_config = {{classname}}{{#isSync}}Sync{{/isSync}}.create_with_config(config={{#lambda.pascalcase}}{{client}}Config{{/lambda.pascalcase}}(...))
+        _client_with_custom_config_and_transporter = {{classname}}{{#isSync}}Sync{{/isSync}}.create_with_config(config={{#lambda.pascalcase}}{{client}}Config{{/lambda.pascalcase}}(...), transporter=Transporter(...))
         """
         if transporter is None:
             transporter = Transporter(config)
 
-        return {{classname}}{{^isAsync}}Sync{{/isAsync}}(app_id=config.app_id, api_key=config.api_key, {{#hasRegionalHost}}region=config.region, {{/hasRegionalHost}}transporter=transporter, config=config)
+        return {{classname}}{{#isSync}}Sync{{/isSync}}(app_id=config.app_id, api_key=config.api_key, {{#hasRegionalHost}}region=config.region, {{/hasRegionalHost}}transporter=transporter, config=config)
 
-    {{#isAsync}}
+    {{^isSync}}
     async def __aenter__(self) -> None:
         return self
 
     async def __aexit__(self, exc_type, exc_value, traceback) -> None:
         """Closes the underlying `transporter` of the API client."""
-        {{#isAsync}}await {{/isAsync}}self.close()
+        {{^isSync}}await {{/isSync}}self.close()
 
     async def close(self) -> None:
         """Closes the underlying `transporter` of the API client."""
-        return {{#isAsync}}await {{/isAsync}}self._transporter.close()
-    {{/isAsync}}
+        return {{^isSync}}await {{/isSync}}self._transporter.close()
+    {{/isSync}}
 
     {{#isSearchClient}}
 {{> search_helpers}}
     {{/isSearchClient}}
 
     {{#operation}}
 
-    {{#isAsync}}async {{/isAsync}}def {{operationId}}_with_http_info{{> partial_api_args}} -> ApiResponse[str]:
+    {{^isSync}}async {{/isSync}}def {{operationId}}_with_http_info{{> partial_api_args}} -> ApiResponse[str]:
         """
         {{#isDeprecated}}
         (Deprecated) {{operationId}}
@@ -151,7 +151,7 @@ class {{classname}}{{^isAsync}}Sync{{/isAsync}}:
             _data = {{paramName}}
         {{/bodyParam}}
 
-        return {{#isAsync}}await {{/isAsync}}self._transporter.request(
+        return {{^isSync}}await {{/isSync}}self._transporter.request(
             verb=Verb.{{httpMethod}},
             path={{#vendorExtensions}}'{{{path}}}'{{#pathParams}}.replace({{=<% %>=}}'{<%baseName%>}'<%={{ }}=%>, {{#x-is-custom-request}}{{paramName}}{{/x-is-custom-request}}{{^x-is-custom-request}}quote(str({{paramName}}), safe=''){{/x-is-custom-request}}){{/pathParams}},{{/vendorExtensions}}
             request_options=self._request_options.merge(
@@ -165,7 +165,7 @@ class {{classname}}{{^isAsync}}Sync{{/isAsync}}:
             {{/vendorExtensions}}
         )
 
-    {{#isAsync}}async {{/isAsync}}def {{operationId}}{{> partial_api_args}} -> {{{returnType}}}{{^returnType}}None{{/returnType}}:
+    {{^isSync}}async {{/isSync}}def {{operationId}}{{> partial_api_args}} -> {{{returnType}}}{{^returnType}}None{{/returnType}}:
         """
         {{#isDeprecated}}
         (Deprecated) {{operationId}}
@@ -186,7 +186,7 @@ class {{classname}}{{^isAsync}}Sync{{/isAsync}}:
         :return: Returns the deserialized response in a '{{{returnType}}}' result object.
         {{/returnType}}
         """
-        return ({{#isAsync}}await {{/isAsync}}self.{{operationId}}_with_http_info({{#allParams}}{{paramName}},{{/allParams}}request_options)).deserialize({{{returnType}}})
+        return ({{^isSync}}await {{/isSync}}self.{{operationId}}_with_http_info({{#allParams}}{{paramName}},{{/allParams}}request_options)).deserialize({{{returnType}}})
 
 {{/operation}}
 {{/operations}}