From 4dff1ffb5875e3e92f8774e1e6086e018c21a5d9 Mon Sep 17 00:00:00 2001 From: openshift-trt Date: Thu, 16 Jul 2026 19:14:42 +0000 Subject: [PATCH] TRT-2795: Fix variant filter matching in regressions table The variant filter comparison was extracting only the value part of regression variants via parseVariantName (e.g. "gcp" from "Platform:gcp") but comparing it against the full filter value ("Platform:gcp"), so the match always failed. Fix by comparing the full variant string first, then falling back to the parsed name for filters without a key prefix (e.g. "never-stable", "aggregated"). Co-Authored-By: Claude Opus 4.6 --- sippy-ng/src/tests/TestRegressionsTable.js | 11 ++++++----- 1 file changed, 6 insertions(+), 5 deletions(-) diff --git a/sippy-ng/src/tests/TestRegressionsTable.js b/sippy-ng/src/tests/TestRegressionsTable.js index a91bfd15d..c422bd4b3 100644 --- a/sippy-ng/src/tests/TestRegressionsTable.js +++ b/sippy-ng/src/tests/TestRegressionsTable.js @@ -68,13 +68,14 @@ export default function TestRegressionsTable({ if (variantFilters.length === 0) return filtered return filtered.filter((regression) => { - const variantValues = (regression.variants || []).map( - (v) => parseVariantName(v).name - ) + const variants = regression.variants || [] return variantFilters.every((filter) => { - const hasMatch = variantValues.some( - (v) => v.toLowerCase() === filter.value.toLowerCase() + const filterVal = filter.value.toLowerCase() + const hasMatch = variants.some( + (v) => + v.toLowerCase() === filterVal || + parseVariantName(v).name.toLowerCase() === filterVal ) if (filter.not) { return !hasMatch