Skip to content

fix: introduce more randomization in the fuzzers - #5275

Open
soltysh wants to merge 1 commit into
zarf-dev:mainfrom
soltysh:improve_fuzzing
Open

fix: introduce more randomization in the fuzzers#5275
soltysh wants to merge 1 commit into
zarf-dev:mainfrom
soltysh:improve_fuzzing

Conversation

@soltysh

@soltysh soltysh commented Aug 25, 2026

Copy link
Copy Markdown
Contributor

Description

This change:

  • increases the length randomization for slices up to 10
  • increases the length randomization for strings up to 20
  • introduces unicode coverage for strings
  • switches to using 64-bit randoms for uints, ints and floats
  • shortens the number of iterations from 1000 to 20

This is followup to #5249

Checklist before merging

@soltysh
soltysh requested review from a team as code owners August 25, 2026 15:01
@netlify

netlify Bot commented Aug 25, 2026

Copy link
Copy Markdown

Deploy Preview for zarf-docs canceled.

Name Link
🔨 Latest commit 3648a02
🔍 Latest deploy log https://app.netlify.com/projects/zarf-docs/deploys/6a997f4fa822ad0008ea58f4

This change:
- increases the length randomization for slices up to 10
- increases the length randomization for strings up to 20
- introduces unicode coverage for strings
- switches to using 64-bit randoms for uints, ints and floats
- shortens the number of iterations from 1000 to 20

Signed-off-by: Maciej Szulik <maciej@defenseunicorns.com>
@codecov

codecov Bot commented Sep 3, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 87.50000% with 2 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
src/test/testutil/fuzz.go 87.50% 2 Missing ⚠️
Files with missing lines Coverage Δ
src/test/testutil/fuzz.go 92.30% <87.50%> (+1.61%) ⬆️

... and 1 file 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.

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

Labels

None yet

Projects

Status: No status

Development

Successfully merging this pull request may close these issues.

2 participants