Skip to content

chore: add more tests, fuzzing, and a separate fuzz Workflow #1352

Merged
egibs merged 2 commits intochainguard-dev:mainfrom
egibs:more-tests-fuzzing
Feb 4, 2026
Merged

chore: add more tests, fuzzing, and a separate fuzz Workflow #1352
egibs merged 2 commits intochainguard-dev:mainfrom
egibs:more-tests-fuzzing

Conversation

@egibs
Copy link
Member

@egibs egibs commented Feb 4, 2026

This PR adds additional testing and fuzzing coverage for some of the core functionality. Some of what malcontent does is harder to test outside of running a refresh or actual scans, but this will give us additional confidence that the basic concepts are behaving as they should.

It was also getting a little unwieldy to run so manny fuzzers serially during CI so I added a separate Workflow to run these on push to main, on a schedule, and via workflow_dispatch.

Of note, some of the tests resulted in small tweaks to general functionality which is exactly the point of having this kind of coverage. The main change here is how some third-party rules are rendered.

@egibs egibs force-pushed the more-tests-fuzzing branch 3 times, most recently from 1c05005 to b04ef1f Compare February 4, 2026 15:22
@egibs egibs changed the title [wip] add more tests and fuzzing chore: add more tests, fuzzing, and a separate fuzz Workflow Feb 4, 2026
@egibs egibs requested a review from antitree February 4, 2026 15:29
@egibs egibs marked this pull request as ready for review February 4, 2026 15:29
@egibs egibs force-pushed the more-tests-fuzzing branch from 31d6a27 to 0a9a4ef Compare February 4, 2026 15:43
Signed-off-by: egibs <20933572+egibs@users.noreply.github.com>
@egibs egibs force-pushed the more-tests-fuzzing branch from 0a9a4ef to 78ddb8a Compare February 4, 2026 16:04
Copy link
Contributor

@eslerm eslerm left a comment

Choose a reason for hiding this comment

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

Looks good. Nice fuzzing workflow. Note some rule name changes. Fuzzing workflow may desire harden-runner. Instead of on push, may be best as cron. I had a friend setup fuzzing to use up any free ci left over.

@egibs egibs merged commit 515b776 into chainguard-dev:main Feb 4, 2026
17 checks passed
@egibs egibs deleted the more-tests-fuzzing branch February 5, 2026 03:35
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