Refactor tests to reduce duplication by extracting helper methods

Co-authored-by: baronfel <573979+baronfel@users.noreply.github.com>

Author: Copilot

src/Tasks/Microsoft.NET.Build.Tasks.UnitTests/GivenAGenerateDepsFile.cs

@@ -25,31 +25,26 @@ public GivenAGenerateDepsFile()
         [Fact]
         public void ItDoesNotOverwriteFileWithSameContent()
         {
-            var task = new TestableGenerateDepsFile
-            {
-                BuildEngine = new MockNeverCacheBuildEngine4(),
-                ProjectPath = "TestProject.csproj",
-                DepsFilePath = _depsFilePath,
-                TargetFramework = "net8.0",
-                AssemblyName = "TestProject",
-                AssemblyExtension = ".dll",
-                AssemblyVersion = "1.0.0.0",
-                IncludeMainProject = true,
-                CompileReferences = new MockTaskItem[0],
-                ResolvedNuGetFiles = new MockTaskItem[0],
-                ResolvedRuntimeTargetsFiles = new MockTaskItem[0],
-                RuntimeGraphPath = ""
-            };
-
             // Execute task first time
+            var task = CreateTestTask();
             task.PublicExecuteCore();
             var firstWriteTime = File.GetLastWriteTimeUtc(_depsFilePath);
 
             // Wait a bit to ensure timestamp would change if file is rewritten
             Thread.Sleep(100);
 
             // Execute task again with same configuration
-            var task2 = new TestableGenerateDepsFile
+            var task2 = CreateTestTask();
+            task2.PublicExecuteCore();
+            var secondWriteTime = File.GetLastWriteTimeUtc(_depsFilePath);
+
+            // File should not have been rewritten when content is the same
+            secondWriteTime.Should().Be(firstWriteTime, "file should not be rewritten when content is unchanged");
+        }
+
+        private TestableGenerateDepsFile CreateTestTask()
+        {
+            return new TestableGenerateDepsFile
             {
                 BuildEngine = new MockNeverCacheBuildEngine4(),
                 ProjectPath = "TestProject.csproj",
@@ -64,12 +59,6 @@ public void ItDoesNotOverwriteFileWithSameContent()
                 ResolvedRuntimeTargetsFiles = new MockTaskItem[0],
                 RuntimeGraphPath = ""
             };
-
-            task2.PublicExecuteCore();
-            var secondWriteTime = File.GetLastWriteTimeUtc(_depsFilePath);
-
-            // File should not have been rewritten when content is the same
-            secondWriteTime.Should().Be(firstWriteTime, "file should not be rewritten when content is unchanged");
         }
 
         private class TestableGenerateDepsFile : GenerateDepsFile

src/Tasks/Microsoft.NET.Build.Tasks.UnitTests/GivenAGenerateRuntimeConfigurationFiles.cs

@@ -221,33 +221,26 @@ public void GivenTargetMonikerItGeneratesShortName()
         [Fact]
         public void ItDoesNotOverwriteFileWithSameContent()
         {
-            var task = new TestableGenerateRuntimeConfigurationFiles
-            {
-                BuildEngine = new MockNeverCacheBuildEngine4(),
-                TargetFrameworkMoniker = $".NETCoreApp,Version=v{ToolsetInfo.CurrentTargetFrameworkVersion}",
-                RuntimeConfigPath = _runtimeConfigPath,
-                RuntimeFrameworks = new[]
-                {
-                    new MockTaskItem(
-                        "Microsoft.NETCore.App",
-                        new Dictionary<string, string>
-                        {
-                            {"FrameworkName", "Microsoft.NETCore.App"}, {"Version", $"{ToolsetInfo.CurrentTargetFrameworkVersion}.0"}
-                        }
-                    )
-                },
-                RollForward = "LatestMinor"
-            };
-
             // Execute task first time
+            var task = CreateBasicTestTask();
             task.PublicExecuteCore();
             var firstWriteTime = File.GetLastWriteTimeUtc(_runtimeConfigPath);
 
             // Wait a bit to ensure timestamp would change if file is rewritten
             Thread.Sleep(100);
 
             // Execute task again with same configuration
-            var task2 = new TestableGenerateRuntimeConfigurationFiles
+            var task2 = CreateBasicTestTask();
+            task2.PublicExecuteCore();
+            var secondWriteTime = File.GetLastWriteTimeUtc(_runtimeConfigPath);
+
+            // File should not have been rewritten when content is the same
+            secondWriteTime.Should().Be(firstWriteTime, "file should not be rewritten when content is unchanged");
+        }
+
+        private TestableGenerateRuntimeConfigurationFiles CreateBasicTestTask()
+        {
+            return new TestableGenerateRuntimeConfigurationFiles
             {
                 BuildEngine = new MockNeverCacheBuildEngine4(),
                 TargetFrameworkMoniker = $".NETCoreApp,Version=v{ToolsetInfo.CurrentTargetFrameworkVersion}",
@@ -264,12 +257,6 @@ public void ItDoesNotOverwriteFileWithSameContent()
                 },
                 RollForward = "LatestMinor"
             };
-
-            task2.PublicExecuteCore();
-            var secondWriteTime = File.GetLastWriteTimeUtc(_runtimeConfigPath);
-
-            // File should not have been rewritten when content is the same
-            secondWriteTime.Should().Be(firstWriteTime, "file should not be rewritten when content is unchanged");
         }
 
         private class TestableGenerateRuntimeConfigurationFiles : GenerateRuntimeConfigurationFiles