Merge pull request #65 from christianhelle/improve-code-quality

christianhelle · web-flow · commit c4b28cbdd870 · 2019-10-28T16:27:37.000+01:00
Improve code quality
diff --git a/src/ApiClientCodeGen.Tests/Generators/CodeGeneratorFactoryTests.cs b/src/ApiClientCodeGen.Tests/Generators/CodeGeneratorFactoryTests.cs
@@ -1,4 +1,5 @@
-﻿using ChristianHelle.DeveloperTools.CodeGenerators.ApiClient.Core;
+﻿using System;
+using ChristianHelle.DeveloperTools.CodeGenerators.ApiClient.Core;
 using ChristianHelle.DeveloperTools.CodeGenerators.ApiClient.Generators;
 using ChristianHelle.DeveloperTools.CodeGenerators.ApiClient.Generators.AutoRest;
 using ChristianHelle.DeveloperTools.CodeGenerators.ApiClient.Generators.NSwag;
@@ -23,6 +24,9 @@ public class CodeGeneratorFactoryTests
         public void Init()
         {
             var mockFactory = new Mock<IOptionsFactory>();
+            mockFactory
+                .Setup(c => c.Create<IAutoRestOptions, AutoRestOptionsPage>())
+                .Returns(Test.CreateDummy<IAutoRestOptions>());
             mockFactory
                 .Setup(c => c.Create<INSwagOptions, NSwagOptionsPage>())
                 .Returns(Test.CreateDummy<INSwagOptions>());
@@ -79,5 +83,17 @@ public void Can_Create_OpenApiCodeGenerator()
                     SupportedCodeGenerator.OpenApi)
                 .Should()
                 .BeOfType<OpenApiCSharpCodeGenerator>();
+
+        [TestMethod]
+        public void Create_NSwagStudio_Throws_NotSupported()
+            => new Action(
+                    () => sut.Create(
+                        string.Empty,
+                        string.Empty,
+                        string.Empty,
+                        SupportedLanguage.CSharp,
+                        SupportedCodeGenerator.NSwagStudio))
+                .Should()
+                .ThrowExactly<NotSupportedException>();
     }
 }
diff --git a/src/ApiClientCodeGen.Tests/Generators/NSwag/NSwagCSharpCodeGeneratorTests.cs b/src/ApiClientCodeGen.Tests/Generators/NSwag/NSwagCSharpCodeGeneratorTests.cs
@@ -35,8 +35,6 @@ public async Task Init()
 
             var sut = new NSwagCSharpCodeGenerator(
                 "Swagger.json",
-                Test.CreateAnnonymous<string>(),
-                optionsMock.Object,
                 documentFactoryMock.Object,
                 settingsMock.Object);
             
