From 2cc62cdf407d51f17d143b7e7ef38f27bbded6c2 Mon Sep 17 00:00:00 2001 From: Nishant Das Date: Wed, 1 Jun 2022 21:15:50 +0800 Subject: [PATCH] Fix Fuzzing (#10798) --- .github/workflows/go.yml | 8 ++++---- crypto/bls/blst/stub.go | 4 ++-- fuzzbuzz.yaml | 1 - 3 files changed, 6 insertions(+), 7 deletions(-) diff --git a/.github/workflows/go.yml b/.github/workflows/go.yml index 1e57b14406..fd15171fe1 100644 --- a/.github/workflows/go.yml +++ b/.github/workflows/go.yml @@ -89,10 +89,10 @@ jobs: # Use blst tag to allow go and bazel builds for blst. run: go build -v ./... - # fuzz and blst_disabled leverage go tag based stubs at compile time. - # Building with these tags should be checked and enforced at pre-submit. - - name: Build for fuzzing - run: go build -tags=fuzz,blst_disabled ./... + # fuzz leverage go tag based stubs at compile time. + # Building and testing with these tags should be checked and enforced at pre-submit. + - name: Test for fuzzing + run: go test -tags=fuzz,develop ./... -test.run=^Fuzz # Tests run via Bazel for now... # - name: Test diff --git a/crypto/bls/blst/stub.go b/crypto/bls/blst/stub.go index b6c3323a22..9e37ad9dd2 100644 --- a/crypto/bls/blst/stub.go +++ b/crypto/bls/blst/stub.go @@ -1,5 +1,5 @@ -//go:build blst_disabled || fuzz -// +build blst_disabled fuzz +//go:build blst_disabled +// +build blst_disabled package blst diff --git a/fuzzbuzz.yaml b/fuzzbuzz.yaml index 58382e86a3..7f5b0f9bd0 100644 --- a/fuzzbuzz.yaml +++ b/fuzzbuzz.yaml @@ -2,4 +2,3 @@ base: language: go build_tags: - fuzz - - blst_disabled