Skip to content

test: v1beta1 to v1alpha1 roundtrip fuzzing - #5249

Merged
soltysh merged 12 commits into
mainfrom
roundtrip-fuzzer
Aug 25, 2026
Merged

test: v1beta1 to v1alpha1 roundtrip fuzzing#5249
soltysh merged 12 commits into
mainfrom
roundtrip-fuzzer

Conversation

@AustinAbro321

@AustinAbro321 AustinAbro321 commented Aug 19, 2026

Copy link
Copy Markdown
Member

Description

Roundtripping is not 1:1 between v1beta1 and v1alpha1. This PR includes tests which serve as living documentation that we can reference to know which fields are not 1:1.

There is some extra logic done for charts and repos where oneOf is not respected by the fuzzer.

Checklist before merging

Signed-off-by: Austin Abro <austinabro321@gmail.com>
Signed-off-by: Austin Abro <austinabro321@gmail.com>
Signed-off-by: Austin Abro <austinabro321@gmail.com>
Signed-off-by: Austin Abro <austinabro321@gmail.com>
Signed-off-by: Austin Abro <austinabro321@gmail.com>
Signed-off-by: Austin Abro <austinabro321@gmail.com>
Signed-off-by: Austin Abro <austinabro321@gmail.com>
Signed-off-by: Austin Abro <austinabro321@gmail.com>
@netlify

netlify Bot commented Aug 19, 2026

Copy link
Copy Markdown

Deploy Preview for zarf-docs canceled.

Name Link
🔨 Latest commit 7f0b1c5
🔍 Latest deploy log https://app.netlify.com/projects/zarf-docs/deploys/6a8c82481d081900089d7536

Signed-off-by: Austin Abro <austinabro321@gmail.com>
@codecov

codecov Bot commented Aug 19, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

Files with missing lines Coverage Δ
src/api/v1beta1/component.go 81.81% <ø> (ø)
src/internal/api/v1alpha1/convert.go 99.51% <100.00%> (+2.30%) ⬆️
src/internal/api/v1beta1/convert.go 99.19% <100.00%> (+1.19%) ⬆️
src/test/testutil/fuzz.go 90.69% <100.00%> (+0.69%) ⬆️

... and 10 files with indirect coverage changes

🚀 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.

Signed-off-by: Austin Abro <austinabro321@gmail.com>
@AustinAbro321 AustinAbro321 changed the title test: roundtrip fuzzing test: v1beta1 to v1alpha1 roundtrip fuzzing Aug 19, 2026
Signed-off-by: Austin Abro <austinabro321@gmail.com>
@AustinAbro321
AustinAbro321 marked this pull request as ready for review August 19, 2026 19:46
@AustinAbro321
AustinAbro321 requested review from a team as code owners August 19, 2026 19:46

@brandtkeller brandtkeller left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

minor nit and question for my own education as it feels intentional.

Comment thread src/api/v1beta1/component.go Outdated
Comment thread src/internal/api/v1beta1/roundtrip_test.go
@github-project-automation github-project-automation Bot moved this to In progress in Zarf Aug 24, 2026
Signed-off-by: Austin Abro <austinabro321@gmail.com>

@brandtkeller brandtkeller left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

lgtm

@soltysh soltysh 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.

Non-blocking suggestion, I might open a follow-up fixing it, since I'd like to expand our fuzzer a bit.

lgtm

Comment thread src/test/testutil/fuzz.go
@soltysh
soltysh enabled auto-merge August 25, 2026 11:48
@soltysh
soltysh added this pull request to the merge queue Aug 25, 2026
Merged via the queue into main with commit 2ce2cc1 Aug 25, 2026
31 checks passed
@soltysh
soltysh deleted the roundtrip-fuzzer branch August 25, 2026 12:59
@github-project-automation github-project-automation Bot moved this from In progress to Done in Zarf Aug 25, 2026
chaospuppy pushed a commit to chaospuppy/zarf that referenced this pull request Sep 1, 2026
Signed-off-by: Austin Abro <austinabro321@gmail.com>
Signed-off-by: Tim Seagren <timseagren@defenseunicorns.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

3 participants