diff --git a/src/ApiClientCodeGen.VSIX/Commands/AddNew/NewAutoRestClientCommand.cs b/src/ApiClientCodeGen.VSIX/Commands/AddNew/NewAutoRestClientCommand.cs
@@ -1,5 +1,5 @@
-﻿using ChristianHelle.DeveloperTools.CodeGenerators.ApiClient.Core;
-using System.Diagnostics.CodeAnalysis;
+﻿using System.Diagnostics.CodeAnalysis;
+using ChristianHelle.DeveloperTools.CodeGenerators.ApiClient.Core;
 
 namespace ChristianHelle.DeveloperTools.CodeGenerators.ApiClient.Commands.AddNew
 {
diff --git a/src/ApiClientCodeGen.VSIX/Commands/AddNew/NewNSwagClientCommand.cs b/src/ApiClientCodeGen.VSIX/Commands/AddNew/NewNSwagClientCommand.cs
@@ -1,5 +1,5 @@
-﻿using ChristianHelle.DeveloperTools.CodeGenerators.ApiClient.Core;
-using System.Diagnostics.CodeAnalysis;
+﻿using System.Diagnostics.CodeAnalysis;
+using ChristianHelle.DeveloperTools.CodeGenerators.ApiClient.Core;
 
 namespace ChristianHelle.DeveloperTools.CodeGenerators.ApiClient.Commands.AddNew
 {
diff --git a/src/ApiClientCodeGen.VSIX/Commands/AddNew/NewNSwagStudioClientCommand.cs b/src/ApiClientCodeGen.VSIX/Commands/AddNew/NewNSwagStudioClientCommand.cs
@@ -1,5 +1,5 @@
-﻿using ChristianHelle.DeveloperTools.CodeGenerators.ApiClient.Core;
-using System.Diagnostics.CodeAnalysis;
+﻿using System.Diagnostics.CodeAnalysis;
+using ChristianHelle.DeveloperTools.CodeGenerators.ApiClient.Core;
 
 namespace ChristianHelle.DeveloperTools.CodeGenerators.ApiClient.Commands.AddNew
 {
diff --git a/src/ApiClientCodeGen.VSIX/Commands/AddNew/NewOpenApiClientCommand.cs b/src/ApiClientCodeGen.VSIX/Commands/AddNew/NewOpenApiClientCommand.cs
@@ -1,5 +1,5 @@
-﻿using ChristianHelle.DeveloperTools.CodeGenerators.ApiClient.Core;
-using System.Diagnostics.CodeAnalysis;
+﻿using System.Diagnostics.CodeAnalysis;
+using ChristianHelle.DeveloperTools.CodeGenerators.ApiClient.Core;
 
 namespace ChristianHelle.DeveloperTools.CodeGenerators.ApiClient.Commands.AddNew
 {
diff --git a/src/ApiClientCodeGen.VSIX/Commands/AddNew/NewRestClientCommand.cs b/src/ApiClientCodeGen.VSIX/Commands/AddNew/NewRestClientCommand.cs
@@ -6,14 +6,11 @@
 using ChristianHelle.DeveloperTools.CodeGenerators.ApiClient.Core;
 using ChristianHelle.DeveloperTools.CodeGenerators.ApiClient.Extensions;
 using ChristianHelle.DeveloperTools.CodeGenerators.ApiClient.Generators.NSwagStudio;
-using ChristianHelle.DeveloperTools.CodeGenerators.ApiClient.Options;
 using ChristianHelle.DeveloperTools.CodeGenerators.ApiClient.Options.General;
-using ChristianHelle.DeveloperTools.CodeGenerators.ApiClient.Options.NSwag;
 using ChristianHelle.DeveloperTools.CodeGenerators.ApiClient.Options.NSwagStudio;
 using ChristianHelle.DeveloperTools.CodeGenerators.ApiClient.Windows;
 using EnvDTE;
 using Microsoft.VisualStudio.Shell;
-using Microsoft.VisualStudio.Threading;
 using Newtonsoft.Json;
 using VSLangProj;
 using Task = System.Threading.Tasks.Task;
@@ -104,4 +101,4 @@ private static string FindFolder(object item, DTE dte)
             }
         }
     }
-}
+}
diff --git a/src/ApiClientCodeGen.VSIX/Commands/AddNew/NewSwaggerClientCommand.cs b/src/ApiClientCodeGen.VSIX/Commands/AddNew/NewSwaggerClientCommand.cs
@@ -1,5 +1,5 @@
-﻿using ChristianHelle.DeveloperTools.CodeGenerators.ApiClient.Core;
-using System.Diagnostics.CodeAnalysis;
+﻿using System.Diagnostics.CodeAnalysis;
+using ChristianHelle.DeveloperTools.CodeGenerators.ApiClient.Core;
 
 namespace ChristianHelle.DeveloperTools.CodeGenerators.ApiClient.Commands.AddNew
 {
diff --git a/src/ApiClientCodeGen.VSIX/Commands/NSwagStudio/NSwagStudioCommand.cs b/src/ApiClientCodeGen.VSIX/Commands/NSwagStudio/NSwagStudioCommand.cs
@@ -4,7 +4,6 @@
 using ChristianHelle.DeveloperTools.CodeGenerators.ApiClient.Core;
 using ChristianHelle.DeveloperTools.CodeGenerators.ApiClient.Extensions;
 using ChristianHelle.DeveloperTools.CodeGenerators.ApiClient.Generators.NSwagStudio;
-using ChristianHelle.DeveloperTools.CodeGenerators.ApiClient.Options;
 using ChristianHelle.DeveloperTools.CodeGenerators.ApiClient.Options.General;
 using EnvDTE;
 using Microsoft.VisualStudio.Shell;
@@ -38,9 +37,9 @@ private static async Task OnExecuteAsync(DTE dte, AsyncPackage package)
             var nswagStudioFile = item.FileNames[0];
             var codeGenerator = new NSwagStudioCodeGenerator(nswagStudioFile, new CustomPathOptions());
             codeGenerator.GenerateCode(null);
-            
+
             var project = ProjectExtensions.GetActiveProject(dte);
             await project.InstallMissingPackagesAsync(package, SupportedCodeGenerator.NSwag);
         }
     }
