Add Test target to execution plan

MRmlik12 · May 11, 2024 · 6340997 · 6340997
1 parent 523e1aa
commit 6340997
Show file tree

Hide file tree

Showing 3 changed files with 23 additions and 7 deletions.
diff --git a/.github/workflows/build.yml b/.github/workflows/build.yml
@@ -14,7 +14,7 @@ jobs:
           restore-keys: |
             ${{ runner.os }}-nuget- 
       - name: Run NUKE
-        run: .\build.ps1
+        run: .\build.ps1 --skip-tests --configuration Release
       - name: Upload artifacts
         uses: actions/upload-artifact@v4
         with:
@@ -33,7 +33,7 @@ jobs:
           restore-keys: |
             ${{ runner.os }}-nuget- 
       - name: Run NUKE
-        run: chmod +x build.sh && ./build.sh --configuration Release
+        run: chmod +x build.sh && ./build.sh --skip-tests --configuration Release
       - name: Upload artifacts
         uses: actions/upload-artifact@v4
         with:
@@ -52,7 +52,7 @@ jobs:
           restore-keys: |
             ${{ runner.os }}-nuget- 
       - name: Run NUKE
-        run: chmod +x build.sh && ./build.sh --configuration Release
+        run: chmod +x build.sh && ./build.sh --skip-tests --configuration Release
       - name: Upload artifacts
         uses: actions/upload-artifact@v4
         with:
@@ -71,7 +71,7 @@ jobs:
           restore-keys: |
             ${{ runner.os }}-nuget- 
       - name: Run NUKE
-        run: chmod +x build.sh && ./build.sh CompileBrowser --allow-browser-build --allow-desktop-build false --configuration Release
+        run: chmod +x build.sh && ./build.sh CompileBrowser --skip-tests --allow-browser-build --allow-desktop-build false --configuration Release
         env:
           NetlifySiteId: ${{ secrets.NETLIFY_SITE_ID }}
           NetlifyAccessToken: ${{ secrets.NETLIFY_ACCESS_TOKEN }}

diff --git a/.nuke/build.schema.json b/.nuke/build.schema.json
@@ -101,10 +101,15 @@
               "CompileBrowser",
               "Pack",
               "Restore",
-              "Setup"
+              "Setup",
+              "Test"
             ]
           }
         },
+        "SkipTests": {
+          "type": "boolean",
+          "description": "Skip tests"
+        },
         "Solution": {
           "type": "string",
           "description": "Path to a solution file that is automatically loaded"
@@ -120,7 +125,8 @@
               "CompileBrowser",
               "Pack",
               "Restore",
-              "Setup"
+              "Setup",
+              "Test"
             ]
           }
         },

diff --git a/build/Build.cs b/build/Build.cs
@@ -31,6 +31,8 @@ class Build : NukeBuild
 
     [Parameter("Allow browser build")] readonly bool AllowBrowserBuild;
 
+    [Parameter("Skip tests")] readonly bool SkipTests;
+
     [Parameter("Artifacts path")] readonly AbsolutePath ArtifactsPath = RootDirectory / "artifacts";
 
     [Parameter("Configuration to build - Default is 'Debug' (local) or 'Release' (server)")]
@@ -117,9 +119,17 @@ class Build : NukeBuild
                 .SetConfiguration(Configuration)
                 .SetProjectFile(Solution.AllProjects.Single(x => x.Name == "NoteSHR.Browser").Path));
         });
+
+    Target Test => _ => _
+        .DependsOn(Restore)
+        .OnlyWhenDynamic(() => !SkipTests)
+        .Executes(() =>
+        {
+            DotNetTasks.DotNetTest();
+        });
 
     Target Compile => _ => _
-        .DependsOn(Restore)
+        .DependsOn(Test)
         .Executes(() =>
         {
             var projects = Solution.AllProjects.Where(x => Platforms.Select(p => p.Name).Contains(x.Name));