Merge pull request #1163 from christianhelle/copilot/fix-1157

Fix ArgumentException in OpenApiCSharpCodeGenerator.Sanitize method

Author: christianhelle

src/Core/ApiClientCodeGen.Core.Tests/Generators/OpenApi/OpenApiCSharpCodeGeneratorSanitizeTests.cs

@@ -0,0 +1,27 @@
+using System.Reflection;
+using Rapicgen.Core.Generators.OpenApi;
+using Xunit;
+
+namespace ApiClientCodeGen.Core.Tests.Generators.OpenApi
+{
+    public class OpenApiCSharpCodeGeneratorSanitizeTests
+    {
+        [Fact]
+        public void Sanitize_Removes_SystemNetMime_Using_Statement()
+        {
+            // Arrange
+            const string codeWithMimeUsing = "using System;\nusing System.Net.Mime;\nnamespace Test {}";
+            const string expectedResult = "using System;\n\nnamespace Test {}";
+            
+            // Get the private Sanitize method through reflection
+            var sanitizeMethod = typeof(OpenApiCSharpCodeGenerator)
+                .GetMethod("Sanitize", BindingFlags.NonPublic | BindingFlags.Static);
+            
+            // Act
+            var result = sanitizeMethod?.Invoke(null, new object[] { codeWithMimeUsing }) as string;
+            
+            // Assert
+            Assert.Equal(expectedResult, result);
+        }
+    }
+}

src/Core/ApiClientCodeGen.Core/Generators/OpenApi/OpenApiCSharpCodeGenerator.cs

@@ -81,15 +81,18 @@ public string GenerateCode(IProgressReporter? pGenerateProgress)
                 if (openApiGeneratorOptions.UseConfigurationFile)
                 {
                     var extension = Path.GetExtension(swaggerFile);
-                    var configFilename = swaggerFile.Replace(extension, $".config{extension}");
-                    var jsonConfigFilename = swaggerFile.Replace(extension, ".config.json");
-                    var yamlConfigFilename = swaggerFile.Replace(extension, ".config.yaml");
-
-                    var configFilenames = new[] { configFilename, jsonConfigFilename, yamlConfigFilename };
-                    var configFile = Array.Find(configFilenames, File.Exists);
-                    if (configFile != null)
+                    if (extension != null)
                     {
-                        arguments += $"-c \"{configFile}\" ";
+                        var configFilename = swaggerFile.Replace(extension, $".config{extension}");
+                        var jsonConfigFilename = swaggerFile.Replace(extension, ".config.json");
+                        var yamlConfigFilename = swaggerFile.Replace(extension, ".config.yaml");
+
+                        var configFilenames = new[] { configFilename, jsonConfigFilename, yamlConfigFilename };
+                        var configFile = Array.Find(configFilenames, File.Exists);
+                        if (configFile != null)
+                        {
+                            arguments += $"-c \"{configFile}\" ";
+                        }
                     }
                 }
 
@@ -167,6 +170,6 @@ public string GenerateCode(IProgressReporter? pGenerateProgress)
         }
 
         private static string Sanitize(string code) =>
-            code.Replace("using System.Net.Mime;", null);
+            code.Replace("using System.Net.Mime;", string.Empty);
     }
 }