-}
+}
diff --git a/src/ApiClientCodeGen.VSIX/Extensions/StringExtensions.cs b/src/ApiClientCodeGen.VSIX/Extensions/StringExtensions.cs
@@ -1,4 +1,5 @@
 ﻿using System;
+using System.Collections.Generic;
 using System.Runtime.InteropServices;
 using System.Text;
 using Newtonsoft.Json;
@@ -9,18 +10,12 @@ namespace ChristianHelle.DeveloperTools.CodeGenerators.ApiClient.Extensions
 {
     public static class StringExtension
     {
-        private static readonly JsonSerializerSettings jsonSettings;
-
-        static StringExtension()
+        private static readonly JsonSerializerSettings JsonSettings = new JsonSerializerSettings
         {
-            jsonSettings = new JsonSerializerSettings
-            {
-                NullValueHandling = NullValueHandling.Ignore,
-                ContractResolver = new CamelCasePropertyNamesContractResolver()
-            };
-
-            jsonSettings.Converters.Add(new StringEnumConverter());
-        }
+            NullValueHandling = NullValueHandling.Ignore,
+            ContractResolver = new CamelCasePropertyNamesContractResolver(),
+            Converters = new List<JsonConverter> { new StringEnumConverter() }
+        };
 
         public static IntPtr ConvertToIntPtr(this string code, out uint pcbOutput)
         {
@@ -36,6 +31,6 @@ public static IntPtr ConvertToIntPtr(this string code, out uint pcbOutput)
         public static string ToJson(this object value)
             => JsonConvert.SerializeObject(
                 value,
-                jsonSettings);
+                JsonSettings);
     }
 }
diff --git a/src/ApiClientCodeGen.VSIX/Generators/AutoRest/AutoRestCSharpCodeGenerator.cs b/src/ApiClientCodeGen.VSIX/Generators/AutoRest/AutoRestCSharpCodeGenerator.cs
@@ -1,7 +1,6 @@
 ﻿using System;
 using System.IO;
 using ChristianHelle.DeveloperTools.CodeGenerators.ApiClient.Core;
-using ChristianHelle.DeveloperTools.CodeGenerators.ApiClient.Options;
 using ChristianHelle.DeveloperTools.CodeGenerators.ApiClient.Options.AutoRest;
 using ChristianHelle.DeveloperTools.CodeGenerators.ApiClient.Options.General;
 
@@ -62,4 +61,4 @@ protected override string GetCommand()
             return autorestCmd;
         }
     }
-}
+}
diff --git a/src/ApiClientCodeGen.VSIX/Generators/CSharpFileMerger.cs b/src/ApiClientCodeGen.VSIX/Generators/CSharpFileMerger.cs
@@ -72,37 +72,33 @@ private static IEnumerable<string> GetSourceFileNames(string path)
         {
             var queue = new Queue<string>();
             queue.Enqueue(path);
-            
+
             while (queue.Count > 0)
             {
                 string[] files = null;
                 path = queue.Dequeue();
                 try
                 {
                     files = Directory.GetFiles(path);
-                    foreach (var subDir in Directory.GetDirectories(path))
-                    {
-                        queue.Enqueue(subDir);
-                    }
+                    foreach (var subDir in Directory.GetDirectories(path)) queue.Enqueue(subDir);
                 }
                 catch (Exception ex)
                 {
                     Trace.TraceError(ex.ToString());
                 }
 
-                if (files == null) 
+                if (files == null)
                     continue;
 
-                foreach (var file in files.Where(Predicate()))
-                {
-                    yield return file;
-                }
+                foreach (var file in files.Where(Predicate())) yield return file;
             }
         }
 
