Add SLNX Tests

Author: nicholass-alcidion

src/dotnet-affected/Commands/AffectedGlobalOptions.cs

@@ -28,7 +28,7 @@ internal static class AffectedGlobalOptions
             {
                 "--filter-file-path"
             },
-            description: "Path to a filter file (.sln) used to discover projects that may be affected.\n" +
+            description: "Path to a filter file (.sln, .slnx) used to discover projects that may be affected.\n" +
                          "When omitted, will search for project files inside --repository-path.");
 
         public static readonly Option<bool> VerboseOption = new(aliases: new[]

test/DotnetAffected.Core.Tests/ChangeDetectionUsingXmlSolutionTests.cs

@@ -0,0 +1,82 @@
+using DotnetAffected.Testing.Utils;
+using System.IO;
+using System.Linq;
+using System.Threading.Tasks;
+using Xunit;
+
+namespace DotnetAffected.Core.Tests
+{
+    /// <summary>
+    /// Tests for detecting changed projects when using a SolutionFile to filter.
+    /// This should cover all tests where filtering should be applied
+    /// </summary>
+    public class ChangeDetectionUsingXmlSolutionTests
+        : BaseDotnetAffectedTest
+    {
+        private readonly string _solutionPath = "test-solution.slnx";
+
+        protected override AffectedOptions Options => new AffectedOptions(
+            this.Repository.Path,
+            Path.Combine(this.Repository.Path, this._solutionPath));
+
+        [Fact]
+        public async Task When_project_inside_solution_has_changes_project_should_have_changed()
+        {
+            // Create a project
+            var projectName = "InventoryManagement";
+            var msBuildProject = this.Repository.CreateCsProject(projectName);
+
+            // Create a solution which includes the project
+            await this.Repository.CreateXmlSolutionAsync(_solutionPath, msBuildProject.FullPath);
+
+            Assert.Single(AffectedSummary.ProjectsWithChangedFiles);
+            Assert.Empty(AffectedSummary.AffectedProjects);
+
+            var projectInfo = AffectedSummary.ProjectsWithChangedFiles.Single();
+            Assert.Equal(projectName, projectInfo.GetProjectName());
+            Assert.Equal(msBuildProject.FullPath, projectInfo.GetFullPath());
+        }
+
+        [Fact]
+        public async Task When_project_inside_solution_should_ignore_changes_outside_solution()
+        {
+            // Create a project
+            var projectName = "InventoryManagement";
+            var msBuildProject = this.Repository.CreateCsProject(projectName);
+
+            // Create a solution which includes the project
+            await this.Repository.CreateXmlSolutionAsync(_solutionPath, msBuildProject.FullPath);
+
+            // Create a project that is outside the solution
+            var outsiderproject = "OutsiderProject";
+            this.Repository.CreateCsProject(outsiderproject);
+
+            Assert.Single(AffectedSummary.ProjectsWithChangedFiles);
+            Assert.Empty(AffectedSummary.AffectedProjects);
+
+            var projectInfo = AffectedSummary.ProjectsWithChangedFiles.Single();
+            Assert.Equal(projectName, projectInfo.GetProjectName());
+            Assert.Equal(msBuildProject.FullPath, projectInfo.GetFullPath());
+        }
+
+        [Fact]
+        public async Task When_project_outside_solution_has_changed_nothing_should_be_affected()
+        {
+            // Create a project
+            var projectName = "InventoryManagement";
+            var msBuildProject = this.Repository.CreateCsProject(projectName);
+
+            // Create a solution which includes the project
+            await this.Repository.CreateXmlSolutionAsync(_solutionPath, msBuildProject.FullPath);
+
+            // Commit so there are no changes
+            this.Repository.StageAndCommit();
+
+            // Create a project that is outside the solution
+            var outsiderName = "OutsiderProject";
+            this.Repository.CreateCsProject(outsiderName);
+
+            Assert.Empty(AffectedSummary.AffectedProjects);
+        }
+    }
+}

test/DotnetAffected.Testing.Utils/Repository/TemporaryRepositoryExtensions.cs

@@ -149,6 +149,20 @@ public static async Task CreateSolutionAsync(
             await File.WriteAllTextAsync(solutionPath, solutionContents);
         }
 
+        public static async Task CreateXmlSolutionAsync(
+            this TemporaryRepository repo,
+            string solutionName,
+            params string[] projectPaths)
+        {
+            var i = 0;
+            var solutionContents = new SolutionFileBuilder { Projects = projectPaths.ToDictionary(p => i++.ToString()) }
+                .BuildXmlSolution();
+
+            var solutionPath = Path.Combine(repo.Path, solutionName);
+
+            await File.WriteAllTextAsync(solutionPath, solutionContents);
+        }
+
         public static Task CreateTraversalProjectAsync(
             this TemporaryRepository repo,
             string traversalProjectPath,

test/DotnetAffected.Testing.Utils/SolutionFileBuilder.cs

@@ -97,5 +97,26 @@ public string BuildSolution()
 
             return sb.ToString();
         }
+
+        public string BuildXmlSolution()
+        {
+            var sb = new StringBuilder();
+            sb.AppendLine(@"<Solution>");
+            sb.AppendLine(@"  <Configurations>");
+            sb.AppendLine(@"    <Platform Name=""Any CPU"" />");
+            sb.AppendLine(@"    <Platform Name=""x64"" />");
+            sb.AppendLine(@"    <Platform Name=""x86"" />");
+            sb.AppendLine(@"  </Configurations>");
+
+            foreach (var project in Projects.Values)
+            {
+                sb.AppendLine(@$"  <Project Path=""{project}"" />");
+
+            }
+
+            sb.AppendLine(@"</Solution>");
+
+            return sb.ToString();
+        }
     }
 }