Skip to content
/ go Public
forked from golang/go

Commit

Permalink
crypto/internal/fipstest: fix TestACVP not to write to module cache
Browse files Browse the repository at this point in the history
Not sure how this ever worked; the module cache is read-only.

Change-Id: I22e4945fa0bac101995ab9101d22dcfdcee40776
Reviewed-on: https://go-review.googlesource.com/c/go/+/629976
Reviewed-by: Dmitri Shuralyov <[email protected]>
Reviewed-by: Filippo Valsorda <[email protected]>
LUCI-TryBot-Result: Go LUCI <[email protected]>
  • Loading branch information
rsc committed Nov 20, 2024
1 parent 5165f54 commit 30b09bc
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion src/crypto/internal/fipstest/acvp_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -368,9 +368,11 @@ func TestACVP(t *testing.T) {
t.Log("building acvptool")

// Build the acvptool binary.
toolPath := filepath.Join(t.TempDir(), "acvptool.exe")
goTool := testenv.GoToolPath(t)
cmd := testenv.Command(t, goTool,
"build",
"-o", toolPath,
"./util/fipstools/acvp/acvptool")
cmd.Dir = bsslDir
out := &strings.Builder{}
Expand All @@ -387,7 +389,6 @@ func TestACVP(t *testing.T) {
t.Fatalf("failed to fetch cwd: %s", err)
}
configPath := filepath.Join(cwd, "acvp_test.config.json")
toolPath := filepath.Join(bsslDir, "acvptool")
t.Logf("running check_expected.go\ncwd: %q\ndata_dir: %q\nconfig: %q\ntool: %q\nmodule-wrapper: %q\n",
cwd, dataDir, configPath, toolPath, os.Args[0])

Expand Down

0 comments on commit 30b09bc

Please sign in to comment.