-        private static Func<string, bool> Predicate() 
-            => file => file.EndsWith(".cs") && 
-                       !file.Contains("AssemblyInfo.cs");
+        private static Func<string, bool> Predicate()
+        {
+            return file => file.EndsWith(".cs") &&
+                           !file.Contains("AssemblyInfo.cs");
+        }
 
 
         private static IEnumerable<string> GetUniqueNamespaces(IEnumerable<string> files)
@@ -118,7 +114,7 @@ private static IEnumerable<string> GetUniqueNamespaces(IEnumerable<string> files
                 foreach (var sourceLine in sourceLines)
                 {
                     var trimmedLine = sourceLine.Trim().Replace("  ", " ");
-                    if (!trimmedLine.StartsWith(openingTag) || !trimmedLine.EndsWith(";")) 
+                    if (!trimmedLine.StartsWith(openingTag) || !trimmedLine.EndsWith(";"))
                         continue;
 
                     var name = trimmedLine.Substring(namespaceStartIndex, trimmedLine.Length - namespaceStartIndex - 1);
@@ -129,6 +125,5 @@ private static IEnumerable<string> GetUniqueNamespaces(IEnumerable<string> files
 
             return names;
         }
-
     }
 }
diff --git a/src/ApiClientCodeGen.VSIX/Generators/CodeGenerator.cs b/src/ApiClientCodeGen.VSIX/Generators/CodeGenerator.cs
@@ -12,8 +12,8 @@ public interface ICodeGenerator
 
     public abstract class CodeGenerator : ICodeGenerator
     {
-        protected readonly string SwaggerFile;
         protected readonly string DefaultNamespace;
+        protected readonly string SwaggerFile;
 
         protected CodeGenerator(string swaggerFile, string defaultNamespace)
         {
@@ -49,4 +49,4 @@ public virtual string GenerateCode(IVsGeneratorProgress pGenerateProgress)
         protected abstract string GetArguments(string outputFile);
         protected abstract string GetCommand();
     }
-}
+}
diff --git a/src/ApiClientCodeGen.VSIX/Generators/CodeGeneratorFactory.cs b/src/ApiClientCodeGen.VSIX/Generators/CodeGeneratorFactory.cs
@@ -1,4 +1,5 @@
-﻿using ChristianHelle.DeveloperTools.CodeGenerators.ApiClient.Core;
+﻿using System;
+using ChristianHelle.DeveloperTools.CodeGenerators.ApiClient.Core;
 using ChristianHelle.DeveloperTools.CodeGenerators.ApiClient.Generators.AutoRest;
 using ChristianHelle.DeveloperTools.CodeGenerators.ApiClient.Generators.NSwag;
 using ChristianHelle.DeveloperTools.CodeGenerators.ApiClient.Generators.OpenApi;
@@ -7,7 +8,6 @@
 using ChristianHelle.DeveloperTools.CodeGenerators.ApiClient.Options.AutoRest;
 using ChristianHelle.DeveloperTools.CodeGenerators.ApiClient.Options.General;
 using ChristianHelle.DeveloperTools.CodeGenerators.ApiClient.Options.NSwag;
-using System;
 
 namespace ChristianHelle.DeveloperTools.CodeGenerators.ApiClient.Generators
 {
@@ -48,10 +48,10 @@ public ICodeGenerator Create(
                 case SupportedCodeGenerator.NSwag:
                     return new NSwagCSharpCodeGenerator(
                         inputFilePath,
-                        defaultNamespace,
-                        optionsFactory.Create<INSwagOptions, NSwagOptionsPage>(),
                         new OpenApiDocumentFactory(),
-                        new NSwagCodeGeneratorSettingsFactory(defaultNamespace, optionsFactory.Create<INSwagOptions, NSwagOptionsPage>()));
+                        new NSwagCodeGeneratorSettingsFactory(
+                            defaultNamespace,
+                            optionsFactory.Create<INSwagOptions, NSwagOptionsPage>()));
 
                 case SupportedCodeGenerator.Swagger:
                     return new SwaggerCSharpCodeGenerator(
@@ -68,7 +68,6 @@ public ICodeGenerator Create(
                 default:
                     throw new NotSupportedException();
             }
-
         }
     }
