From 37dd1c5e03b706dece8bc1c050dfbc3d627ab8cb Mon Sep 17 00:00:00 2001
From: kurilova <kurilova@google.com>
Date: Fri, 17 Oct 2025 11:58:33 +0000
Subject: [PATCH] Highlight row when test result non compliant and required

---
 .../testrun-table/testrun-table.component.html           | 3 +++
 .../testrun-table/testrun-table.component.scss           | 8 ++++++++
 .../components/testrun-table/testrun-table.component.ts  | 9 +++++++++
 3 files changed, 20 insertions(+)

diff --git a/modules/ui/src/app/pages/testrun/components/testrun-table/testrun-table.component.html b/modules/ui/src/app/pages/testrun/components/testrun-table/testrun-table.component.html
index 697254acf..ac02c7078 100644
--- a/modules/ui/src/app/pages/testrun/components/testrun-table/testrun-table.component.html
+++ b/modules/ui/src/app/pages/testrun/components/testrun-table/testrun-table.component.html
@@ -43,6 +43,9 @@
       <mat-cell mat-cell *matCellDef="let element">
         <span
           class="cell-result-text"
+          [class.highlight]="
+            isNonCompliantAndRequired(element.result, element.required_result)
+          "
           [ngClass]="getResultClass(element.result)">
           <span>{{ element.result }}</span>
         </span>
diff --git a/modules/ui/src/app/pages/testrun/components/testrun-table/testrun-table.component.scss b/modules/ui/src/app/pages/testrun/components/testrun-table/testrun-table.component.scss
index 6f2ae801d..416be1de9 100644
--- a/modules/ui/src/app/pages/testrun/components/testrun-table/testrun-table.component.scss
+++ b/modules/ui/src/app/pages/testrun/components/testrun-table/testrun-table.component.scss
@@ -29,6 +29,14 @@ mat-header-row {
   cursor: pointer;
 }
 
+mat-row:has(.cell-result-text.highlight) {
+  background: colors.$error-container;
+  .cell-result-text.red {
+    color: colors.$white;
+    background: colors.$on-error-container;
+  }
+}
+
 mat-cell,
 mat-header-cell,
 .item-cell {
diff --git a/modules/ui/src/app/pages/testrun/components/testrun-table/testrun-table.component.ts b/modules/ui/src/app/pages/testrun/components/testrun-table/testrun-table.component.ts
index e3627b064..0a6a4d1c8 100644
--- a/modules/ui/src/app/pages/testrun/components/testrun-table/testrun-table.component.ts
+++ b/modules/ui/src/app/pages/testrun/components/testrun-table/testrun-table.component.ts
@@ -21,6 +21,8 @@ import {
 } from '@angular/core';
 import {
   IResult,
+  RequiredResult,
+  StatusOfTestResult,
   StatusResultClassName,
 } from '../../../../model/testrun-status';
 import { CalloutType } from '../../../../model/callout-type';
@@ -100,4 +102,11 @@ export class TestrunTableComponent {
   public trackTest(index: number, item: IResult) {
     return item.name + item.result;
   }
+
+  public isNonCompliantAndRequired(result: string, requiredResult: string) {
+    return (
+      result === StatusOfTestResult.NonCompliant &&
+      requiredResult === RequiredResult.Required
+    );
+  }
 }