Merge pull request #1 from pBouillon/add-flag-to-skip-tests

Allows for a puzzle's tests to be skipped

Author: pBouillon

src/AdventOfCode.Commons/AdventOfCode.Commons.csproj

@@ -8,6 +8,14 @@
     <IsPackable>false</IsPackable>
   </PropertyGroup>
 
+  <PropertyGroup Condition="'$(Configuration)|$(Platform)'=='Release|AnyCPU'">
+    <LangVersion>preview</LangVersion>
+  </PropertyGroup>
+
+  <PropertyGroup Condition="'$(Configuration)|$(Platform)'=='Debug|AnyCPU'">
+    <LangVersion>preview</LangVersion>
+  </PropertyGroup>
+
   <ItemGroup>
     <PackageReference Include="FluentAssertions" Version="6.8.0" />
     <PackageReference Include="Microsoft.NET.Test.Sdk" Version="17.3.2" />
@@ -20,6 +28,7 @@
       <IncludeAssets>runtime; build; native; contentfiles; analyzers; buildtransitive</IncludeAssets>
       <PrivateAssets>all</PrivateAssets>
     </PackageReference>
+    <PackageReference Include="Xunit.SkippableFact" Version="1.4.13" />
   </ItemGroup>
 
 </Project>

src/AdventOfCode.Commons/TestEngine.cs

@@ -47,6 +47,14 @@ public record Example
     /// </summary>
     public record Puzzle
     {
+        /// <summary>
+        /// If set to <c>true</c>, the tests for this puzzle will be skipped
+        /// </summary>
+        /// <remarks>
+        /// Default is <c>false</c>
+        /// </remarks>
+        public bool ShouldSkipTests { get; init; } = false;
+
         /// <summary>
         /// The <see cref="Example"/> to use to test <see cref="Solver{TInput, TResult}"/>
         /// </summary>
@@ -70,23 +78,33 @@ protected TestEngine()
 
     public abstract Puzzle PartOne { get; }
 
-    [Fact(DisplayName = "Part One - Example")]
+    [SkippableFact(DisplayName = "Part One - Example")]
     public void PartOneExampleTest()
     {
+        Skip.If(PartOne.ShouldSkipTests, "Puzzle.ShouldSkipTests has been set to true, test skipped");
+        
+        // Arrange
         var input = PartOne.Example.Input;
 
+        // Act
         var result = _solver.PartOne(input);
 
+        // Assert
         result.Should().Be(PartOne.Example.Result);
     }
 
-    [Fact(DisplayName = "Part One - Solution")]
+    [SkippableFact(DisplayName = "Part One - Solution")]
     public void PartOneSolutionTest()
     {
-        var input = _solver.Input;
+        Skip.If(PartOne.ShouldSkipTests, "Puzzle.ShouldSkipTests has been set to true, test skipped");
 
+        // Arrange
+        var input = _solver.Input;
+        
+        // Act
         var result = _solver.PartOne(input);
 
+        // Assert
         result.Should().Be(PartOne.Solution);
     }
 
@@ -96,23 +114,33 @@ public void PartOneSolutionTest()
 
     public abstract Puzzle PartTwo { get; }
 
-    [Fact(DisplayName = "Part Two - Example")]
+    [SkippableFact(DisplayName = "Part Two - Example")]
     public void PartTwoExampleTest()
     {
+        Skip.If(PartTwo.ShouldSkipTests, "Puzzle.ShouldSkipTests has been set to true, test skipped");
+
+        // Arrange
         var input = PartTwo.Example.Input;
 
+        // Act
         var result = _solver.PartTwo(input);
 
+        // Assert
         result.Should().Be(PartTwo.Example.Result);
     }
 
-    [Fact(DisplayName = "Part Two - Solution")]
+    [SkippableFact(DisplayName = "Part Two - Solution")]
     public void PartTwoSolutionTest()
     {
+        Skip.If(PartTwo.ShouldSkipTests, "Puzzle.ShouldSkipTests has been set to true, test skipped");
+
+        // Arrange
         var input = _solver.Input;
 
+        // Act
         var result = _solver.PartTwo(input);
 
+        // Assert
         result.Should().Be(PartTwo.Solution);
     }
 

src/AdventOfCode.Usage/SolverTest.cs

@@ -17,7 +17,6 @@ public class SolverTest : TestEngine<Solver, int[], int>
             Input = new[] { 1, 2, 3 },
             Result = 3,
         },
-
         Solution = 5,
     };
 
@@ -28,7 +27,6 @@ public class SolverTest : TestEngine<Solver, int[], int>
             Input = new[] { 1, 2, 3 },
             Result = 6,
         },
-
         Solution = 15,
     };
 }