]> Cypherpunks repositories - gostls13.git/commitdiff
misc/ios: use the default go test timeout
authorElias Naur <elias.naur@gmail.com>
Wed, 1 Feb 2017 11:28:47 +0000 (12:28 +0100)
committerElias Naur <elias.naur@gmail.com>
Wed, 1 Feb 2017 22:09:48 +0000 (22:09 +0000)
If -test.timeout is not specified to go test, it will time out after
a default 10 minutes.

The iOS exec wrapper also contains a fail safe timeout mechanism for
a stuck device. However, if no explicit -test.timeout is specified,
it will use a timeout of 0, plus some constant amount.

Use the same default timeout in the exec wrapper as for go test,
10 minutes.

Change-Id: I6465ccd9f7b9ce08fa302e6697f7938a0ea9af34
Reviewed-on: https://go-review.googlesource.com/36062
Reviewed-by: David Crawshaw <crawshaw@golang.org>
Run-TryBot: Elias Naur <elias.naur@gmail.com>
TryBot-Result: Gobot Gobot <gobot@golang.org>

misc/ios/go_darwin_arm_exec.go

index 9ec55b11be0caf6dac7191d55fe30c13a79ddade..bbc17f06ff9b8d80a1fc3bbf0c5e70c7de2989a6 100644 (file)
@@ -448,7 +448,7 @@ func parseArgs(binArgs []string) (opts options, remainingArgs []string) {
                remainingArgs = append(remainingArgs, arg)
        }
        f := flag.NewFlagSet("", flag.ContinueOnError)
-       f.DurationVar(&opts.timeout, "test.timeout", 0, "")
+       f.DurationVar(&opts.timeout, "test.timeout", 10*time.Minute, "")
        f.BoolVar(&opts.lldb, "lldb", false, "")
        f.Parse(flagArgs)
        return opts, remainingArgs