From 0a3c5229e4c63585991b2fdae64bd046260549af Mon Sep 17 00:00:00 2001
From: Julien Jomier <jjomier@nvidia.com>
Date: Tue, 26 May 2026 20:10:02 -0400
Subject: [PATCH 1/4] Handle skipped tests in "Not run" column

This commit introduces a new `notRunSkippedDetailsRegex` property to the Project model, allowing users to define regex patterns for not-run test details. A corresponding validation rule is added to ensure the regex is valid. The Build model now includes a method to count not-run tests that do not match the specified patterns, and the GraphQL schema is updated to expose this functionality. Additionally, UI components are modified to display the warning count and adjust test status colors based on the new regex patterns. Tests are also added to verify the new functionality.
---
 .../UpdateProjectInputValidator.php           |   6 +
 app/Models/Build.php                          |  19 +++
 app/Models/Project.php                        |   4 +
 app/Rules/NotRunSkippedDetailsRegexRule.php   |  26 ++++
 app/Utils/TestDisplay.php                     | 142 ++++++++++++++++++
 app/cdash/app/Controller/Api/Index.php        |  14 ++
 app/cdash/app/Controller/Api/QueryTests.php   |   8 +-
 app/cdash/tests/test_testhistory.php          |   3 +
 ...otrun_skipped_details_regex_to_project.php |  15 ++
 graphql/schema.graphql                        |  19 +++
 .../js/angular/views/partials/build.html      |   2 +-
 resources/js/vue/components/BuildSummary.vue  |   9 +-
 .../js/vue/components/BuildTestsPage.vue      |  33 ++--
 .../components/ProjectSettings/GeneralTab.vue |  13 +-
 .../js/vue/components/ProjectSettingsPage.vue |  17 +++
 .../js/vue/components/TestDetailsPage.vue     |  16 ++
 .../components/shared/BuildSummaryCard.vue    |  28 +++-
 .../js/vue/components/shared/FormSection.vue  |  11 +-
 .../js/vue/components/shared/TestDisplay.js   |  99 ++++++++++++
 tests/Browser/Pages/BuildTestsPageTest.php    |  46 ++++++
 tests/Feature/GraphQL/BuildTypeTest.php       |  54 +++++++
 .../GraphQL/Mutations/UpdateProjectTest.php   |  26 ++++
 tests/Feature/GraphQL/ProjectTypeTest.php     |   2 +
 tests/Unit/Utils/TestDisplayTest.php          |  82 ++++++++++
 24 files changed, 665 insertions(+), 29 deletions(-)
 create mode 100644 app/Rules/NotRunSkippedDetailsRegexRule.php
 create mode 100644 app/Utils/TestDisplay.php
 create mode 100644 database/migrations/2026_05_26_120000_add_notrun_skipped_details_regex_to_project.php
 create mode 100644 resources/js/vue/components/shared/TestDisplay.js
 create mode 100644 tests/Unit/Utils/TestDisplayTest.php

diff --git a/app/GraphQL/Validators/UpdateProjectInputValidator.php b/app/GraphQL/Validators/UpdateProjectInputValidator.php
index 0dc314ed3a..9d176a6436 100644
--- a/app/GraphQL/Validators/UpdateProjectInputValidator.php
+++ b/app/GraphQL/Validators/UpdateProjectInputValidator.php
@@ -3,6 +3,7 @@
 namespace App\GraphQL\Validators;
 
 use App\Models\Project;
+use App\Rules\NotRunSkippedDetailsRegexRule;
 use App\Rules\ProjectAuthenticateSubmissionsRule;
 use App\Rules\ProjectNameRule;
 use App\Rules\ProjectVisibilityRule;
@@ -25,6 +26,11 @@ public function rules(): array
             'authenticateSubmissions' => [
                 new ProjectAuthenticateSubmissionsRule(),
             ],
+            'notRunSkippedDetailsRegex' => [
+                'nullable',
+                'string',
+                new NotRunSkippedDetailsRegexRule(),
+            ],
         ];
     }
 
diff --git a/app/Models/Build.php b/app/Models/Build.php
index a0d8fe0185..98402a0684 100644
--- a/app/Models/Build.php
+++ b/app/Models/Build.php
@@ -2,6 +2,7 @@
 
 namespace App\Models;
 
+use App\Utils\TestDisplay;
 use Illuminate\Database\Eloquent\Builder;
 use Illuminate\Database\Eloquent\Model;
 use Illuminate\Database\Eloquent\Relations\BelongsTo;
@@ -400,4 +401,22 @@ public function percentCoverageForPath(string $path): ?float
 
         return ($loctested / $total_lines) * 100;
     }
+
+    /**
+     * The number of not-run tests whose details do not match the project's skipped pattern.
+     */
+    public function notRunTestsWarningCount(): int
+    {
+        $patterns = $this->project?->notrun_skipped_details_regex
+            ?? TestDisplay::DEFAULT_NOTRUN_SKIPPED_DETAILS_REGEX;
+
+        $count = 0;
+        foreach ($this->tests()->where('status', Test::NOTRUN)->pluck('details') as $details) {
+            if (!TestDisplay::isAcceptableNotRun($details, $patterns)) {
+                $count++;
+            }
+        }
+
+        return $count;
+    }
 }
diff --git a/app/Models/Project.php b/app/Models/Project.php
index 11b7987ed4..c102b9200e 100644
--- a/app/Models/Project.php
+++ b/app/Models/Project.php
@@ -3,6 +3,7 @@
 namespace App\Models;
 
 use App\Enums\ProjectRole;
+use App\Utils\TestDisplay;
 use Illuminate\Database\Eloquent\Builder;
 use Illuminate\Database\Eloquent\Casts\Attribute;
 use Illuminate\Database\Eloquent\Model;
@@ -49,6 +50,7 @@
  * @property ?string $banner
  * @property ?string $logoUrl
  * @property ?string $cmakeprojectroot
+ * @property string $notrun_skipped_details_regex
  *
  * @method Builder<Project> forUser()
  * @method Builder<Project> administeredByUser()
@@ -95,6 +97,7 @@ class Project extends Model
         'ldapfilter',
         'banner',
         'cmakeprojectroot',
+        'notrun_skipped_details_regex',
     ];
 
     protected $casts = [
@@ -137,6 +140,7 @@ class Project extends Model
         'uploadquota' => 10,
         'showcoveragecode' => true,
         'authenticatesubmissions' => false,
+        'notrun_skipped_details_regex' => TestDisplay::DEFAULT_NOTRUN_SKIPPED_DETAILS_REGEX,
     ];
 
     public const ACCESS_PRIVATE = 0;
diff --git a/app/Rules/NotRunSkippedDetailsRegexRule.php b/app/Rules/NotRunSkippedDetailsRegexRule.php
new file mode 100644
index 0000000000..d6d9c402dd
--- /dev/null
+++ b/app/Rules/NotRunSkippedDetailsRegexRule.php
@@ -0,0 +1,26 @@
+<?php
+
+namespace App\Rules;
+
+use App\Utils\TestDisplay;
+use Closure;
+use Illuminate\Contracts\Validation\ValidationRule;
+
+final class NotRunSkippedDetailsRegexRule implements ValidationRule
+{
+    public function validate(string $attribute, mixed $value, Closure $fail): void
+    {
+        if ($value === null || $value === '') {
+            return;
+        }
+
+        if (!is_string($value)) {
+            $fail('The skipped test details pattern must be a string.');
+            return;
+        }
+
+        if (!TestDisplay::isValidPatternsText($value)) {
+            $fail('The skipped test details pattern contains an invalid regular expression.');
+        }
+    }
+}
diff --git a/app/Utils/TestDisplay.php b/app/Utils/TestDisplay.php
new file mode 100644
index 0000000000..a4e67670c7
--- /dev/null
+++ b/app/Utils/TestDisplay.php
@@ -0,0 +1,142 @@
+<?php
+
+namespace App\Utils;
+
+use App\Models\Test;
+
+final class TestDisplay
+{
+    public const DEFAULT_NOTRUN_SKIPPED_DETAILS_REGEX = '*skip*';
+
+    /**
+     * @return list<string>
+     */
+    public static function parsePatterns(?string $patternsText): array
+    {
+        if ($patternsText === null || trim($patternsText) === '') {
+            return [];
+        }
+
+        $patterns = [];
+        foreach (preg_split("/\r\n|\n|\r/", $patternsText) as $line) {
+            $line = trim($line);
+            if ($line !== '') {
+                $patterns[] = $line;
+            }
+        }
+
+        return $patterns;
+    }
+
+    public static function patternToPcre(string $pattern): string
+    {
+        $pattern = trim($pattern);
+        if ($pattern === '') {
+            return '';
+        }
+
+        if (str_starts_with($pattern, '/')) {
+            return $pattern;
+        }
+
+        $parts = preg_split('/(\*|\?)/', $pattern, -1, PREG_SPLIT_DELIM_CAPTURE);
+        if ($parts === false) {
+            return '';
+        }
+
+        $regex = '';
+        foreach ($parts as $part) {
+            if ($part === '*') {
+                $regex .= '.*';
+            } elseif ($part === '?') {
+                $regex .= '.';
+            } else {
+                $regex .= preg_quote($part, '/');
+            }
+        }
+
+        return '/(?i)' . $regex . '/';
+    }
+
+    public static function detailsMatchesSkippedPattern(?string $details, ?string $patternsText): bool
+    {
+        if ($details === null || $details === '') {
+            return false;
+        }
+
+        foreach (self::parsePatterns($patternsText) as $pattern) {
+            $pcre = self::patternToPcre($pattern);
+            if ($pcre === '') {
+                continue;
+            }
+
+            $result = @preg_match($pcre, $details);
+            if ($result === 1) {
+                return true;
+            }
+        }
+
+        return false;
+    }
+
+    public static function isAcceptableNotRun(?string $details, ?string $patternsText): bool
+    {
+        return self::detailsMatchesSkippedPattern($details, $patternsText);
+    }
+
+    public static function statusColorClass(string $status, ?string $details, ?string $patternsText): string
+    {
+        if ($status === Test::NOTRUN && self::isAcceptableNotRun($details, $patternsText)) {
+            return 'normal';
+        }
+
+        return match ($status) {
+            Test::PASSED => 'normal',
+            Test::FAILED => 'error',
+            Test::NOTRUN => 'warning',
+            default => '',
+        };
+    }
+
+    public static function statusTextColorClass(string $status, ?string $details, ?string $patternsText): string
+    {
+        return match (self::statusColorClass($status, $details, $patternsText)) {
+            'normal' => 'normal-text',
+            'warning' => 'warning-text',
+            'error' => 'error-text',
+            default => '',
+        };
+    }
+
+    public static function graphqlStatusColorClass(string $status, ?string $details, ?string $patternsText): string
+    {
+        $dbStatus = match ($status) {
+            'NOT_RUN' => Test::NOTRUN,
+            'PASSED' => Test::PASSED,
+            'FAILED' => Test::FAILED,
+            default => strtolower($status),
+        };
+
+        return self::statusColorClass($dbStatus, $details, $patternsText);
+    }
+
+    public static function isValidPatternsText(?string $patternsText): bool
+    {
+        foreach (self::parsePatterns($patternsText) as $pattern) {
+            $pcre = self::patternToPcre($pattern);
+            if ($pcre === '') {
+                return false;
+            }
+
+            set_error_handler(static fn (): bool => true);
+            $result = preg_match($pcre, '');
+            restore_error_handler();
+
+            if ($result === false) {
+                return false;
+            }
+        }
+
+        return true;
+    }
+}
diff --git a/app/cdash/app/Controller/Api/Index.php b/app/cdash/app/Controller/Api/Index.php
index b70f94fd90..7542f771bf 100644
--- a/app/cdash/app/Controller/Api/Index.php
+++ b/app/cdash/app/Controller/Api/Index.php
@@ -17,6 +17,7 @@
 
 namespace CDash\Controller\Api;
 