-}
+}
diff --git a/src/ApiClientCodeGen.VSIX/Generators/FileHelper.cs b/src/ApiClientCodeGen.VSIX/Generators/FileHelper.cs
@@ -22,10 +22,12 @@ public static string CalculateChecksum(string filename)
         {
             using (var hashAlgorithm = MD5.Create())
             using (var stream = File.OpenRead(filename))
+            {
                 return BitConverter
                     .ToString(hashAlgorithm.ComputeHash(stream))
                     .Replace("-", "")
                     .ToUpperInvariant();
+            }
         }
     }
 }
diff --git a/src/ApiClientCodeGen.VSIX/Generators/NSwag/NSwagCSharpCodeGenerator.cs b/src/ApiClientCodeGen.VSIX/Generators/NSwag/NSwagCSharpCodeGenerator.cs
@@ -1,32 +1,25 @@
 ﻿using System;
 using ChristianHelle.DeveloperTools.CodeGenerators.ApiClient.Extensions;
-using ChristianHelle.DeveloperTools.CodeGenerators.ApiClient.Options;
-using ChristianHelle.DeveloperTools.CodeGenerators.ApiClient.Options.NSwag;
 using Microsoft.VisualStudio.Shell.Interop;
 using NSwag.CodeGeneration.CSharp;
 
 namespace ChristianHelle.DeveloperTools.CodeGenerators.ApiClient.Generators.NSwag
 {
     public class NSwagCSharpCodeGenerator : ICodeGenerator
     {
-        private readonly string swaggerFile;
-        private readonly string defaultNamespace;
-        private readonly NSwagCSharpOptions options;
         private readonly IOpenApiDocumentFactory documentFactory;
         private readonly INSwagCodeGeneratorSettingsFactory generatorSettingsFactory;
+        private readonly string swaggerFile;
 
         public NSwagCSharpCodeGenerator(
-            string swaggerFile, 
-            string defaultNamespace, 
-            INSwagOptions options,
+            string swaggerFile,
             IOpenApiDocumentFactory documentFactory,
             INSwagCodeGeneratorSettingsFactory generatorSettingsFactory)
         {
             this.swaggerFile = swaggerFile ?? throw new ArgumentNullException(nameof(swaggerFile));
-            this.defaultNamespace = defaultNamespace ?? throw new ArgumentNullException(nameof(defaultNamespace));
             this.documentFactory = documentFactory ?? throw new ArgumentNullException(nameof(documentFactory));
-            this.generatorSettingsFactory = generatorSettingsFactory ?? throw new ArgumentNullException(nameof(generatorSettingsFactory));
-            this.options = new NSwagCSharpOptions(options ?? throw new ArgumentNullException(nameof(options)));
+            this.generatorSettingsFactory = generatorSettingsFactory ??
+                                            throw new ArgumentNullException(nameof(generatorSettingsFactory));
         }
 
         public string GenerateCode(IVsGeneratorProgress pGenerateProgress)
@@ -47,4 +40,4 @@ public string GenerateCode(IVsGeneratorProgress pGenerateProgress)
             }
         }
     }
-}
+}
diff --git a/src/ApiClientCodeGen.VSIX/Generators/NSwag/NSwagCodeGeneratorSettingsFactory.cs b/src/ApiClientCodeGen.VSIX/Generators/NSwag/NSwagCodeGeneratorSettingsFactory.cs
@@ -1,9 +1,5 @@
 ﻿using System;
-using System.Diagnostics;
-using System.IO;
-using System.Linq;
 using ChristianHelle.DeveloperTools.CodeGenerators.ApiClient.Extensions;
-using ChristianHelle.DeveloperTools.CodeGenerators.ApiClient.Options;
 using ChristianHelle.DeveloperTools.CodeGenerators.ApiClient.Options.NSwag;
 using NSwag;
 using NSwag.CodeGeneration.CSharp;
diff --git a/src/ApiClientCodeGen.VSIX/Generators/NSwagStudio/NSwagStudioCodeGenerator.cs b/src/ApiClientCodeGen.VSIX/Generators/NSwagStudio/NSwagStudioCodeGenerator.cs
@@ -3,7 +3,6 @@
 using System.IO;
 using ChristianHelle.DeveloperTools.CodeGenerators.ApiClient.Core;
 using ChristianHelle.DeveloperTools.CodeGenerators.ApiClient.Extensions;
