[dev.fuzz] cmd/go: default value of fuzzminimizetime flag is stated twice

[picatz]

👋 Hello! I am really excited about the native fuzzing beta, this is awesome!

I stumbled upon something fairly minor, but I figured I'd make an issue.

$ gotip version
go version devel go1.17-106851ad73 Wed Jun 2 22:39:15 2021 +0000 linux/amd64

$ gotip test -fuzz=FuzzParseQuery -bad-flag-to-trigger-usage-information             
...
  -test.fuzzminimizetime value
    	time to spend minimizing a value after finding a crash; default is to minimize for 60s (default 1m0s)
...

☝️ I would've expected it just to state (default 1m0s) like other flags with default values.

The flag usage is defined here:

go/src/testing/fuzz.go

Line 23 in 106851a

flag.Var(&minimizeDuration, "test.fuzzminimizetime", "time to spend minimizing a value after finding a crash; default is to minimize for 60s")

With a default set here that seems to generate the (default 1m0s) portion:

go/src/testing/fuzz.go

Line 31 in 106851a

minimizeDuration = durationOrCountFlag{d: 60 * time.Second}

[katiehockman]

Thanks for filing this! We should fix that.

[gopherbot]

Change https://golang.org/cl/325070 mentions this issue: [dev.fuzz] testing: fix documentation for fuzzminimizetime