Skip to content

test: add missing tests for transform functions#1239

Open
fabriziosestito wants to merge 1 commit into
kubewarden:mainfrom
fabriziosestito:test/add-missing-transform-test
Open

test: add missing tests for transform functions#1239
fabriziosestito wants to merge 1 commit into
kubewarden:mainfrom
fabriziosestito:test/add-missing-transform-test

Conversation

@fabriziosestito

Copy link
Copy Markdown
Collaborator

Description

Add missing transform tests

Signed-off-by: Fabrizio Sestito <fabrizio.sestito@suse.com>

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

This PR adds missing unit tests for storage-layer cache transform helpers, ensuring the “strip” transforms correctly remove large payload fields while preserving object identity metadata.

Changes:

  • Add a test for TransformStripNodeSBOM verifying the SPDX payload is removed and name is preserved.
  • Add a test for TransformStripNodeVulnerabilityReport verifying the report results are removed and name is preserved.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@codecov

codecov Bot commented Jun 16, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 49.94%. Comparing base (5adb1bd) to head (d730f42).
⚠️ Report is 36 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1239      +/-   ##
==========================================
- Coverage   53.44%   49.94%   -3.50%     
==========================================
  Files          61       77      +16     
  Lines        5340     6479    +1139     
==========================================
+ Hits         2854     3236     +382     
- Misses       2088     2784     +696     
- Partials      398      459      +61     
Flag Coverage Δ
unit-tests 49.94% <ø> (-3.50%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: Pending Review

Development

Successfully merging this pull request may close these issues.

2 participants