go: Fix fuzzer building to fail on error

.buildkite/code.pipeline.yml

@@ -184,6 +184,7 @@ steps:
       # TODO: Consider making this a part of the development Docker image.
       - go get -u github.com/dvyukov/go-fuzz/go-fuzz github.com/dvyukov/go-fuzz/go-fuzz-build
       - make -C go build-fuzz
+    soft_fail: true
     retry:
       <<: *retry_agent_failure
     plugins:

go/Makefile

@@ -92,17 +92,17 @@ if [ "$(FUZZ_NO_BUILD)" != "1" ]; then \
 	mkdir -p "$$WORKDIR"; \
 	pushd $$TARGETDIR >/dev/null; \
 		$(ECHO) "$(CYAN)*** Building fuzzer for $@...$(OFF)"; \
-		go-fuzz-build -o $$WORKDIR/fuzz.zip; \
+		go-fuzz-build -o $$WORKDIR/fuzz.zip || exit 1; \
 	popd >/dev/null; \
 fi; \
 if [ "$(FUZZ_BUILD_ONLY)" != "1" ]; then \
 	mkdir -p "$$WORKDIR"; \
 	cd "$$WORKDIR"; \
 	$(ECHO) "$(CYAN)*** Running fuzzer for $@...$(OFF)"; \
 	if [ "$(@D)" == "." ]; then \
-		go-fuzz -bin=$$WORKDIR/fuzz.zip -workdir=$$WORKDIR; \
+		go-fuzz -bin=$$WORKDIR/fuzz.zip -workdir=$$WORKDIR || exit 1; \
 	else \
-		go-fuzz -bin=$$WORKDIR/fuzz.zip -workdir=$$WORKDIR -func Fuzz$(@F); \
+		go-fuzz -bin=$$WORKDIR/fuzz.zip -workdir=$$WORKDIR -func Fuzz$(@F) || exit 1; \
 	fi; \
 fi;
 endef

go/consensus/tendermint/fuzz/fuzz.go

@@ -14,6 +14,7 @@ import (
 	"github.com/oasisprotocol/oasis-core/go/common/fuzz"
 	"github.com/oasisprotocol/oasis-core/go/consensus/api/transaction"
 	"github.com/oasisprotocol/oasis-core/go/consensus/tendermint/abci"
+	"github.com/oasisprotocol/oasis-core/go/consensus/tendermint/api"
 	epochtimemock "github.com/oasisprotocol/oasis-core/go/consensus/tendermint/apps/epochtime_mock"
 	registryApp "github.com/oasisprotocol/oasis-core/go/consensus/tendermint/apps/registry"
 	stakingApp "github.com/oasisprotocol/oasis-core/go/consensus/tendermint/apps/staking"
@@ -24,7 +25,7 @@ var (
 	txSigner signature.Signer = memory.NewTestSigner("consensus-fuzz")
 
 	// FuzzableApps is a list of ABCI apps the fuzzer can fuzz.
-	FuzzableApps []abci.Application = []abci.Application{
+	FuzzableApps []api.Application = []api.Application{
 		epochtimemock.New(),
 		registryApp.New(),
 		stakingApp.New(),
@@ -48,11 +49,11 @@ func Fuzz(data []byte) int {
 	var pruneCfg abci.PruneConfig
 
 	appConfig := &abci.ApplicationConfig{
-		DataDir:            "/tmp/oasis-node-fuzz-consensus",
-		Pruning:            pruneCfg,
-		HaltEpochHeight:    1000000,
-		MinGasPrice:        1,
-		CheckpointInterval: 1 * time.Minute,
+		DataDir:                   "/tmp/oasis-node-fuzz-consensus",
+		Pruning:                   pruneCfg,
+		HaltEpochHeight:           1000000,
+		MinGasPrice:               1,
+		CheckpointerCheckInterval: 1 * time.Minute,
 	}
 
 	// The muxer will start with the previous state, if it exists (the state database isn't cleared).

go/storage/fuzz/fuzz.go

@@ -26,7 +26,10 @@ var (
 )
 
 func init() {
-	signerFactory := file.NewFactory(identityDir, signature.SignerNode, signature.SignerP2P, signature.SignerConsensus)
+	signerFactory, err := file.NewFactory(identityDir, signature.SignerNode, signature.SignerP2P, signature.SignerConsensus)
+	if err != nil {
+		panic(err)
+	}
 	identity, err := identity.Load(identityDir, signerFactory)
 	if err != nil {
 		panic(err)