+use App\Models\Build as EloquentBuild;
 use App\Utils\TestingDay;
 use CDash\Database;
 use CDash\Model\BuildGroup;
@@ -1098,6 +1099,7 @@ public function generateBuildResponseFromRow(array $build_array): array|false
             }
 
             $test_response['notrun'] = $nnotrun;
+            $test_response['notrunwarning'] = $this->computeNotRunTestsWarningCount((int) $buildid, $nnotrun);
             $test_response['fail'] = $nfail;
             $test_response['pass'] = $npass;
 
@@ -1555,4 +1557,16 @@ public function recordGenerationTime(array &$response): void
     {
         $this->pageTimer->end($response);
     }
+
+    /**
+     * Count not-run tests that should display as warnings on the index page.
+     */
+    private function computeNotRunTestsWarningCount(int $buildid, int $notRunCount): int
+    {
+        if ($notRunCount <= 0) {
+            return 0;
+        }
+
+        return EloquentBuild::find($buildid)?->notRunTestsWarningCount() ?? 0;
+    }
 }
diff --git a/app/cdash/app/Controller/Api/QueryTests.php b/app/cdash/app/Controller/Api/QueryTests.php
index 7f00f75295..0899abe869 100644
--- a/app/cdash/app/Controller/Api/QueryTests.php
+++ b/app/cdash/app/Controller/Api/QueryTests.php
@@ -20,6 +20,7 @@
 use App\Models\PinnedTestMeasurement;
 use App\Models\Project as EloquentProject;
 use App\Models\TestMeasurement;
+use App\Utils\TestDisplay;
 use CDash\Database;
 use CDash\Model\Build;
 use CDash\Model\Project;
@@ -375,6 +376,7 @@ public function getResponse(): array
                 ", $query_params);
 
         // Rows of test data to be displayed to the user.
+        $notRunSkippedDetailsRegex = EloquentProject::findOrFail($this->project->Id)->notrun_skipped_details_regex;
         $tests = [];
         foreach ($rows as $row) {
             $test = [];
@@ -423,7 +425,11 @@ public function getResponse(): array
 
                 case 'notrun':
                     $test['status'] = 'Not Run';
-                    $test['statusclass'] = 'warning';
+                    $test['statusclass'] = TestDisplay::statusColorClass(
+                        'notrun',
+                        $row->details,
+                        $notRunSkippedDetailsRegex,
+                    );
                     break;
             }
 
diff --git a/app/cdash/tests/test_testhistory.php b/app/cdash/tests/test_testhistory.php
index 7f06fb55fc..e9bd2f3a24 100644
--- a/app/cdash/tests/test_testhistory.php
+++ b/app/cdash/tests/test_testhistory.php
@@ -134,6 +134,9 @@ private function generateXML($mm, $timestamp, $include_sporadic, $flaky_passed)
                   <FullName>./notrun</FullName>
                   <FullCommandLine></FullCommandLine>
                   <Results>
+                    <NamedMeasurement type="text/string" name="Exit Code">
+                      <Value>Skipped</Value>
+                    </NamedMeasurement>
                     <NamedMeasurement type="text/string" name="Command Line">
                       <Value></Value>
                     </NamedMeasurement>
