Skip to content

Commit df4d992

Browse files
committed
test(operator): exclude PlacementStatus from cross-field fuzz filler
Register no-op custom fuzzers for the v1alpha1 and v1beta1 PlacementStatus so the round-trip fuzz filler does not draw random bits for Placement. Rationale: - Placement round-trip is already covered explicitly by TestDGD_RoundTrip_Status (Score=ptr.To(0.87), State=Reported). - Not consuming RNG bits for Placement keeps the fuzz stream stable as DGDStatus grows, so field additions do not shift the stream onto payloads that surface unrelated latent bugs in downstream conversion paths. - Follows the existing dynamoFuzzerFuncs pattern for constraining fuzzer input to admissible shapes. Signed-off-by: ashnamehrotra <ashnamehrotra@gmail.com>
1 parent e3b76e7 commit df4d992

1 file changed

Lines changed: 8 additions & 0 deletions

File tree

deploy/operator/api/roundtrip_fuzz_test.go

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -216,6 +216,14 @@ func dynamoFuzzerFuncs(_ runtimeserializer.CodecFactory) []any {
216216
fuzzAlphaDGDRStatus,
217217
fuzzBetaDGDRSpec,
218218
fuzzBetaDGDRStatus,
219+
// PlacementStatus (v1alpha1 + v1beta1): keep the fuzz filler from
220+
// drawing random bits for this pointer field. Placement conversion
221+
// is exercised by TestDGD_RoundTrip_Status; excluding it here keeps
222+
// the fuzz RNG stream stable across schema growth on DGDStatus so
223+
// that field additions do not surface unrelated latent bugs in
224+
// downstream conversion paths.
225+
func(p *v1beta1.PlacementStatus, _ randfill.Continue) {},
226+
func(p *v1alpha1.PlacementStatus, _ randfill.Continue) {},
219227
// v1beta1 Components: the listMapKey marker requires name
220228
// to be non-empty and unique; MaxItems caps the length at 25.
221229
// Enforce both so the input is admissible.

0 commit comments

Comments
 (0)