-using ChristianHelle.DeveloperTools.CodeGenerators.ApiClient.Options;
 using ChristianHelle.DeveloperTools.CodeGenerators.ApiClient.Options.General;
 using Microsoft.VisualStudio.Shell.Interop;
 using Newtonsoft.Json;
@@ -60,4 +59,4 @@ private static void TryRemoveSwaggerJsonSpec(string nswagFile)
             }
         }
     }
-}
+}
diff --git a/src/ApiClientCodeGen.VSIX/Generators/NSwagStudio/NSwagStudioFileHelper.cs b/src/ApiClientCodeGen.VSIX/Generators/NSwagStudio/NSwagStudioFileHelper.cs
diff --git a/src/ApiClientCodeGen.VSIX/Generators/OpenApi/OpenApiCSharpCodeGenerator.cs b/src/ApiClientCodeGen.VSIX/Generators/OpenApi/OpenApiCSharpCodeGenerator.cs
diff --git a/src/ApiClientCodeGen.VSIX/Generators/Swagger/SwaggerCSharpCodeGenerator.cs b/src/ApiClientCodeGen.VSIX/Generators/Swagger/SwaggerCSharpCodeGenerator.cs
diff --git a/src/ApiClientCodeGen.VSIX/GlobalSuppressions.cs b/src/ApiClientCodeGen.VSIX/GlobalSuppressions.cs
diff --git a/src/ApiClientCodeGen.VSIX/Options/OptionsBase.cs b/src/ApiClientCodeGen.VSIX/Options/OptionsBase.cs
diff --git a/src/ApiClientCodeGen.VSIX/Windows/GeneralOptionsPageCustom.cs b/src/ApiClientCodeGen.VSIX/Windows/GeneralOptionsPageCustom.cs
diff --git a/src/ApiClientCodegen.IntegrationTests/Generators/NSwagCodeGeneratorTests.cs b/src/ApiClientCodegen.IntegrationTests/Generators/NSwagCodeGeneratorTests.cs

Original file line number	Diff line number	Diff line change
`@@ -1,5 +1,5 @@`
`1`		`-using ChristianHelle.DeveloperTools.CodeGenerators.ApiClient.Core;`
`2`		`-using System.Diagnostics.CodeAnalysis;`
	`1`	`+using System.Diagnostics.CodeAnalysis;`
	`2`	`+using ChristianHelle.DeveloperTools.CodeGenerators.ApiClient.Core;`
`3`	`3`
`4`	`4`	`namespace ChristianHelle.DeveloperTools.CodeGenerators.ApiClient.Commands.AddNew`
`5`	`5`	`{`
Original file line number	Diff line number	Diff line change
`@@ -1,7 +1,6 @@`
`1`	`1`	`using System;`
`2`	`2`	`using System.IO;`
`3`	`3`	`using ChristianHelle.DeveloperTools.CodeGenerators.ApiClient.Core;`
`4`		`-using ChristianHelle.DeveloperTools.CodeGenerators.ApiClient.Options;`
`5`	`4`	`using ChristianHelle.DeveloperTools.CodeGenerators.ApiClient.Options.AutoRest;`
`6`	`5`	`using ChristianHelle.DeveloperTools.CodeGenerators.ApiClient.Options.General;`
`7`	`6`
`@@ -62,4 +61,4 @@ protected override string GetCommand()`
`62`	`61`	`return autorestCmd;`
`63`	`62`	`}`
`64`	`63`	`}`
`65`		`-}`
	`64`	`+}`
Original file line number	Diff line number	Diff line change
`@@ -72,37 +72,33 @@ private static IEnumerable<string> GetSourceFileNames(string path)`
`72`	`72`	`{`
`73`	`73`	`var queue = new Queue<string>();`
`74`	`74`	`queue.Enqueue(path);`
`75`		`-`
	`75`	`+`