diff --git a/database/migrations/2026_05_26_120000_add_notrun_skipped_details_regex_to_project.php b/database/migrations/2026_05_26_120000_add_notrun_skipped_details_regex_to_project.php
new file mode 100644
index 0000000000..e2a61e4547
--- /dev/null
+++ b/database/migrations/2026_05_26_120000_add_notrun_skipped_details_regex_to_project.php
@@ -0,0 +1,15 @@
+<?php
+
+use Illuminate\Database\Migrations\Migration;
+use Illuminate\Support\Facades\DB;
+
+return new class extends Migration {
+    public function up(): void
+    {
+        DB::statement("ALTER TABLE project ADD COLUMN notrun_skipped_details_regex text NOT NULL DEFAULT '*skip*'");
+    }
+
+    public function down(): void
+    {
+    }
+};
diff --git a/graphql/schema.graphql b/graphql/schema.graphql
index 1eab0ea949..ba63fd858d 100644
--- a/graphql/schema.graphql
+++ b/graphql/schema.graphql
@@ -398,6 +398,13 @@ type Project {
   "Invitations to this project which have not been accepted yet."
   invitations: [ProjectInvitation!]! @canRoot(ability: "inviteUser") @hasMany(type: CONNECTION) @orderBy(column: "id")
 
+  """
+  Regular expression(s), one per line, matched against not-run test details. Matching tests are
+  displayed in green instead of yellow. Glob wildcards (*) are supported. The default pattern is
+  *skip* (case insensitive).
+  """
+  notRunSkippedDetailsRegex: String! @rename(attribute: "notrun_skipped_details_regex")
+
   """
   Pinned test measurements are test measurements, identified by name, which should be shown
   alongside Test details throughout the site.
@@ -515,6 +522,12 @@ input UpdateProjectInput @validator {
   showCoverageCode: Boolean @rename(attribute: "showcoveragecode")
 
   banner: String
+
+  """
+  Regular expression(s), one per line, matched against not-run test details. Matching tests are
+  displayed in green instead of yellow.
+  """
+  notRunSkippedDetailsRegex: String @rename(attribute: "notrun_skipped_details_regex")
 }
 
 
@@ -856,6 +869,12 @@ type Build {
   "The number of tests not run for this build."
   notRunTestsCount: Int @rename(attribute: "testnotrun")
 
+  """
+  The number of not-run tests counted as warnings (excludes tests whose details match the
+  project's skipped-test pattern).
+  """
+  notRunTestsWarningCount: Int! @method(name: "notRunTestsWarningCount")
+
   "The duration of the test step in seconds."
   testDuration: Int! @rename(attribute: "testduration")
 
diff --git a/resources/js/angular/views/partials/build.html b/resources/js/angular/views/partials/build.html
index ed5b8dc1a9..ca631fbcd8 100644
--- a/resources/js/angular/views/partials/build.html
+++ b/resources/js/angular/views/partials/build.html
@@ -329,7 +329,7 @@
   </div>
 </td>
 
-<td ng-if="::buildgroup.hastestdata" align="center" ng-class="::{'warning': build.test.notrun > 0, 'normal': build.test.notrun == 0}">
+<td ng-if="::buildgroup.hastestdata" align="center" ng-class="::{'warning': build.test.notrunwarning > 0, 'normal': build.test.notrunwarning == 0}">
   <div ng-if="::build.hastest"
        ng-class="::{'valuewithsub': build.test.nnotrundiffp > 0 || build.test.nnotrundiffn > 0}">
     <a class="cdash-link" ng-href="{{ 'builds/' + build.id + '/tests?filters=%7B%22all%22%3A%5B%7B%22eq%22%3A%7B%22status%22%3A%22NOT_RUN%22%7D%7D%5D%7D' }}">
diff --git a/resources/js/vue/components/BuildSummary.vue b/resources/js/vue/components/BuildSummary.vue
index 373471eaec..05ff0daa11 100644
--- a/resources/js/vue/components/BuildSummary.vue
+++ b/resources/js/vue/components/BuildSummary.vue
@@ -516,6 +516,7 @@ export default {
             buildWarningsCount
             failedTestsCount
             notRunTestsCount
+            notRunTestsWarningCount
             site {
               id
               name
@@ -539,6 +540,7 @@ export default {
             buildWarningsCount
             failedTestsCount
             notRunTestsCount
+            notRunTestsWarningCount
           }
           nextBuild: build(id: $nextId) @include(if: $hasNext) {
             id
@@ -548,6 +550,7 @@ export default {
             buildWarningsCount
             failedTestsCount
             notRunTestsCount
+            notRunTestsWarningCount
           }
         }
       `,
@@ -576,7 +579,7 @@ export default {
             nerrors: Math.max(0, prev.buildErrorsCount),
             nwarnings: Math.max(0, prev.buildWarningsCount),
             ntestfailed: Math.max(0, prev.failedTestsCount),
-            ntestnotrun: Math.max(0, prev.notRunTestsCount),
+            ntestnotrun: Math.max(0, prev.notRunTestsWarningCount),
           };
         } else {
           this.cdash.previousbuild = null;
@@ -591,7 +594,7 @@ export default {
             nerrors: Math.max(0, next.buildErrorsCount),
             nwarnings: Math.max(0, next.buildWarningsCount),
             ntestfailed: Math.max(0, next.failedTestsCount),
-            ntestnotrun: Math.max(0, next.notRunTestsCount),
+            ntestnotrun: Math.max(0, next.notRunTestsWarningCount),
           };
         } else {
           this.cdash.nextbuild = null;
@@ -616,7 +619,7 @@ export default {
 
         this.cdash.test = {
           nfailed: Math.max(0, build.failedTestsCount),
-          nnotrun: Math.max(0, build.notRunTestsCount),
+          nnotrun: Math.max(0, build.notRunTestsWarningCount),
         };
 
         this.cdash.projectname_encoded = encodeURIComponent(build.project.name);
diff --git a/resources/js/vue/components/BuildTestsPage.vue b/resources/js/vue/components/BuildTestsPage.vue
index 788c83b846..7422904b7b 100644
--- a/resources/js/vue/components/BuildTestsPage.vue
+++ b/resources/js/vue/components/BuildTestsPage.vue
@@ -73,6 +73,7 @@ import LoadingIndicator from './shared/LoadingIndicator.vue';
 import BuildSummaryCard from './shared/BuildSummaryCard.vue';
 import BuildSidebar from './shared/BuildSidebar.vue';
 import { DateTime } from 'luxon';
+import { testStatusToColorClass } from './shared/TestDisplay';
 
 const TEST_QUERY = gql`
   query(
@@ -82,6 +83,9 @@ const TEST_QUERY = gql`
   ) {
     build(id: $buildid) {
       id
+      project {
+        notRunSkippedDetailsRegex
+      }
       tests(filters: $filters, first: 1000000) {
         edges {
           node {
@@ -205,6 +209,11 @@ export default {
         });
         return tests;
       },
+      result({ data }) {
+        if (data?.build?.project?.notRunSkippedDetailsRegex !== undefined) {
+          this.notRunSkippedDetailsRegex = data.build.project.notRunSkippedDetailsRegex;
+        }
+      },
       variables() {
         return {
           buildid: this.buildId,
@@ -237,6 +246,11 @@ export default {
         });
         return tests;
       },
+      result({ data }) {
+        if (data?.build?.project?.notRunSkippedDetailsRegex !== undefined) {
+          this.notRunSkippedDetailsRegex = data.build.project.notRunSkippedDetailsRegex;
+        }
+      },
       variables() {
         return {
           buildid: this.previousBuildId,
@@ -259,6 +273,7 @@ export default {
   data() {
     return {
       changedFilters: JSON.parse(JSON.stringify(this.initialFilters)),
+      notRunSkippedDetailsRegex: '*skip*',
     };
   },
 
@@ -308,6 +323,7 @@ export default {
     },
 
     formattedTestRows() {
+      const notRunSkippedDetailsRegex = this.notRunSkippedDetailsRegex;
       return this.filteredTests?.map((edge) => {
         return {
           name: {
@@ -325,14 +341,14 @@ export default {
             value: edge.node.status,
             text: this.humanReadableTestStatus(edge.node.status),
             href: `${this.$baseURL}/tests/${edge.node.id}`,
-            classes: [this.testStatusToColorClass(edge.node.status)],
+            classes: [testStatusToColorClass(edge.node.status, edge.node.details, notRunSkippedDetailsRegex)],
           },
           subProject: edge.subProject ?? '',
           timeStatus: {
             value: edge.node.timeStatusCategory,
             text: this.humanReadableTestStatus(edge.node.timeStatusCategory),
             href: `${this.$baseURL}/tests/${edge.node.id}?graph=time`,
-            classes: [this.testStatusToColorClass(edge.node.timeStatusCategory)],
+            classes: [testStatusToColorClass(edge.node.timeStatusCategory, edge.node.details, notRunSkippedDetailsRegex)],
           },
           history: {
             value: '',
@@ -349,19 +365,6 @@ export default {
   },
 
   methods: {
-    testStatusToColorClass(status) {
-      switch (status) {
-        case 'PASSED':
-          return 'normal';
-        case 'FAILED':
-          return 'error';
-        case 'NOT_RUN':
-          return 'warning';
-        default:
-          return '';
-      }
-    },
-
     humanReadableTestStatus(status) {
       switch (status) {
         case 'PASSED':
diff --git a/resources/js/vue/components/ProjectSettings/GeneralTab.vue b/resources/js/vue/components/ProjectSettings/GeneralTab.vue
index 2e68a274c6..96d9468323 100644
--- a/resources/js/vue/components/ProjectSettings/GeneralTab.vue
+++ b/resources/js/vue/components/ProjectSettings/GeneralTab.vue
@@ -362,7 +362,8 @@
       </FormSection>
 
       <FormSection
-        title="Tests"
+        title="Testing"
+        section-id="Testing"
       >
         <CheckboxField
           v-model="form.enableTestTiming"
@@ -416,6 +417,14 @@
           min="0"
           test-id="test-time-std-multiplier-input"
         />
+
+        <TextAreaField
+          v-model="form.notRunSkippedDetailsRegex"
+          :validation-error="validationErrors?.notRunSkippedDetailsRegex?.[0]"
+          label="Skipped Not Run Test Details Pattern"
+          description="One pattern per line. When a not-run test's details match a pattern, it is shown in green. Glob wildcards (*) are supported; the default *skip* matches skip case-insensitively."
+          test-id="not-run-skipped-details-regex-input"
+        />
       </FormSection>
 
       <FormSection title="">
@@ -541,6 +550,7 @@ export default {
         fileUploadLimit: 50,
         showCoverageCode: true,
         banner: '',
+        notRunSkippedDetailsRegex: '*skip*',
       },
       validationErrors: {},
       fatalError: null,
@@ -584,6 +594,7 @@ export default {
             fileUploadLimit
             showCoverageCode
             banner
+            notRunSkippedDetailsRegex
           }
         }
       `,
diff --git a/resources/js/vue/components/ProjectSettingsPage.vue b/resources/js/vue/components/ProjectSettingsPage.vue
index 66a6d89bc7..eeb4012426 100644
--- a/resources/js/vue/components/ProjectSettingsPage.vue
+++ b/resources/js/vue/components/ProjectSettingsPage.vue
@@ -97,5 +97,22 @@ export default {
       currentSection: 'general',
     };
   },
+
+  mounted() {
+    this.navigateToHashSection();
+  },
+
+  methods: {
+    navigateToHashSection() {
+      if (window.location.hash !== '#Testing') {
+        return;
+      }
+
+      this.currentSection = 'general';
+      this.$nextTick(() => {
+        document.getElementById('Testing')?.scrollIntoView();
+      });
+    },
+  },
 };
 </script>
diff --git a/resources/js/vue/components/TestDetailsPage.vue b/resources/js/vue/components/TestDetailsPage.vue
index 65cfeb8f93..f6a4309d5e 100644
--- a/resources/js/vue/components/TestDetailsPage.vue
+++ b/resources/js/vue/components/TestDetailsPage.vue
@@ -334,6 +334,7 @@ import {
   faChartLine,
   faLink,
 } from '@fortawesome/free-solid-svg-icons';
+import { detailsMatchesSkippedPattern } from './shared/TestDisplay';
 
 export default {
   name: 'TestDetails',
@@ -382,6 +383,7 @@ export default {
               enableTestTiming
               testTimeStdMultiplier
               testTimeStdThreshold
+              notRunSkippedDetailsRegex
             }
           }
         }
@@ -517,6 +519,20 @@ export default {
     },
 
     testStatusPillClass() {
+      if (!this.test) {
+        return 'tw-bg-neutral tw-text-neutral-content';
+      }
+
+      if (
+        this.test.status === 'NOT_RUN'
+        && detailsMatchesSkippedPattern(
+          this.test.details,
+          this.build?.project?.notRunSkippedDetailsRegex ?? '*skip*',
+        )
+      ) {
+        return 'tw-bg-success tw-text-success-content';
+      }
+
       switch (this.test.status) {
         case 'PASSED':
           return 'tw-bg-success tw-text-success-content';
diff --git a/resources/js/vue/components/shared/BuildSummaryCard.vue b/resources/js/vue/components/shared/BuildSummaryCard.vue
index ee46268fe1..5dfe2b74b7 100644
--- a/resources/js/vue/components/shared/BuildSummaryCard.vue
+++ b/resources/js/vue/components/shared/BuildSummaryCard.vue
@@ -348,6 +348,7 @@ export default {
             passedTestsCount
             failedTestsCount
             notRunTestsCount
+            notRunTestsWarningCount
             site {
               id
               name
@@ -455,9 +456,15 @@ export default {
         return 'No Submission';
       }
 
+      const notRunWarningCount = Math.max(0, this.build.notRunTestsWarningCount);
+      const notRunSkippedCount = Math.max(0, this.build.notRunTestsCount - notRunWarningCount);
+
       let retval = '';
-      if (this.build.notRunTestsCount > 0) {
-        retval += `${this.build.notRunTestsCount} Not Run${this.commaSeparator(this.build.failedTestsCount > 0 || this.build.passedTestsCount)}`;
+      if (notRunWarningCount > 0) {
+        retval += `${notRunWarningCount} Not Run${this.commaSeparator(this.build.failedTestsCount > 0 || this.build.passedTestsCount > 0 || notRunSkippedCount > 0)}`;
+      }
+      if (notRunSkippedCount > 0) {
+        retval += `${notRunSkippedCount} Skipped${this.commaSeparator(this.build.failedTestsCount > 0 || this.build.passedTestsCount > 0)}`;
       }
       if (this.build.failedTestsCount > 0) {
         retval += `${this.build.failedTestsCount} Failed${this.commaSeparator(this.build.passedTestsCount)}`;
@@ -565,19 +572,26 @@ export default {
     testColor() {
       if (this.build.failedTestsCount > 0) {
         return 'tw-bg-red-400';
-      } else {
-        return 'tw-bg-green-400';
       }
+      if (this.build.notRunTestsWarningCount > 0) {
+        return 'tw-bg-orange-400';
+      }
+
+      return 'tw-bg-green-400';
     },
 
     testHighlightColor() {
       if (!this.hasTest) {
         return 'tw-border-x-gray-400';
-      } else if (this.build.failedTestsCount > 0) {
+      }
+      if (this.build.failedTestsCount > 0) {
         return 'tw-border-x-red-400';
-      } else {
-        return 'tw-border-x-green-400';
       }
+      if (this.build.notRunTestsWarningCount > 0) {
+        return 'tw-border-x-orange-400';
+      }
+
+      return 'tw-border-x-green-400';
     },
 
     /**
diff --git a/resources/js/vue/components/shared/FormSection.vue b/resources/js/vue/components/shared/FormSection.vue
index c95a864cb6..3061ef9487 100644
--- a/resources/js/vue/components/shared/FormSection.vue
+++ b/resources/js/vue/components/shared/FormSection.vue
@@ -1,5 +1,8 @@
 <template>
-  <section class="tw-container tw-mx-auto">
+  <section
+    :id="sectionId"
+    class="tw-container tw-mx-auto"
+  >
     <h4 class="tw-text-xl tw-font-semibold tw-border-b tw-pb-2 tw-mb-2">
       {{ title }}
     </h4>
@@ -18,6 +21,12 @@ export default {
       type: String,
       required: true,
     },
+
+    sectionId: {
+      type: String,
+      required: false,
+      default: null,
+    },
   },
 };
 </script>
diff --git a/resources/js/vue/components/shared/TestDisplay.js b/resources/js/vue/components/shared/TestDisplay.js
new file mode 100644
index 0000000000..5a0979a390
--- /dev/null
+++ b/resources/js/vue/components/shared/TestDisplay.js
@@ -0,0 +1,99 @@
+/**
+ * @param {string} pattern
+ * @returns {RegExp|null}
+ */
+export function patternToRegExp(pattern) {
+  const trimmed = pattern.trim();
+  if (trimmed === '') {
+    return null;
+  }
+
+  if (trimmed.startsWith('/')) {
+    const lastSlash = trimmed.lastIndexOf('/');
+    if (lastSlash > 0) {
+      const body = trimmed.slice(1, lastSlash);
+      const flags = trimmed.slice(lastSlash + 1);
+      try {
+        return new RegExp(body, flags);
+      } catch {
+        return null;
+      }
+    }
+  }
+
+  let regexBody = '';
+  for (const part of trimmed.split(/([*?])/)) {
+    if (part === '*') {
+      regexBody += '.*';
+    } else if (part === '?') {
+      regexBody += '.';
+    } else {
+      regexBody += part.replace(/[.*+?^${}()|[\]\\]/g, '\\$&');
+    }
+  }
+
+  try {
+    return new RegExp(regexBody, 'i');
+  } catch {
+    return null;
+  }
+}
+
+/**
+ * @param {string} details
+ * @param {string} patternsText
+ */
+export function detailsMatchesSkippedPattern(details, patternsText) {
+  if (!details || !patternsText?.trim()) {
+    return false;
+  }
+
+  return patternsText
+    .split(/\r\n|\n|\r/)
+    .map((line) => line.trim())
+    .filter((line) => line.length > 0)
+    .some((pattern) => {
+      const regexp = patternToRegExp(pattern);
+      return regexp !== null && regexp.test(details);
+    });
+}
+
+/**
+ * @param {string} status GraphQL TestStatus enum value.
+ * @param {string} details
+ * @param {string} patternsText
+ */
+export function testStatusToColorClass(status, details = '', patternsText = '') {
+  if (status === 'NOT_RUN' && detailsMatchesSkippedPattern(details, patternsText)) {
+    return 'normal';
+  }
+
+  switch (status) {
+  case 'PASSED':
+    return 'normal';
+  case 'FAILED':
+    return 'error';
+  case 'NOT_RUN':
+    return 'warning';
+  default:
+    return '';
+  }
+}
+
+/**
+ * @param {string} status
+ * @param {string} details
+ * @param {string} patternsText
+ */
+export function testStatusToTextColorClass(status, details = '', patternsText = '') {
+  switch (testStatusToColorClass(status, details, patternsText)) {
+  case 'normal':
+    return 'normal-text';
+  case 'warning':
+    return 'warning-text';
+  case 'error':
+    return 'error-text';
+  default:
+    return '';
+  }
+}
diff --git a/tests/Browser/Pages/BuildTestsPageTest.php b/tests/Browser/Pages/BuildTestsPageTest.php
index 3f4c009246..cc7d168fa9 100644
--- a/tests/Browser/Pages/BuildTestsPageTest.php
+++ b/tests/Browser/Pages/BuildTestsPageTest.php
@@ -521,4 +521,50 @@ public function testOnlyDeltaNoResults(): void
             ;
         });
     }
+
+    public function testNotRunMatchingSkippedPatternShownInGreen(): void
+    {
+        $this->project->notrun_skipped_details_regex = '*skip*';
+        $this->project->save();
+
+        /** @var Build $build */
+        $build = $this->project->builds()->create([
+            'siteid' => $this->site->id,
+            'name' => Str::uuid()->toString(),
+            'uuid' => Str::uuid()->toString(),
+        ]);
+
+        /** @var Test $skipped_not_run_test */
+        $skipped_not_run_test = $build->tests()->create([
+            'testname' => Str::uuid()->toString(),
+            'status' => 'notrun',
+            'details' => 'Test SKIPPED intentionally',
+            'outputid' => $this->testOutput->id,
+        ]);
+
+        /** @var Test $warning_not_run_test */
+        $warning_not_run_test = $build->tests()->create([
+            'testname' => Str::uuid()->toString(),
+            'status' => 'notrun',
+            'details' => 'Unable to find executable',
+            'outputid' => $this->testOutput->id,
+        ]);
+
+        $this->browse(function (Browser $browser) use ($skipped_not_run_test, $warning_not_run_test, $build): void {
+            $browser->visit("/builds/{$build->id}/tests")
+                ->waitFor('@tests-table')
+                ->waitForText($skipped_not_run_test->testname)
+            ;
+
+            self::assertTrue($browser->script(
+                'return document.querySelector(\'a[href*="/tests/' . $skipped_not_run_test->id . '"]\')'
+                . '?.closest("tr")?.querySelector("td.normal") !== null',
+            )[0]);
+
+            self::assertTrue($browser->script(
+                'return document.querySelector(\'a[href*="/tests/' . $warning_not_run_test->id . '"]\')'
+                . '?.closest("tr")?.querySelector("td.warning") !== null',
+            )[0]);
+        });
+    }
 }
diff --git a/tests/Feature/GraphQL/BuildTypeTest.php b/tests/Feature/GraphQL/BuildTypeTest.php
index 5f5aa3acaf..c09de760ba 100644
--- a/tests/Feature/GraphQL/BuildTypeTest.php
+++ b/tests/Feature/GraphQL/BuildTypeTest.php
@@ -9,6 +9,7 @@
 use App\Models\Label;
 use App\Models\Project;
 use App\Models\Target;
+use App\Models\TestOutput;
 use Illuminate\Foundation\Testing\DatabaseTransactions;
 use Illuminate\Support\Carbon;
 use Illuminate\Support\Str;
@@ -651,4 +652,57 @@ public function testPercentCoverageForPath(string $path, ?float $expected_percen
             ],
         ]);
     }
+
+    public function testNotRunTestsWarningCountExcludesSkippedPatterns(): void
+    {
+        $this->project->notrun_skipped_details_regex = '*skip*';
+        $this->project->save();
+
+        $output = TestOutput::create([
+            'path' => 'a',
+            'command' => 'b',
+            'output' => 'c',
+        ]);
+
+        /** @var Build $build */
+        $build = $this->project->builds()->create([
+            'name' => 'build-with-skipped-tests',
+            'uuid' => Str::uuid()->toString(),
+            'testnotrun' => 2,
+        ]);
+
+        $build->tests()->create([
+            'testname' => 'skipped_test',
+            'status' => 'notrun',
+            'details' => 'skipped',
+            'outputid' => $output->id,
+        ]);
+
+        $build->tests()->create([
+            'testname' => 'missing_test',
+            'status' => 'notrun',
+            'details' => 'Unable to find executable',
+            'outputid' => $output->id,
+        ]);
+
+        $this->graphQL('
+            query build($id: ID!) {
+                build(id: $id) {
+                    notRunTestsCount
+                    notRunTestsWarningCount
+                }
+            }
+        ', [
+            'id' => $build->id,
+        ])->assertExactJson([
+            'data' => [
+                'build' => [
+                    'notRunTestsCount' => 2,
+                    'notRunTestsWarningCount' => 1,
+                ],
+            ],
+        ]);
+
+        $output->delete();
+    }
 }
diff --git a/tests/Feature/GraphQL/Mutations/UpdateProjectTest.php b/tests/Feature/GraphQL/Mutations/UpdateProjectTest.php
index d454df8785..0505020daa 100644
--- a/tests/Feature/GraphQL/Mutations/UpdateProjectTest.php
+++ b/tests/Feature/GraphQL/Mutations/UpdateProjectTest.php
@@ -207,6 +207,8 @@ public static function fieldValues(): array
             ['fileUploadLimit', 50, 'uploadquota', 53687091200],
             ['showCoverageCode', false, 'showcoveragecode', false],
             ['banner', 'new banner', 'banner', 'new banner'],
+            ['notRunSkippedDetailsRegex', '*skip*', 'notrun_skipped_details_regex', '*skip*'],
+            ['notRunSkippedDetailsRegex', "Skip\nDisabled", 'notrun_skipped_details_regex', "Skip\nDisabled"],
         ];
     }
 
@@ -269,6 +271,30 @@ public function testCannotChangeNameToExistingProjectName(): void
         self::assertSame($original_name, $project1->fresh()?->name);
     }
 
+    public function testCannotSetInvalidNotRunSkippedDetailsRegex(): void
+    {
+        $project = $this->makePublicProject();
+        $user = $this->makeAdminUser();
+
+        $this->actingAs($user)->graphQL('
+            mutation updateProject($input: UpdateProjectInput!) {
+                updateProject(input: $input) {
+                    project {
+                        notRunSkippedDetailsRegex
+                    }
+                    message
+                }
+            }
+        ', [
+            'input' => [
+                'id' => $project->id,
+                'notRunSkippedDetailsRegex' => '(',
+            ],
+        ])->assertGraphQLErrorMessage('Validation failed for the field [updateProject].');
+
+        self::assertSame('*skip*', $project->fresh()?->notrun_skipped_details_regex);
+    }
+
     public function testCannotChangeNameToInvalidName(): void
     {
         $project = $this->makePublicProject();
diff --git a/tests/Feature/GraphQL/ProjectTypeTest.php b/tests/Feature/GraphQL/ProjectTypeTest.php
index e5719bd30f..45b70b4b31 100644
--- a/tests/Feature/GraphQL/ProjectTypeTest.php
+++ b/tests/Feature/GraphQL/ProjectTypeTest.php
@@ -166,6 +166,8 @@ public static function fieldValues(): array
             ['showcoveragecode', false, 'showCoverageCode', false],
             ['banner', 'test', 'banner', 'test'],
             ['banner', null, 'banner', null],
+            ['notrun_skipped_details_regex', '*skip*', 'notRunSkippedDetailsRegex', '*skip*'],
+            ['notrun_skipped_details_regex', 'Disabled', 'notRunSkippedDetailsRegex', 'Disabled'],
         ];
     }
 
diff --git a/tests/Unit/Utils/TestDisplayTest.php b/tests/Unit/Utils/TestDisplayTest.php
new file mode 100644
index 0000000000..8376ee988c
--- /dev/null
+++ b/tests/Unit/Utils/TestDisplayTest.php
@@ -0,0 +1,82 @@
+<?php
+
+namespace Tests\Unit\Utils;
+
+use App\Models\Test;
+use App\Utils\TestDisplay;
+use PHPUnit\Framework\Attributes\DataProvider;
+use Tests\TestCase;
+
+class TestDisplayTest extends TestCase
+{
+    #[DataProvider('detailsMatchesSkippedPatternCases')]
+    public function testDetailsMatchesSkippedPattern(
+        string $details,
+        string $patternsText,
+        bool $expected,
+    ): void {
+        self::assertSame(
+            $expected,
+            TestDisplay::detailsMatchesSkippedPattern($details, $patternsText),
+        );
+    }
+
+    /**
+     * @return list<array{string, string, bool}>
+     */
+    public static function detailsMatchesSkippedPatternCases(): array
+    {
+        return [
+            ['Test skipped by user', '*skip*', true],
+            ['Test SKIPPED by user', '*skip*', true],
+            ['Test SkIpPeD by user', '*skip*', true],
+            ['Disabled', '*skip*', false],
+            ['', '*skip*', false],
+            ['Skipped', 'Skipped', true],
+            ['Skipped', 'Disabled', false],
+            ['foo bar baz', "*bar*", true],
+        ];
+    }
+
+    #[DataProvider('statusColorClassCases')]
+    public function testStatusColorClass(
+        string $status,
+        ?string $details,
+        string $patternsText,
+        string $expected,
+    ): void {
+        self::assertSame(
+            $expected,
+            TestDisplay::statusColorClass($status, $details, $patternsText),
+        );
+    }
+
+    /**
+     * @return list<array{string, ?string, string, string}>
+     */
+    public static function statusColorClassCases(): array
+    {
+        return [
+            [Test::PASSED, null, '*skip*', 'normal'],
+            [Test::FAILED, null, '*skip*', 'error'],
+            [Test::NOTRUN, 'Some reason', '*skip*', 'warning'],
+            [Test::NOTRUN, 'Test skipped', '*skip*', 'normal'],
+            [Test::NOTRUN, 'Test SKIPPED', '*skip*', 'normal'],
+        ];
+    }
+
+    public function testDefaultPatternConstant(): void
+    {
+        self::assertSame('*skip*', TestDisplay::DEFAULT_NOTRUN_SKIPPED_DETAILS_REGEX);
+    }
+
+    public function testIsValidPatternsTextRejectsInvalidRegex(): void
+    {
+        self::assertFalse(TestDisplay::isValidPatternsText('('));
+    }
+
+    public function testIsValidPatternsTextAcceptsDefaultPattern(): void
+    {
+        self::assertTrue(TestDisplay::isValidPatternsText('*skip*'));
+    }
+}

From fc657637a9532b8703dc856292b6a6becac0428b Mon Sep 17 00:00:00 2001
From: Julien Jomier <jjomier@nvidia.com>
Date: Thu, 6 Aug 2026 19:00:09 -0400
Subject: [PATCH 2/4] Refactor handling of not-run test details

This commit removes the `notRunSkippedDetailsRegex` property from the Project model and associated validation rule, simplifying the logic for handling not-run tests. The `notRunTestsWarningCount` method in the Build model is updated to exclude tests with details marked as "Disabled". The GraphQL schema and UI components are adjusted accordingly to reflect these changes. Additionally, tests are updated to ensure correct behavior without the regex pattern handling.
---
 .../UpdateProjectInputValidator.php           |   6 -
 app/Models/Build.php                          |   7 +-
 app/Models/Project.php                        |   4 -
 app/Rules/NotRunSkippedDetailsRegexRule.php   |  26 -----
 app/Utils/TestDisplay.php                     | 108 ++----------------
 app/cdash/app/Controller/Api/QueryTests.php   |   2 -
 ...otrun_skipped_details_regex_to_project.php |  15 ---
 graphql/schema.graphql                        |  16 +--
 .../js/vue/components/BuildTestsPage.vue      |  19 +--
 .../components/ProjectSettings/GeneralTab.vue |  10 --
 .../js/vue/components/TestDetailsPage.vue     |  11 +-
 .../js/vue/components/shared/TestDisplay.js   |  69 ++---------
 tests/Browser/Pages/BuildTestsPageTest.php    |  17 ++-
 tests/Feature/GraphQL/BuildTypeTest.php       |  11 +-
 .../GraphQL/Mutations/UpdateProjectTest.php   |  26 -----
 tests/Feature/GraphQL/ProjectTypeTest.php     |   2 -
 tests/Unit/Utils/TestDisplayTest.php          |  63 ++++------
 17 files changed, 61 insertions(+), 351 deletions(-)
 delete mode 100644 app/Rules/NotRunSkippedDetailsRegexRule.php
 delete mode 100644 database/migrations/2026_05_26_120000_add_notrun_skipped_details_regex_to_project.php

diff --git a/app/GraphQL/Validators/UpdateProjectInputValidator.php b/app/GraphQL/Validators/UpdateProjectInputValidator.php
index 9d176a6436..0dc314ed3a 100644
--- a/app/GraphQL/Validators/UpdateProjectInputValidator.php
+++ b/app/GraphQL/Validators/UpdateProjectInputValidator.php
@@ -3,7 +3,6 @@
 namespace App\GraphQL\Validators;
 
 use App\Models\Project;
-use App\Rules\NotRunSkippedDetailsRegexRule;
 use App\Rules\ProjectAuthenticateSubmissionsRule;
 use App\Rules\ProjectNameRule;
 use App\Rules\ProjectVisibilityRule;
@@ -26,11 +25,6 @@ public function rules(): array
             'authenticateSubmissions' => [
                 new ProjectAuthenticateSubmissionsRule(),
             ],
-            'notRunSkippedDetailsRegex' => [
-                'nullable',
-                'string',
-                new NotRunSkippedDetailsRegexRule(),
-            ],
         ];
     }
 
diff --git a/app/Models/Build.php b/app/Models/Build.php
index 98402a0684..6d856f9e4f 100644
--- a/app/Models/Build.php
+++ b/app/Models/Build.php
@@ -403,16 +403,13 @@ public function percentCoverageForPath(string $path): ?float
     }
 
     /**
-     * The number of not-run tests whose details do not match the project's skipped pattern.
+     * The number of not-run tests whose details are not "Disabled".
      */
     public function notRunTestsWarningCount(): int
     {
-        $patterns = $this->project?->notrun_skipped_details_regex
-            ?? TestDisplay::DEFAULT_NOTRUN_SKIPPED_DETAILS_REGEX;
-
         $count = 0;
         foreach ($this->tests()->where('status', Test::NOTRUN)->pluck('details') as $details) {
-            if (!TestDisplay::isAcceptableNotRun($details, $patterns)) {
+            if (!TestDisplay::isAcceptableNotRun($details)) {
                 $count++;
             }
         }
diff --git a/app/Models/Project.php b/app/Models/Project.php
index c102b9200e..11b7987ed4 100644
--- a/app/Models/Project.php
+++ b/app/Models/Project.php
@@ -3,7 +3,6 @@
 namespace App\Models;
 
 use App\Enums\ProjectRole;
-use App\Utils\TestDisplay;
 use Illuminate\Database\Eloquent\Builder;
 use Illuminate\Database\Eloquent\Casts\Attribute;
 use Illuminate\Database\Eloquent\Model;
@@ -50,7 +49,6 @@
  * @property ?string $banner
  * @property ?string $logoUrl
  * @property ?string $cmakeprojectroot
- * @property string $notrun_skipped_details_regex
  *
  * @method Builder<Project> forUser()
  * @method Builder<Project> administeredByUser()
@@ -97,7 +95,6 @@ class Project extends Model
         'ldapfilter',
         'banner',
         'cmakeprojectroot',
-        'notrun_skipped_details_regex',
     ];
 
     protected $casts = [
@@ -140,7 +137,6 @@ class Project extends Model
         'uploadquota' => 10,
         'showcoveragecode' => true,
         'authenticatesubmissions' => false,
-        'notrun_skipped_details_regex' => TestDisplay::DEFAULT_NOTRUN_SKIPPED_DETAILS_REGEX,
     ];
 
     public const ACCESS_PRIVATE = 0;
diff --git a/app/Rules/NotRunSkippedDetailsRegexRule.php b/app/Rules/NotRunSkippedDetailsRegexRule.php
deleted file mode 100644
index d6d9c402dd..0000000000
--- a/app/Rules/NotRunSkippedDetailsRegexRule.php
+++ /dev/null
@@ -1,26 +0,0 @@
-<?php
-
-namespace App\Rules;
-
-use App\Utils\TestDisplay;
-use Closure;
-use Illuminate\Contracts\Validation\ValidationRule;
-
-final class NotRunSkippedDetailsRegexRule implements ValidationRule
-{
-    public function validate(string $attribute, mixed $value, Closure $fail): void
-    {
-        if ($value === null || $value === '') {
-            return;
-        }
-
-        if (!is_string($value)) {
-            $fail('The skipped test details pattern must be a string.');
-            return;
-        }
-
-        if (!TestDisplay::isValidPatternsText($value)) {
-            $fail('The skipped test details pattern contains an invalid regular expression.');
-        }
-    }
-}
diff --git a/app/Utils/TestDisplay.php b/app/Utils/TestDisplay.php
index a4e67670c7..f1bb526307 100644
--- a/app/Utils/TestDisplay.php
+++ b/app/Utils/TestDisplay.php
@@ -6,87 +6,19 @@
 
 final class TestDisplay
 {
-    public const DEFAULT_NOTRUN_SKIPPED_DETAILS_REGEX = '*skip*';
-
     /**
-     * @return list<string>
+     * CTest sets Details="Disabled" for tests marked DISABLED.
      */
-    public static function parsePatterns(?string $patternsText): array
-    {
-        if ($patternsText === null || trim($patternsText) === '') {
-            return [];
-        }
-
-        $patterns = [];
-        foreach (preg_split("/\r\n|\n|\r/", $patternsText) as $line) {
-            $line = trim($line);
-            if ($line !== '') {
-                $patterns[] = $line;
-            }
-        }
+    public const DISABLED_DETAILS = 'Disabled';
 
-        return $patterns;
-    }
-
-    public static function patternToPcre(string $pattern): string
+    public static function isAcceptableNotRun(?string $details): bool
     {
-        $pattern = trim($pattern);
-        if ($pattern === '') {
-            return '';
-        }
-
-        if (str_starts_with($pattern, '/')) {
-            return $pattern;
-        }
-
-        $parts = preg_split('/(\*|\?)/', $pattern, -1, PREG_SPLIT_DELIM_CAPTURE);
-        if ($parts === false) {
-            return '';
-        }
-
-        $regex = '';
-        foreach ($parts as $part) {
-            if ($part === '*') {
-                $regex .= '.*';
-            } elseif ($part === '?') {
-                $regex .= '.';
-            } else {
-                $regex .= preg_quote($part, '/');
-            }
-        }
-
-        return '/(?i)' . $regex . '/';
+        return $details === self::DISABLED_DETAILS;
     }
 
-    public static function detailsMatchesSkippedPattern(?string $details, ?string $patternsText): bool
+    public static function statusColorClass(string $status, ?string $details): string
     {
-        if ($details === null || $details === '') {
-            return false;
-        }
-
-        foreach (self::parsePatterns($patternsText) as $pattern) {
-            $pcre = self::patternToPcre($pattern);
-            if ($pcre === '') {
-                continue;
-            }
-
-            $result = @preg_match($pcre, $details);
-            if ($result === 1) {
-                return true;
-            }
-        }
-
-        return false;
-    }
-
-    public static function isAcceptableNotRun(?string $details, ?string $patternsText): bool
-    {
-        return self::detailsMatchesSkippedPattern($details, $patternsText);
-    }
-
-    public static function statusColorClass(string $status, ?string $details, ?string $patternsText): string
-    {
-        if ($status === Test::NOTRUN && self::isAcceptableNotRun($details, $patternsText)) {
+        if ($status === Test::NOTRUN && self::isAcceptableNotRun($details)) {
             return 'normal';
         }
 
@@ -98,9 +30,9 @@ public static function statusColorClass(string $status, ?string $details, ?strin
         };
     }
 
-    public static function statusTextColorClass(string $status, ?string $details, ?string $patternsText): string
+    public static function statusTextColorClass(string $status, ?string $details): string
     {
-        return match (self::statusColorClass($status, $details, $patternsText)) {
+        return match (self::statusColorClass($status, $details)) {
             'normal' => 'normal-text',
             'warning' => 'warning-text',
             'error' => 'error-text',
@@ -108,7 +40,7 @@ public static function statusTextColorClass(string $status, ?string $details, ?s
         };
     }
 
-    public static function graphqlStatusColorClass(string $status, ?string $details, ?string $patternsText): string
+    public static function graphqlStatusColorClass(string $status, ?string $details): string
     {
         $dbStatus = match ($status) {
             'NOT_RUN' => Test::NOTRUN,
@@ -117,26 +49,6 @@ public static function graphqlStatusColorClass(string $status, ?string $details,
             default => strtolower($status),
         };
 
-        return self::statusColorClass($dbStatus, $details, $patternsText);
-    }
-
-    public static function isValidPatternsText(?string $patternsText): bool
-    {
-        foreach (self::parsePatterns($patternsText) as $pattern) {
-            $pcre = self::patternToPcre($pattern);
-            if ($pcre === '') {
-                return false;
-            }
-
-            set_error_handler(static fn (): bool => true);
-            $result = preg_match($pcre, '');
-            restore_error_handler();
-
-            if ($result === false) {
-                return false;
-            }
-        }
-
-        return true;
+        return self::statusColorClass($dbStatus, $details);
     }
 }
diff --git a/app/cdash/app/Controller/Api/QueryTests.php b/app/cdash/app/Controller/Api/QueryTests.php
index 0899abe869..e82746126c 100644
--- a/app/cdash/app/Controller/Api/QueryTests.php
+++ b/app/cdash/app/Controller/Api/QueryTests.php
@@ -376,7 +376,6 @@ public function getResponse(): array
                 ", $query_params);
 
         // Rows of test data to be displayed to the user.
-        $notRunSkippedDetailsRegex = EloquentProject::findOrFail($this->project->Id)->notrun_skipped_details_regex;
         $tests = [];
         foreach ($rows as $row) {
             $test = [];
@@ -428,7 +427,6 @@ public function getResponse(): array
                     $test['statusclass'] = TestDisplay::statusColorClass(
                         'notrun',
                         $row->details,
-                        $notRunSkippedDetailsRegex,
                     );
                     break;
             }
diff --git a/database/migrations/2026_05_26_120000_add_notrun_skipped_details_regex_to_project.php b/database/migrations/2026_05_26_120000_add_notrun_skipped_details_regex_to_project.php
deleted file mode 100644
index e2a61e4547..0000000000
--- a/database/migrations/2026_05_26_120000_add_notrun_skipped_details_regex_to_project.php
+++ /dev/null
@@ -1,15 +0,0 @@
-<?php
-
-use Illuminate\Database\Migrations\Migration;
-use Illuminate\Support\Facades\DB;
-
-return new class extends Migration {
-    public function up(): void
-    {
-        DB::statement("ALTER TABLE project ADD COLUMN notrun_skipped_details_regex text NOT NULL DEFAULT '*skip*'");
-    }
-
-    public function down(): void
-    {
-    }
-};
diff --git a/graphql/schema.graphql b/graphql/schema.graphql
index ba63fd858d..1ae17527b8 100644
--- a/graphql/schema.graphql
+++ b/graphql/schema.graphql
@@ -398,13 +398,6 @@ type Project {
   "Invitations to this project which have not been accepted yet."
   invitations: [ProjectInvitation!]! @canRoot(ability: "inviteUser") @hasMany(type: CONNECTION) @orderBy(column: "id")
 
-  """
-  Regular expression(s), one per line, matched against not-run test details. Matching tests are
-  displayed in green instead of yellow. Glob wildcards (*) are supported. The default pattern is
-  *skip* (case insensitive).
-  """
-  notRunSkippedDetailsRegex: String! @rename(attribute: "notrun_skipped_details_regex")
-
   """
   Pinned test measurements are test measurements, identified by name, which should be shown
   alongside Test details throughout the site.
@@ -522,12 +515,6 @@ input UpdateProjectInput @validator {
   showCoverageCode: Boolean @rename(attribute: "showcoveragecode")
 
   banner: String
-
-  """
-  Regular expression(s), one per line, matched against not-run test details. Matching tests are
-  displayed in green instead of yellow.
-  """
-  notRunSkippedDetailsRegex: String @rename(attribute: "notrun_skipped_details_regex")
 }
 
 
@@ -870,8 +857,7 @@ type Build {
   notRunTestsCount: Int @rename(attribute: "testnotrun")
 
   """
-  The number of not-run tests counted as warnings (excludes tests whose details match the
-  project's skipped-test pattern).
+  The number of not-run tests counted as warnings (excludes tests whose details are "Disabled").
   """
   notRunTestsWarningCount: Int! @method(name: "notRunTestsWarningCount")
 
diff --git a/resources/js/vue/components/BuildTestsPage.vue b/resources/js/vue/components/BuildTestsPage.vue
index 7422904b7b..0484367b46 100644
--- a/resources/js/vue/components/BuildTestsPage.vue
+++ b/resources/js/vue/components/BuildTestsPage.vue
@@ -83,9 +83,6 @@ const TEST_QUERY = gql`
   ) {
     build(id: $buildid) {
       id
-      project {
-        notRunSkippedDetailsRegex
-      }
       tests(filters: $filters, first: 1000000) {
         edges {
           node {
@@ -209,11 +206,6 @@ export default {
         });
         return tests;
       },
-      result({ data }) {
-        if (data?.build?.project?.notRunSkippedDetailsRegex !== undefined) {
-          this.notRunSkippedDetailsRegex = data.build.project.notRunSkippedDetailsRegex;
-        }
-      },
       variables() {
         return {
           buildid: this.buildId,
@@ -246,11 +238,6 @@ export default {
         });
         return tests;
       },
-      result({ data }) {
-        if (data?.build?.project?.notRunSkippedDetailsRegex !== undefined) {
-          this.notRunSkippedDetailsRegex = data.build.project.notRunSkippedDetailsRegex;
-        }
-      },
       variables() {
         return {
           buildid: this.previousBuildId,
@@ -273,7 +260,6 @@ export default {
   data() {
     return {
       changedFilters: JSON.parse(JSON.stringify(this.initialFilters)),
-      notRunSkippedDetailsRegex: '*skip*',
     };
   },
 
@@ -323,7 +309,6 @@ export default {
     },
 
     formattedTestRows() {
-      const notRunSkippedDetailsRegex = this.notRunSkippedDetailsRegex;
       return this.filteredTests?.map((edge) => {
         return {
           name: {
@@ -341,14 +326,14 @@ export default {
             value: edge.node.status,
             text: this.humanReadableTestStatus(edge.node.status),
             href: `${this.$baseURL}/tests/${edge.node.id}`,
-            classes: [testStatusToColorClass(edge.node.status, edge.node.details, notRunSkippedDetailsRegex)],
+            classes: [testStatusToColorClass(edge.node.status, edge.node.details)],
           },
           subProject: edge.subProject ?? '',
           timeStatus: {
             value: edge.node.timeStatusCategory,
             text: this.humanReadableTestStatus(edge.node.timeStatusCategory),
             href: `${this.$baseURL}/tests/${edge.node.id}?graph=time`,
-            classes: [testStatusToColorClass(edge.node.timeStatusCategory, edge.node.details, notRunSkippedDetailsRegex)],
+            classes: [testStatusToColorClass(edge.node.timeStatusCategory, edge.node.details)],
           },
           history: {
             value: '',
diff --git a/resources/js/vue/components/ProjectSettings/GeneralTab.vue b/resources/js/vue/components/ProjectSettings/GeneralTab.vue
index 96d9468323..48a6838793 100644
--- a/resources/js/vue/components/ProjectSettings/GeneralTab.vue
+++ b/resources/js/vue/components/ProjectSettings/GeneralTab.vue
@@ -417,14 +417,6 @@
           min="0"
           test-id="test-time-std-multiplier-input"
         />
-
-        <TextAreaField
-          v-model="form.notRunSkippedDetailsRegex"
-          :validation-error="validationErrors?.notRunSkippedDetailsRegex?.[0]"
-          label="Skipped Not Run Test Details Pattern"
-          description="One pattern per line. When a not-run test's details match a pattern, it is shown in green. Glob wildcards (*) are supported; the default *skip* matches skip case-insensitively."
-          test-id="not-run-skipped-details-regex-input"
-        />
       </FormSection>
 
       <FormSection title="">
@@ -550,7 +542,6 @@ export default {
         fileUploadLimit: 50,
         showCoverageCode: true,
         banner: '',
-        notRunSkippedDetailsRegex: '*skip*',
       },
       validationErrors: {},
       fatalError: null,
@@ -594,7 +585,6 @@ export default {
             fileUploadLimit
             showCoverageCode
             banner
-            notRunSkippedDetailsRegex
           }
         }
       `,
diff --git a/resources/js/vue/components/TestDetailsPage.vue b/resources/js/vue/components/TestDetailsPage.vue
index f6a4309d5e..d17854a359 100644
--- a/resources/js/vue/components/TestDetailsPage.vue
+++ b/resources/js/vue/components/TestDetailsPage.vue
@@ -334,7 +334,7 @@ import {
   faChartLine,
   faLink,
 } from '@fortawesome/free-solid-svg-icons';
-import { detailsMatchesSkippedPattern } from './shared/TestDisplay';
+import { isAcceptableNotRun } from './shared/TestDisplay';
 
 export default {
   name: 'TestDetails',
@@ -383,7 +383,6 @@ export default {
               enableTestTiming
               testTimeStdMultiplier
               testTimeStdThreshold
-              notRunSkippedDetailsRegex
             }
           }
         }
@@ -523,13 +522,7 @@ export default {
         return 'tw-bg-neutral tw-text-neutral-content';
       }
 
-      if (
-        this.test.status === 'NOT_RUN'
-        && detailsMatchesSkippedPattern(
-          this.test.details,
-          this.build?.project?.notRunSkippedDetailsRegex ?? '*skip*',
-        )
-      ) {
+      if (this.test.status === 'NOT_RUN' && isAcceptableNotRun(this.test.details)) {
         return 'tw-bg-success tw-text-success-content';
       }
 
diff --git a/resources/js/vue/components/shared/TestDisplay.js b/resources/js/vue/components/shared/TestDisplay.js
index 5a0979a390..4a78b8d055 100644
--- a/resources/js/vue/components/shared/TestDisplay.js
+++ b/resources/js/vue/components/shared/TestDisplay.js
@@ -1,70 +1,22 @@
 /**
- * @param {string} pattern
- * @returns {RegExp|null}
+ * CTest sets Details="Disabled" for tests marked DISABLED.
  */
-export function patternToRegExp(pattern) {
-  const trimmed = pattern.trim();
-  if (trimmed === '') {
-    return null;
-  }
-
-  if (trimmed.startsWith('/')) {
-    const lastSlash = trimmed.lastIndexOf('/');
-    if (lastSlash > 0) {
-      const body = trimmed.slice(1, lastSlash);
-      const flags = trimmed.slice(lastSlash + 1);
-      try {
-        return new RegExp(body, flags);
-      } catch {
-        return null;
-      }
-    }
-  }
-
-  let regexBody = '';
-  for (const part of trimmed.split(/([*?])/)) {
-    if (part === '*') {
-      regexBody += '.*';
-    } else if (part === '?') {
-      regexBody += '.';
-    } else {
-      regexBody += part.replace(/[.*+?^${}()|[\]\\]/g, '\\$&');
-    }
-  }
-
-  try {
-    return new RegExp(regexBody, 'i');
-  } catch {
-    return null;
-  }
-}
+export const DISABLED_DETAILS = 'Disabled';
 
 /**
- * @param {string} details
- * @param {string} patternsText
+ * @param {string|null|undefined} details
+ * @returns {boolean}
  */
-export function detailsMatchesSkippedPattern(details, patternsText) {
-  if (!details || !patternsText?.trim()) {
-    return false;
-  }
-
-  return patternsText
-    .split(/\r\n|\n|\r/)
-    .map((line) => line.trim())
-    .filter((line) => line.length > 0)
-    .some((pattern) => {
-      const regexp = patternToRegExp(pattern);
-      return regexp !== null && regexp.test(details);
-    });
+export function isAcceptableNotRun(details) {
+  return details === DISABLED_DETAILS;
 }
 
 /**
  * @param {string} status GraphQL TestStatus enum value.
  * @param {string} details
- * @param {string} patternsText
  */
-export function testStatusToColorClass(status, details = '', patternsText = '') {
-  if (status === 'NOT_RUN' && detailsMatchesSkippedPattern(details, patternsText)) {
+export function testStatusToColorClass(status, details = '') {
+  if (status === 'NOT_RUN' && isAcceptableNotRun(details)) {
     return 'normal';
   }
 
@@ -83,10 +35,9 @@ export function testStatusToColorClass(status, details = '', patternsText = '')
 /**
  * @param {string} status
  * @param {string} details
- * @param {string} patternsText
  */
-export function testStatusToTextColorClass(status, details = '', patternsText = '') {
-  switch (testStatusToColorClass(status, details, patternsText)) {
+export function testStatusToTextColorClass(status, details = '') {
+  switch (testStatusToColorClass(status, details)) {
   case 'normal':
     return 'normal-text';
   case 'warning':
diff --git a/tests/Browser/Pages/BuildTestsPageTest.php b/tests/Browser/Pages/BuildTestsPageTest.php
index cc7d168fa9..4f09cc8a19 100644
--- a/tests/Browser/Pages/BuildTestsPageTest.php
+++ b/tests/Browser/Pages/BuildTestsPageTest.php
@@ -522,11 +522,8 @@ public function testOnlyDeltaNoResults(): void
         });
     }
 
-    public function testNotRunMatchingSkippedPatternShownInGreen(): void
+    public function testNotRunWithDisabledDetailsShownInGreen(): void
     {
-        $this->project->notrun_skipped_details_regex = '*skip*';
-        $this->project->save();
-
         /** @var Build $build */
         $build = $this->project->builds()->create([
             'siteid' => $this->site->id,
@@ -534,11 +531,11 @@ public function testNotRunMatchingSkippedPatternShownInGreen(): void
             'uuid' => Str::uuid()->toString(),
         ]);
 
-        /** @var Test $skipped_not_run_test */
-        $skipped_not_run_test = $build->tests()->create([
+        /** @var Test $disabled_not_run_test */
+        $disabled_not_run_test = $build->tests()->create([
             'testname' => Str::uuid()->toString(),
             'status' => 'notrun',
-            'details' => 'Test SKIPPED intentionally',
+            'details' => 'Disabled',
             'outputid' => $this->testOutput->id,
         ]);
 
@@ -550,14 +547,14 @@ public function testNotRunMatchingSkippedPatternShownInGreen(): void
             'outputid' => $this->testOutput->id,
         ]);
 
-        $this->browse(function (Browser $browser) use ($skipped_not_run_test, $warning_not_run_test, $build): void {
+        $this->browse(function (Browser $browser) use ($disabled_not_run_test, $warning_not_run_test, $build): void {
             $browser->visit("/builds/{$build->id}/tests")
                 ->waitFor('@tests-table')
-                ->waitForText($skipped_not_run_test->testname)
+                ->waitForText($disabled_not_run_test->testname)
             ;
 
             self::assertTrue($browser->script(
-                'return document.querySelector(\'a[href*="/tests/' . $skipped_not_run_test->id . '"]\')'
+                'return document.querySelector(\'a[href*="/tests/' . $disabled_not_run_test->id . '"]\')'
                 . '?.closest("tr")?.querySelector("td.normal") !== null',
             )[0]);
 
diff --git a/tests/Feature/GraphQL/BuildTypeTest.php b/tests/Feature/GraphQL/BuildTypeTest.php
index c09de760ba..96f437b2e5 100644
--- a/tests/Feature/GraphQL/BuildTypeTest.php
+++ b/tests/Feature/GraphQL/BuildTypeTest.php
@@ -653,11 +653,8 @@ public function testPercentCoverageForPath(string $path, ?float $expected_percen
         ]);
     }
 
-    public function testNotRunTestsWarningCountExcludesSkippedPatterns(): void
+    public function testNotRunTestsWarningCountExcludesDisabledDetails(): void
     {
-        $this->project->notrun_skipped_details_regex = '*skip*';
-        $this->project->save();
-
         $output = TestOutput::create([
             'path' => 'a',
             'command' => 'b',
@@ -666,15 +663,15 @@ public function testNotRunTestsWarningCountExcludesSkippedPatterns(): void
 
         /** @var Build $build */
         $build = $this->project->builds()->create([
-            'name' => 'build-with-skipped-tests',
+            'name' => 'build-with-disabled-tests',
             'uuid' => Str::uuid()->toString(),
             'testnotrun' => 2,
         ]);
 
         $build->tests()->create([
-            'testname' => 'skipped_test',
+            'testname' => 'disabled_test',
             'status' => 'notrun',
-            'details' => 'skipped',
+            'details' => 'Disabled',
             'outputid' => $output->id,
         ]);
 
diff --git a/tests/Feature/GraphQL/Mutations/UpdateProjectTest.php b/tests/Feature/GraphQL/Mutations/UpdateProjectTest.php
index 0505020daa..d454df8785 100644
--- a/tests/Feature/GraphQL/Mutations/UpdateProjectTest.php
+++ b/tests/Feature/GraphQL/Mutations/UpdateProjectTest.php
@@ -207,8 +207,6 @@ public static function fieldValues(): array
             ['fileUploadLimit', 50, 'uploadquota', 53687091200],
             ['showCoverageCode', false, 'showcoveragecode', false],
             ['banner', 'new banner', 'banner', 'new banner'],
-            ['notRunSkippedDetailsRegex', '*skip*', 'notrun_skipped_details_regex', '*skip*'],
-            ['notRunSkippedDetailsRegex', "Skip\nDisabled", 'notrun_skipped_details_regex', "Skip\nDisabled"],
         ];
     }
 
@@ -271,30 +269,6 @@ public function testCannotChangeNameToExistingProjectName(): void
         self::assertSame($original_name, $project1->fresh()?->name);
     }
 
-    public function testCannotSetInvalidNotRunSkippedDetailsRegex(): void
-    {
-        $project = $this->makePublicProject();
-        $user = $this->makeAdminUser();
-
-        $this->actingAs($user)->graphQL('
-            mutation updateProject($input: UpdateProjectInput!) {
-                updateProject(input: $input) {
-                    project {
-                        notRunSkippedDetailsRegex
-                    }
-                    message
-                }
-            }
-        ', [
-            'input' => [
-                'id' => $project->id,
-                'notRunSkippedDetailsRegex' => '(',
-            ],
-        ])->assertGraphQLErrorMessage('Validation failed for the field [updateProject].');
-
-        self::assertSame('*skip*', $project->fresh()?->notrun_skipped_details_regex);
-    }
-
     public function testCannotChangeNameToInvalidName(): void
     {
         $project = $this->makePublicProject();
diff --git a/tests/Feature/GraphQL/ProjectTypeTest.php b/tests/Feature/GraphQL/ProjectTypeTest.php
index 45b70b4b31..e5719bd30f 100644
--- a/tests/Feature/GraphQL/ProjectTypeTest.php
+++ b/tests/Feature/GraphQL/ProjectTypeTest.php
@@ -166,8 +166,6 @@ public static function fieldValues(): array
             ['showcoveragecode', false, 'showCoverageCode', false],
             ['banner', 'test', 'banner', 'test'],
             ['banner', null, 'banner', null],
-            ['notrun_skipped_details_regex', '*skip*', 'notRunSkippedDetailsRegex', '*skip*'],
-            ['notrun_skipped_details_regex', 'Disabled', 'notRunSkippedDetailsRegex', 'Disabled'],
         ];
     }
 
diff --git a/tests/Unit/Utils/TestDisplayTest.php b/tests/Unit/Utils/TestDisplayTest.php
index 8376ee988c..feeeee1f71 100644
--- a/tests/Unit/Utils/TestDisplayTest.php
+++ b/tests/Unit/Utils/TestDisplayTest.php
@@ -9,32 +9,26 @@
 
 class TestDisplayTest extends TestCase
 {
-    #[DataProvider('detailsMatchesSkippedPatternCases')]
-    public function testDetailsMatchesSkippedPattern(
-        string $details,
-        string $patternsText,
-        bool $expected,
-    ): void {
-        self::assertSame(
-            $expected,
-            TestDisplay::detailsMatchesSkippedPattern($details, $patternsText),
-        );
+    #[DataProvider('isAcceptableNotRunCases')]
+    public function testIsAcceptableNotRun(?string $details, bool $expected): void
+    {
+        self::assertSame($expected, TestDisplay::isAcceptableNotRun($details));
     }
 
     /**
-     * @return list<array{string, string, bool}>
+     * @return list<array{?string, bool}>
      */
-    public static function detailsMatchesSkippedPatternCases(): array
+    public static function isAcceptableNotRunCases(): array
     {
         return [
-            ['Test skipped by user', '*skip*', true],
-            ['Test SKIPPED by user', '*skip*', true],
-            ['Test SkIpPeD by user', '*skip*', true],
-            ['Disabled', '*skip*', false],
-            ['', '*skip*', false],
-            ['Skipped', 'Skipped', true],
-            ['Skipped', 'Disabled', false],
-            ['foo bar baz', "*bar*", true],
+            ['Disabled', true],
+            ['disabled', false],
+            ['DISABLED', false],
+            ['Test Disabled', false],
+            ['Disabled test', false],
+            ['Skipped', false],
+            ['', false],
+            [null, false],
         ];
     }
 
@@ -42,41 +36,30 @@ public static function detailsMatchesSkippedPatternCases(): array
     public function testStatusColorClass(
         string $status,
         ?string $details,
-        string $patternsText,
         string $expected,
     ): void {
         self::assertSame(
             $expected,
-            TestDisplay::statusColorClass($status, $details, $patternsText),
+            TestDisplay::statusColorClass($status, $details),
         );
     }
 
     /**
-     * @return list<array{string, ?string, string, string}>
+     * @return list<array{string, ?string, string}>
      */
     public static function statusColorClassCases(): array
     {
         return [
-            [Test::PASSED, null, '*skip*', 'normal'],
-            [Test::FAILED, null, '*skip*', 'error'],
-            [Test::NOTRUN, 'Some reason', '*skip*', 'warning'],
-            [Test::NOTRUN, 'Test skipped', '*skip*', 'normal'],
-            [Test::NOTRUN, 'Test SKIPPED', '*skip*', 'normal'],
+            [Test::PASSED, null, 'normal'],
+            [Test::FAILED, null, 'error'],
+            [Test::NOTRUN, 'Some reason', 'warning'],
+            [Test::NOTRUN, 'Disabled', 'normal'],
+            [Test::NOTRUN, 'skipped', 'warning'],
         ];
     }
 
-    public function testDefaultPatternConstant(): void
-    {
-        self::assertSame('*skip*', TestDisplay::DEFAULT_NOTRUN_SKIPPED_DETAILS_REGEX);
-    }
-
-    public function testIsValidPatternsTextRejectsInvalidRegex(): void
-    {
-        self::assertFalse(TestDisplay::isValidPatternsText('('));
-    }
-
-    public function testIsValidPatternsTextAcceptsDefaultPattern(): void
+    public function testDisabledDetailsConstant(): void
     {
-        self::assertTrue(TestDisplay::isValidPatternsText('*skip*'));
+        self::assertSame('Disabled', TestDisplay::DISABLED_DETAILS);
     }
 }

From 1c390ee190842385e6dd35263916a4d34a4eba49 Mon Sep 17 00:00:00 2001
From: Julien Jomier <jjomier@nvidia.com>
Date: Fri, 7 Aug 2026 10:29:32 -0400
Subject: [PATCH 3/4] Refactor test status color class functions

This commit improves the formatting of the `testStatusToColorClass` and `testStatusToTextColorClass` functions in the TestDisplay component. The switch cases have been reformatted for better readability, maintaining the same functionality. No changes to logic or behavior were made.
---
 .../js/vue/components/shared/TestDisplay.js   | 32 +++++++++----------
 1 file changed, 16 insertions(+), 16 deletions(-)

diff --git a/resources/js/vue/components/shared/TestDisplay.js b/resources/js/vue/components/shared/TestDisplay.js
index 4a78b8d055..584f277e44 100644
--- a/resources/js/vue/components/shared/TestDisplay.js
+++ b/resources/js/vue/components/shared/TestDisplay.js
@@ -21,14 +21,14 @@ export function testStatusToColorClass(status, details = '') {
   }
 
   switch (status) {
-  case 'PASSED':
-    return 'normal';
-  case 'FAILED':
-    return 'error';
-  case 'NOT_RUN':
-    return 'warning';
-  default:
-    return '';
+    case 'PASSED':
+      return 'normal';
+    case 'FAILED':
+      return 'error';
+    case 'NOT_RUN':
+      return 'warning';
+    default:
+      return '';
   }
 }
 
@@ -38,13 +38,13 @@ export function testStatusToColorClass(status, details = '') {
  */
 export function testStatusToTextColorClass(status, details = '') {
   switch (testStatusToColorClass(status, details)) {
-  case 'normal':
-    return 'normal-text';
-  case 'warning':
-    return 'warning-text';
-  case 'error':
-    return 'error-text';
-  default:
-    return '';
+    case 'normal':
+      return 'normal-text';
+    case 'warning':
+      return 'warning-text';
+    case 'error':
+      return 'error-text';
+    default:
+      return '';
   }
 }

From c5f7ce4b4987cd4393f3b5436e1459775741e296 Mon Sep 17 00:00:00 2001
From: Julien Jomier <jjomier@nvidia.com>
Date: Fri, 7 Aug 2026 12:07:55 -0400
Subject: [PATCH 4/4] Refactor not-run tests warning count logic

This commit simplifies the `notRunTestsWarningCount` method in the Build model by using a more efficient query to count not-run tests that do not have details marked as "Disabled". Additionally, the API response in the Index controller is updated to reflect the new method signature. Unused test methods in the TestDisplayTest class have been removed to clean up the codebase.
---
 app/Models/Build.php                   | 15 +++++++--------
 app/cdash/app/Controller/Api/Index.php |  2 +-
 tests/Unit/Utils/TestDisplayTest.php   |  5 -----
 3 files changed, 8 insertions(+), 14 deletions(-)

diff --git a/app/Models/Build.php b/app/Models/Build.php
index 6d856f9e4f..d1943953ab 100644
--- a/app/Models/Build.php
+++ b/app/Models/Build.php
@@ -407,13 +407,12 @@ public function percentCoverageForPath(string $path): ?float
      */
     public function notRunTestsWarningCount(): int
     {
-        $count = 0;
-        foreach ($this->tests()->where('status', Test::NOTRUN)->pluck('details') as $details) {
-            if (!TestDisplay::isAcceptableNotRun($details)) {
-                $count++;
-            }
-        }
-
-        return $count;
+        return $this->tests()
+            ->where('status', Test::NOTRUN)
+            ->where(static function (Builder $query): void {
+                $query->whereNull('details')
+                    ->orWhere('details', '!=', TestDisplay::DISABLED_DETAILS);
+            })
+            ->count();
     }
 }
diff --git a/app/cdash/app/Controller/Api/Index.php b/app/cdash/app/Controller/Api/Index.php
index 7542f771bf..ebd70ae7c9 100644
--- a/app/cdash/app/Controller/Api/Index.php
+++ b/app/cdash/app/Controller/Api/Index.php
@@ -1099,7 +1099,7 @@ public function generateBuildResponseFromRow(array $build_array): array|false
             }
 
             $test_response['notrun'] = $nnotrun;
-            $test_response['notrunwarning'] = $this->computeNotRunTestsWarningCount((int) $buildid, $nnotrun);
+            $test_response['notrunwarning'] = $this->computeNotRunTestsWarningCount($buildid, $nnotrun);
             $test_response['fail'] = $nfail;
             $test_response['pass'] = $npass;
 
diff --git a/tests/Unit/Utils/TestDisplayTest.php b/tests/Unit/Utils/TestDisplayTest.php
index feeeee1f71..c01b9dc196 100644
--- a/tests/Unit/Utils/TestDisplayTest.php
+++ b/tests/Unit/Utils/TestDisplayTest.php
@@ -57,9 +57,4 @@ public static function statusColorClassCases(): array
             [Test::NOTRUN, 'skipped', 'warning'],
         ];
     }
-
-    public function testDisabledDetailsConstant(): void
-    {
-        self::assertSame('Disabled', TestDisplay::DISABLED_DETAILS);
-    }
 }