Skip to content

task: report test tweaks and dependency lock updates#1593

Merged
jmartin-tech merged 4 commits intoNVIDIA:mainfrom
otavionvidia:kaizen-implementation
Feb 2, 2026
Merged

task: report test tweaks and dependency lock updates#1593
jmartin-tech merged 4 commits intoNVIDIA:mainfrom
otavionvidia:kaizen-implementation

Conversation

@otavionvidia
Copy link
Contributor

@otavionvidia otavionvidia commented Feb 2, 2026

What this change does

Fixes test failures that occur after updating yarn.lock to latest compatible versions, and improves test coverage to meet the 85% function threshold.

Changes:

  • Added missing KUI component mocks (Badge, Stack, Tooltip, Button) to test files
  • Updated ModuleFilterChips tests to match new component behavior (label changed from "Filter by module:" to "Modules")
  • Fixed ProbesChart.test.tsx to handle multiple tooltip elements
  • Added new test files for improved coverage:
    • DetectorChartHeader.test.tsx
    • DetectorFilters.test.tsx
    • ErrorBoundary.test.tsx
  • Added module filtering tests to ProbesChart.test.tsx

Verification

  • cd garak-report && npm install
  • npm test - All 244 tests should pass
  • npm run build - Should build successfully

<Text kind="label/regular/sm" className="text-secondary">
{isInteractive ? "Filter by module:" : "Module:"}
</Text>
<Stack gap="density-xs" paddingTop="density-sm" paddingBottom="density-md">
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cosmetic change in probes header - this is just so things looks more aligned

@jmartin-tech jmartin-tech changed the title Kaizen implementation task: report test tweaks and dependency lock updates Feb 2, 2026
@jmartin-tech jmartin-tech merged commit fa2c034 into NVIDIA:main Feb 2, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants