Improve unit tests by disposing connection to avoid locked temporary …

…files [skip release notes]
frblondin · Mar 24, 2023 · 1604ed4 · 1604ed4
1 parent b6b9f57
commit 1604ed4
Showing 1 changed file with 8 additions and 7 deletions.
diff --git a/src/tests/GitObjectDb.Tests/Commands/CommitCommandTests.cs b/src/tests/GitObjectDb.Tests/Commands/CommitCommandTests.cs
@@ -29,7 +29,7 @@ public void AddNewNodeUsingNodeFolders(IFixture fixture, Application application
         var comparer = fixture.Create<Comparer>();
         var gitUpdateCommand = fixture.Create<IGitUpdateCommand>();
         var sut = fixture.Create<ICommitCommand>();
-        var connection = fixture.Create<IConnectionInternal>();
+        using var connection = fixture.Create<IConnectionInternal>();
 
         // Act
         var composer = new TransformationComposer(connection, "main", gitUpdateCommand, sut);
@@ -54,7 +54,7 @@ public void AddNewNodeWithoutNodeFolders(IFixture fixture, Table table, UniqueId
         var comparer = fixture.Create<Comparer>();
         var gitUpdateCommand = fixture.Create<IGitUpdateCommand>();
         var sut = fixture.Create<ICommitCommand>();
-        var connection = fixture.Create<IConnectionInternal>();
+        using var connection = fixture.Create<IConnectionInternal>();
 
         // Act
         var composer = new TransformationComposer(connection, "main", gitUpdateCommand, sut);
@@ -79,7 +79,7 @@ public void AddNewResource(IFixture fixture, Table table, string fileContent, st
         var comparer = fixture.Create<Comparer>();
         var gitUpdateCommand = fixture.Create<IGitUpdateCommand>();
         var sut = fixture.Create<ICommitCommand>();
-        var connection = fixture.Create<IConnectionInternal>();
+        using var connection = fixture.Create<IConnectionInternal>();
         var resource = new Resource(table, "Some/Folder", "File.txt", new Resource.Data(fileContent));
 
         // Act
@@ -107,7 +107,7 @@ public void DeletingNodeRemovesNestedChildren(IFixture fixture, Table table, str
         var comparer = fixture.Create<Comparer>();
         var gitUpdateCommand = fixture.Create<IGitUpdateCommand>();
         var sut = fixture.Create<ICommitCommand>();
-        var connection = fixture.Create<IConnectionInternal>();
+        using var connection = fixture.Create<IConnectionInternal>();
 
         // Act
         var composer = new TransformationComposer(connection, "main", gitUpdateCommand, sut);
@@ -130,7 +130,7 @@ public void RenamingNonGitFoldersIsSupported(IFixture fixture, Field field, stri
         var comparer = fixture.Create<Comparer>();
         var gitUpdateCommand = fixture.Create<IGitUpdateCommand>();
         var sut = fixture.Create<ICommitCommand>();
-        var connection = fixture.Create<IConnectionInternal>();
+        using var connection = fixture.Create<IConnectionInternal>();
 
         // Act
         var composer = new TransformationComposer(connection, "main", gitUpdateCommand, sut);
@@ -156,7 +156,7 @@ public void RenamingGitFoldersIsNotSupported(IFixture fixture, Table table, stri
         var comparer = fixture.Create<Comparer>();
         var gitUpdateCommand = fixture.Create<IGitUpdateCommand>();
         var sut = fixture.Create<ICommitCommand>();
-        var connection = fixture.Create<IConnectionInternal>();
+        using var connection = fixture.Create<IConnectionInternal>();
 
         // Act
         var composer = new TransformationComposer(connection, "main", gitUpdateCommand, sut);
@@ -174,7 +174,7 @@ public void EditNestedProperty(IFixture fixture, Field field, string message, Si
         var comparer = fixture.Create<Comparer>();
         var gitUpdateCommand = fixture.Create<IGitUpdateCommand>();
         var sut = fixture.Create<ICommitCommand>();
-        var connection = fixture.Create<IConnectionInternal>();
+        using var connection = fixture.Create<IConnectionInternal>();
 
         // Act
         var composer = new TransformationComposer(connection, "main", gitUpdateCommand, sut);
@@ -215,6 +215,7 @@ public void Customize(IFixture fixture)
             A.CallTo(() => connection.Model).Returns(fixture.Create<IDataModel>());
             A.CallTo(() => connection.Serializer).Returns(fixture.Create<INodeSerializer>());
             A.CallTo(() => connection.Cache).Returns(fixture.Create<IMemoryCache>());
+            A.CallTo(() => connection.Dispose()).DoesNothing();
             fixture.Inject(connection);
 
             var validation = A.Fake<ITreeValidation>(x => x.Strict());