chore(csharp): test the support of multiple dotnet versions

Author: millotp

config/clients.config.json

@@ -27,7 +27,12 @@
     "snippets": {
       "extension": ".cs",
       "outputFolder": "src"
-    }
+    },
+    "supportedVersions": [
+      "8.0.416",
+      "9.0.307",
+      "10.0.100"
+    ]
   },
   "dart": {
     "clients": [

config/clients.schema.json

@@ -93,7 +93,7 @@
       },
       "supportedVersions": {
         "type": "array",
-        "description": "hints the CI on what matrix to generate for this client, this must be language specific versions, e.g. versions of node",
+        "description": "hints the CI on what matrix to generate for this client, this must be language specific versions, e.g. versions of node. The last item is the version that will be released",
         "items": { "type": "string" }
       }
     },

config/generation.config.mjs

@@ -26,6 +26,7 @@ export const patterns = [
   '!clients/algoliasearch-client-csharp/algoliasearch/Models/Common/**',
 
   'tests/output/csharp/global.json',
+  'tests/output/csharp/src/Algolia.Search.Tests.csproj',
 
   // Dart
   '!clients/algoliasearch-client-dart/**',

generators/src/main/java/com/algolia/codegen/cts/AlgoliaCTSGenerator.java

@@ -44,7 +44,7 @@ public void processOpts() {
     language = (String) additionalProperties.get("language");
     client = (String) additionalProperties.get("client");
     mode = (String) additionalProperties.get("mode");
-    ctsManager = CTSManagerFactory.getManager(language, client);
+    ctsManager = CTSManagerFactory.getManager(language, client, (String) additionalProperties.getOrDefault("languageVersion", ""));
 
     if (ctsManager == null) {
       // skip the generation
@@ -162,6 +162,9 @@ public Map<String, Object> postProcessSupportingFileData(Map<String, Object> obj
       ctsManager.addMustacheLambdas(lambda);
       bundle.put("lambda", lambda);
 
+      // Set some default values in the bundle
+      bundle.put("packageVersion", ctsManager.getPackageVersion());
+
       String languageVersion = ctsManager.getLanguageVersion((String) additionalProperties.getOrDefault("languageVersion", ""));
       bundle.put("languageVersion", languageVersion);
 

generators/src/main/java/com/algolia/codegen/cts/manager/CSharpCTSManager.java

@@ -4,15 +4,18 @@
 import com.algolia.codegen.exceptions.GeneratorException;
 import com.algolia.codegen.utils.*;
 import com.samskivert.mustache.Mustache.Lambda;
+import java.io.IOException;
 import java.util.*;
 import org.openapitools.codegen.SupportingFile;
 
 public class CSharpCTSManager implements CTSManager {
 
   private final String client;
+  private final String overrideLanguageVersion;
 
-  public CSharpCTSManager(String client) {
+  public CSharpCTSManager(String client, String overrideLanguageVersion) {
     this.client = client;
+    this.overrideLanguageVersion = overrideLanguageVersion;
   }
 
   public String getLanguage() {
@@ -26,6 +29,9 @@ public String getClient() {
   @Override
   public void addTestsSupportingFiles(List<SupportingFile> supportingFiles) {
     supportingFiles.add(new SupportingFile("globaljson.mustache", "tests/output/csharp", "global.json"));
+    supportingFiles.add(
+      new SupportingFile("tests/Algolia.Search.Tests.csproj.mustache", "tests/output/csharp/src", "Algolia.Search.Tests.csproj")
+    );
   }
 
   @Override
@@ -43,7 +49,13 @@ public void addSnippetsSupportingFiles(List<SupportingFile> supportingFiles, Str
 
   @Override
   public void addDataToBundle(Map<String, Object> bundle) throws GeneratorException {
-    bundle.put("packageVersion", getVersion());
+    try {
+      String dotnetVersion = getLanguageVersion(overrideLanguageVersion);
+      String[] parts = dotnetVersion.split("\\.");
+      bundle.put("dotnetVersion", parts[0] + "." + parts[1]);
+    } catch (IOException e) {
+      throw new GeneratorException("Failed to compute dotnet version", e);
+    }
   }
 
   @Override

generators/src/main/java/com/algolia/codegen/cts/manager/CTSManager.java

@@ -16,7 +16,7 @@ public default String getLanguageCased() {
     return Helpers.capitalize(getLanguage());
   }
 
-  public default String getVersion() {
+  public default String getPackageVersion() {
     return Helpers.getClientConfigField(getLanguage(), "packageVersion");
   }
 

generators/src/main/java/com/algolia/codegen/cts/manager/CTSManagerFactory.java

@@ -6,7 +6,7 @@ private CTSManagerFactory() {
     // NO-OP
   }
 
-  public static CTSManager getManager(String language, String client) {
+  public static CTSManager getManager(String language, String client, String overrideLanguageVersion) {
     return switch (language) {
       case "javascript" -> new JavascriptCTSManager(client);
       case "java" -> new JavaCTSManager(client);
@@ -17,7 +17,7 @@ public static CTSManager getManager(String language, String client) {
       case "ruby" -> new RubyCTSManager(client);
       case "scala" -> new ScalaCTSManager(client);
       case "python" -> new PythonCTSManager(client);
-      case "csharp" -> new CSharpCTSManager(client);
+      case "csharp" -> new CSharpCTSManager(client, overrideLanguageVersion);
       case "swift" -> new SwiftCTSManager(client);
       default -> null;
     };

generators/src/main/java/com/algolia/codegen/cts/manager/DartCTSManager.java

@@ -27,7 +27,6 @@ public String getClient() {
 
   @Override
   public void addDataToBundle(Map<String, Object> bundle) throws GeneratorException {
-    bundle.put("packageVersion", getVersion());
     if (client.equals("algoliasearch")) {
       bundle.put("import", "package:algoliasearch/algoliasearch_lite.dart");
       bundle.put("client", "SearchClient");

generators/src/main/java/com/algolia/codegen/cts/manager/JavaCTSManager.java

@@ -26,11 +26,7 @@ public String getClient() {
 
   @Override
   public String getLanguageVersion(String override) throws IOException {
-    if (override != null && !override.isEmpty()) {
-      return override.split("\\.")[0];
-    }
-
-    return Helpers.getLanguageVersion(getLanguage()).split("\\.")[0];
+    return CTSManager.super.getLanguageVersion(override).split("\\.")[0];
   }
 
   @Override
@@ -46,7 +42,6 @@ public void addSnippetsSupportingFiles(List<SupportingFile> supportingFiles, Str
 
   @Override
   public void addDataToBundle(Map<String, Object> bundle) throws GeneratorException {
-    bundle.put("packageVersion", getVersion());
     bundle.put("import", Helpers.camelize(this.client).toLowerCase());
   }
 

generators/src/main/java/com/algolia/codegen/cts/manager/JavascriptCTSManager.java

@@ -27,7 +27,7 @@ public String getClient() {
     return client;
   }
 
-  public String getVersion() {
+  public String getPackageVersion() {
     return Helpers.getPackageJsonVersion(AlgoliaJavascriptGenerator.getPackageName(client));
   }