`76`	`76`	`while (queue.Count > 0)`
`77`	`77`	`{`
`78`	`78`	`string[] files = null;`
`79`	`79`	`path = queue.Dequeue();`
`80`	`80`	`try`
`81`	`81`	`{`
`82`	`82`	`files = Directory.GetFiles(path);`
`83`		`- foreach (var subDir in Directory.GetDirectories(path))`
`84`		`- {`
`85`		`- queue.Enqueue(subDir);`
`86`		`- }`
	`83`	`+ foreach (var subDir in Directory.GetDirectories(path)) queue.Enqueue(subDir);`
`87`	`84`	`}`
`88`	`85`	`catch (Exception ex)`
`89`	`86`	`{`
`90`	`87`	`Trace.TraceError(ex.ToString());`
`91`	`88`	`}`
`92`	`89`
`93`		`- if (files == null)`
	`90`	`+ if (files == null)`
`94`	`91`	`continue;`
`95`	`92`
`96`		`- foreach (var file in files.Where(Predicate()))`
`97`		`- {`
`98`		`- yield return file;`
`99`		`- }`
	`93`	`+ foreach (var file in files.Where(Predicate())) yield return file;`
`100`	`94`	`}`
`101`	`95`	`}`
`102`	`96`
`103`		`- private static Func<string, bool> Predicate()`
`104`		`- => file => file.EndsWith(".cs") &&`
`105`		`- !file.Contains("AssemblyInfo.cs");`
	`97`	`+ private static Func<string, bool> Predicate()`
	`98`	`+ {`
	`99`	`+ return file => file.EndsWith(".cs") &&`
	`100`	`+ !file.Contains("AssemblyInfo.cs");`
	`101`	`+ }`
`106`	`102`
`107`	`103`
`108`	`104`	`private static IEnumerable<string> GetUniqueNamespaces(IEnumerable<string> files)`
`@@ -118,7 +114,7 @@ private static IEnumerable<string> GetUniqueNamespaces(IEnumerable<string> files`
`118`	`114`	`foreach (var sourceLine in sourceLines)`
`119`	`115`	`{`
`120`	`116`	`var trimmedLine = sourceLine.Trim().Replace(" ", " ");`
`121`		`- if (!trimmedLine.StartsWith(openingTag) \|\| !trimmedLine.EndsWith(";"))`
	`117`	`+ if (!trimmedLine.StartsWith(openingTag) \|\| !trimmedLine.EndsWith(";"))`
`122`	`118`	`continue;`
`123`	`119`
`124`	`120`	`var name = trimmedLine.Substring(namespaceStartIndex, trimmedLine.Length - namespaceStartIndex - 1);`
`@@ -129,6 +125,5 @@ private static IEnumerable<string> GetUniqueNamespaces(IEnumerable<string> files`
`129`	`125`
`130`	`126`	`return names;`
`131`	`127`	`}`
`132`		`-`
`133`	`128`	`}`
`134`	`129`	`}`
Original file line number	Diff line number	Diff line change
`@@ -12,8 +12,8 @@ public interface ICodeGenerator`
`12`	`12`
`13`	`13`	`public abstract class CodeGenerator : ICodeGenerator`
`14`	`14`	`{`
`15`		`- protected readonly string SwaggerFile;`
`16`	`15`	`protected readonly string DefaultNamespace;`
	`16`	`+ protected readonly string SwaggerFile;`
`17`	`17`
`18`	`18`	`protected CodeGenerator(string swaggerFile, string defaultNamespace)`
`19`	`19`	`{`
`@@ -49,4 +49,4 @@ public virtual string GenerateCode(IVsGeneratorProgress pGenerateProgress)`
`49`	`49`	`protected abstract string GetArguments(string outputFile);`
`50`	`50`	`protected abstract string GetCommand();`
`51`	`51`	`}`
`52`		`-}`
	`52`	`+}`
Original file line number	Diff line number	Diff line change
`@@ -22,10 +22,12 @@ public static string CalculateChecksum(string filename)`
`22`	`22`	`{`
`23`	`23`	`using (var hashAlgorithm = MD5.Create())`
`24`	`24`	`using (var stream = File.OpenRead(filename))`
	`25`	`+ {`
`25`	`26`	`return BitConverter`
`26`	`27`	`.ToString(hashAlgorithm.ComputeHash(stream))`
`27`	`28`	`.Replace("-", "")`
`28`	`29`	`.ToUpperInvariant();`
	`30`	`+ }`
