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

Author: millotp

.github/workflows/check.yml

@@ -353,7 +353,7 @@ jobs:
           [ $(find Sources  -type f | rev | cut -d '/' -f1 | rev | sort | uniq -d | wc -l) -gt 0 ] && echo "Duplicates found" && exit 1 || echo "No duplicate found"
 
       - name: Build the playground
-        run: yarn cli build playground ${{ matrix.client.language }}
+        run: yarn cli build playground ${{ matrix.client.language }} --language-version ${{ matrix.client.version }}
 
       - name: Run Java 'algoliasearch' public API validation
         id: breakingChanges
@@ -397,13 +397,13 @@ jobs:
         run: yarn cli cts run ${{ matrix.client.language }} --benchmark --no-client --no-requests --no-e2e
 
       - name: Generate code snippets
-        run: yarn cli snippets ${{ matrix.client.language }}
+        run: yarn cli snippets ${{ matrix.client.language }} --language-version ${{ matrix.client.version }}
 
       - name: Build the snippets to check validity
         run: yarn cli build snippets ${{ matrix.client.language }}
 
       - name: Generate code guides
-        run: yarn cli guides ${{ matrix.client.language }}
+        run: yarn cli guides ${{ matrix.client.language }} ${{ matrix.client.language != 'java' && format('--language-version {0}', matrix.client.version) || '' }}
 
       - name: Build the guides to check validity
         run: yarn cli build guides ${{ matrix.client.language }}

clients/algoliasearch-client-csharp/.github/workflows/release.yml

@@ -16,7 +16,7 @@ jobs:
       - name: Setup .NET
         uses: actions/setup-dotnet@v5
         with:
-          global-json-file: global.json
+          dotnet-version: 10.0.100
 
       - name: Create Nugget package
         run: dotnet pack --configuration Release --output nugget

clients/algoliasearch-client-csharp/global.json

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

config/clients.config.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/clients.schema.json

@@ -16,7 +16,6 @@ export const patterns = [
   // C#
   'clients/algoliasearch-client-csharp/**',
   '!clients/algoliasearch-client-csharp/*',
-  'clients/algoliasearch-client-csharp/global.json',
   '!clients/algoliasearch-client-csharp/algoliasearch/Clients/AlgoliaConfig.cs',
   '!clients/algoliasearch-client-csharp/algoliasearch/Exceptions/**',
   '!clients/algoliasearch-client-csharp/algoliasearch/Serializer/**',
@@ -25,7 +24,7 @@ export const patterns = [
   '!clients/algoliasearch-client-csharp/algoliasearch/Transport/**',
   '!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/**',

config/generation.config.mjs

@@ -110,7 +110,6 @@ public void processOpts() {
 
     // repository
     supportingFiles.add(new SupportingFile("Solution.mustache", "../", "Algolia.Search.sln"));
-    supportingFiles.add(new SupportingFile("globaljson.mustache", "../", "global.json"));
     supportingFiles.add(new SupportingFile("netcore_project.mustache", "Algolia.Search.csproj"));
     supportingFiles.add(new SupportingFile("Configuration.mustache", "Clients", packageName + "Configuration.cs"));
 

generators/src/main/java/com/algolia/codegen/AlgoliaCSharpGenerator.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/AlgoliaCTSGenerator.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() {
@@ -25,7 +28,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 +48,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/CSharpCTSManager.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");
   }