fix - fix install hook exception and update test SDK version +semver:…

… minor (#129)
guibranco · Nov 20, 2024 · 91b9809 · 91b9809
1 parent 2f179bd
commit 91b9809
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 4 deletions.
diff --git a/Src/AiCommitMessage/Services/InstallHookService.cs b/Src/AiCommitMessage/Services/InstallHookService.cs
@@ -26,7 +26,6 @@ public static void InstallHook(InstallHookOptions options)
         if (string.IsNullOrWhiteSpace(options.Path))
         {
             directory = Path.Combine(GetGitRepositoryRootLevel(), GetHooksDirectory());
-            EnsureDirectoryExists(directory);
         }
 
         var hookPath = Path.Combine(directory, "prepare-commit-msg");
@@ -190,7 +189,7 @@ private static void ExtractEmbeddedResource(
         string file
     )
     {
-        EnsureDirectoryExists(outputDir);
+        EnsureDirectoryExists(Path.Combine(outputDir, file));
         using var stream = typeof(InstallHookService).Assembly.GetManifestResourceStream(
             resourceLocation + "." + file
         );
@@ -219,7 +218,7 @@ string file
     private static void EnsureDirectoryExists(string path)
     {
         var directoryName = Path.GetDirectoryName(path);
-        if (directoryName is { Length: > 0 })
+        if (directoryName is { Length: > 0 } && !Directory.Exists(directoryName))
         {
             Directory.CreateDirectory(directoryName);
         }

diff --git a/Tests/AiCommitMessage.Tests/AiCommitMessage.Tests.csproj b/Tests/AiCommitMessage.Tests/AiCommitMessage.Tests.csproj
@@ -15,7 +15,7 @@
       <IncludeAssets>runtime; build; native; contentfiles; analyzers; buildtransitive</IncludeAssets>
     </PackageReference>
     <PackageReference Include="FluentAssertions" Version="6.12.2" />
-    <PackageReference Include="Microsoft.NET.Test.Sdk" Version="17.11.1" />
+    <PackageReference Include="Microsoft.NET.Test.Sdk" Version="17.12.0" />
     <PackageReference Include="NSubstitute" Version="5.3.0" />
     <PackageReference Include="xunit" Version="2.9.2" />
     <PackageReference Include="xunit.runner.visualstudio" Version="2.8.2">