`29`	`31`	`}`
`30`	`32`	`}`
`31`	`33`	`}`
Original file line number	Diff line number	Diff line change
`@@ -1,32 +1,25 @@`
`1`	`1`	`using System;`
`2`	`2`	`using ChristianHelle.DeveloperTools.CodeGenerators.ApiClient.Extensions;`
`3`		`-using ChristianHelle.DeveloperTools.CodeGenerators.ApiClient.Options;`
`4`		`-using ChristianHelle.DeveloperTools.CodeGenerators.ApiClient.Options.NSwag;`
`5`	`3`	`using Microsoft.VisualStudio.Shell.Interop;`
`6`	`4`	`using NSwag.CodeGeneration.CSharp;`
`7`	`5`
`8`	`6`	`namespace ChristianHelle.DeveloperTools.CodeGenerators.ApiClient.Generators.NSwag`
`9`	`7`	`{`
`10`	`8`	`public class NSwagCSharpCodeGenerator : ICodeGenerator`
`11`	`9`	`{`
`12`		`- private readonly string swaggerFile;`
`13`		`- private readonly string defaultNamespace;`
`14`		`- private readonly NSwagCSharpOptions options;`
`15`	`10`	`private readonly IOpenApiDocumentFactory documentFactory;`
`16`	`11`	`private readonly INSwagCodeGeneratorSettingsFactory generatorSettingsFactory;`
	`12`	`+ private readonly string swaggerFile;`
`17`	`13`
`18`	`14`	`public NSwagCSharpCodeGenerator(`
`19`		`- string swaggerFile,`
`20`		`- string defaultNamespace,`
`21`		`- INSwagOptions options,`
	`15`	`+ string swaggerFile,`
`22`	`16`	`IOpenApiDocumentFactory documentFactory,`
`23`	`17`	`INSwagCodeGeneratorSettingsFactory generatorSettingsFactory)`
`24`	`18`	`{`
`25`	`19`	`this.swaggerFile = swaggerFile ?? throw new ArgumentNullException(nameof(swaggerFile));`
`26`		`- this.defaultNamespace = defaultNamespace ?? throw new ArgumentNullException(nameof(defaultNamespace));`
`27`	`20`	`this.documentFactory = documentFactory ?? throw new ArgumentNullException(nameof(documentFactory));`
`28`		`- this.generatorSettingsFactory = generatorSettingsFactory ?? throw new ArgumentNullException(nameof(generatorSettingsFactory));`
`29`		`- this.options = new NSwagCSharpOptions(options ?? throw new ArgumentNullException(nameof(options)));`
	`21`	`+ this.generatorSettingsFactory = generatorSettingsFactory ??`
	`22`	`+ throw new ArgumentNullException(nameof(generatorSettingsFactory));`
`30`	`23`	`}`
`31`	`24`
`32`	`25`	`public string GenerateCode(IVsGeneratorProgress pGenerateProgress)`
`@@ -47,4 +40,4 @@ public string GenerateCode(IVsGeneratorProgress pGenerateProgress)`
`47`	`40`	`}`
`48`	`41`	`}`
`49`	`42`	`}`
`50`		`-}`
	`43`	`+}`
Original file line number	Diff line number	Diff line change
`@@ -3,7 +3,6 @@`
`3`	`3`	`using System.IO;`
`4`	`4`	`using ChristianHelle.DeveloperTools.CodeGenerators.ApiClient.Core;`
`5`	`5`	`using ChristianHelle.DeveloperTools.CodeGenerators.ApiClient.Extensions;`
`6`		`-using ChristianHelle.DeveloperTools.CodeGenerators.ApiClient.Options;`
`7`	`6`	`using ChristianHelle.DeveloperTools.CodeGenerators.ApiClient.Options.General;`
`8`	`7`	`using Microsoft.VisualStudio.Shell.Interop;`
`9`	`8`	`using Newtonsoft.Json;`
`@@ -60,4 +59,4 @@ private static void TryRemoveSwaggerJsonSpec(string nswagFile)`
`60`	`59`	`}`
`61`	`60`	`}`
`62`	`61`	`}`
`63`		`-}`
	`62`	`+}`