Merge pull request #4885 from justinsb/cleanup_fuzz_tests

tests: move fuzz tests to standard dev/ci/presubmits

Author: google-oss-prow[bot]

.github/workflows/ci-presubmit.yaml

@@ -34,6 +34,26 @@ on:
 
 jobs:
 
+  fuzz-roundtrippers:
+    runs-on: ubuntu-latest
+    timeout-minutes: 60
+    steps:
+      - uses: actions/checkout@v4
+      - uses: actions/setup-go@v5
+        with:
+          go-version-file: 'go.mod'
+      - name: "Run dev/ci/presubmits/fuzz-roundtrippers"
+        run: |
+          ./dev/ci/presubmits/fuzz-roundtrippers
+        env:
+          ARTIFACTS: /tmp/artifacts
+      - name: "Upload artifacts"
+        uses: actions/upload-artifact@v4
+        with:
+          name: artifacts-fuzz-roundtrippers
+          path: /tmp/artifacts/
+
+
   test-mockgcp:
     runs-on: ubuntu-latest
     timeout-minutes: 60

.github/workflows/fuzz.yaml

@@ -22,6 +22,7 @@ cd ${REPO_ROOT}
 
 go test -v ./pkg/fuzztesting/fuzztests/ -fuzz=FuzzAllMappers -fuzztime 600s # fuzz for 10 minutes because we are fuzzing all the registered mappers
 
+# Please do not add more fuzzers here; register them so they are run with FuzzAllMappers (or a similar approach)
 go test -v ./pkg/controller/direct/apigee/ -fuzz=FuzzApigeeEndpointAttachmentSpec -fuzztime 60s
 go test -v ./pkg/controller/direct/apigee/ -fuzz=FuzzApigeeEndpointAttachmentObservedState -fuzztime 60s
 go test -v ./pkg/controller/direct/apigee/ -fuzz=FuzzApigeeEnvgroupSpec -fuzztime 60s