Commit eb033b1d authored by Caleb Spare's avatar Caleb Spare Committed by Brad Fitzpatrick

compress/gzip: clean up zero-mtimes test

- Fix a typo.
- Skip this test on -short on non-builders.

Change-Id: Id102eceb59451694bf92b618e02ccee6603b6852
Reviewed-on: https://go-review.googlesource.com/21113Reviewed-by: default avatarBrad Fitzpatrick <bradfitz@golang.org>
Run-TryBot: Brad Fitzpatrick <bradfitz@golang.org>
parent b88147c3
package gzip package gzip
import ( import (
"internal/testenv"
"os" "os"
"path/filepath" "path/filepath"
"runtime" "runtime"
...@@ -9,9 +10,12 @@ import ( ...@@ -9,9 +10,12 @@ import (
"time" "time"
) )
// Per golang.org/issue/14939, check that every .gz file // Per golang.org/issue/14937, check that every .gz file
// in the tree has a zero mtime. // in the tree has a zero mtime.
func TestGZIPFilesHaveZeroMTimes(t *testing.T) { func TestGZIPFilesHaveZeroMTimes(t *testing.T) {
if testing.Short() && testenv.Builder() == "" {
t.Skip("skipping in short mode")
}
var files []string var files []string
err := filepath.Walk(runtime.GOROOT(), func(path string, info os.FileInfo, err error) error { err := filepath.Walk(runtime.GOROOT(), func(path string, info os.FileInfo, err error) error {
if err != nil { if err != nil {
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment