qbit/go
1
0
Fork 0
mirror of https://github.com/golang/go synced 2024-09-30 08:28:34 -06:00
Code Releases Activity

path/filepath: use testenv.MustHaveSymlink to simplify symlink tests

Browse Source 
Cleanup test code for symbolic links.

Change-Id: I25f561cd34dc4d120a4143f933619d233a6cffc5
Reviewed-on: https://go-review.googlesource.com/27573
Reviewed-by: Brad Fitzpatrick <bradfitz@golang.org>
Run-TryBot: Brad Fitzpatrick <bradfitz@golang.org>
TryBot-Result: Gobot Gobot <gobot@golang.org>
This commit is contained in:
Hiroshi Ioka 2016-08-23 22:59:27 +09:00 committed by Brad Fitzpatrick
parent e893c72f2a
commit 03723c909e
3 changed files with 5 additions and 45 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

11
src/path/filepath/match_test.go
View File

@ -6,6 +6,7 @@ package filepath_test
import (
"fmt"
"internal/testenv"
"io/ioutil"
"os"
. "path/filepath"
@ -174,15 +175,7 @@ var globSymlinkTests = []struct {
}
func TestGlobSymlink(t *testing.T) {
switch runtime.GOOS {
case "android", "nacl", "plan9":
t.Skipf("skipping on %s", runtime.GOOS)
case "windows":
if !supportsSymlinks {
t.Skipf("skipping on %s", runtime.GOOS)
}
}
testenv.MustHaveSymlink(t)
tmpDir, err := ioutil.TempDir("", "globsymlink")
if err != nil {

19
src/path/filepath/path_test.go
View File

@ -6,6 +6,7 @@ package filepath_test
import (
"errors"
"internal/testenv"
"io/ioutil"
"os"
"path/filepath"
@ -15,8 +16,6 @@ import (
"testing"
)
var supportsSymlinks = true
type PathTest struct {
path, result string
}
@ -776,13 +775,7 @@ func simpleJoin(dir, path string) string {
}
func TestEvalSymlinks(t *testing.T) {
switch runtime.GOOS {
case "android", "nacl", "plan9":
t.Skipf("skipping on %s", runtime.GOOS)
}
if !supportsSymlinks {
t.Skip("skipping because symlinks are not supported")
}
testenv.MustHaveSymlink(t)
tmpDir, err := ioutil.TempDir("", "evalsymlink")
if err != nil {
@ -896,13 +889,7 @@ func TestEvalSymlinks(t *testing.T) {
}
func TestIssue13582(t *testing.T) {
switch runtime.GOOS {
case "android", "nacl", "plan9":
t.Skipf("skipping on %s", runtime.GOOS)
}
if !supportsSymlinks {
t.Skip("skipping because symlinks are not supported")
}
testenv.MustHaveSymlink(t)
tmpDir, err := ioutil.TempDir("", "issue13582")
if err != nil {

20
src/path/filepath/path_windows_test.go
View File

@ -13,29 +13,9 @@ import (
"path/filepath"
"reflect"
"strings"
"syscall"
"testing"
)
func init() {
tmpdir, err := ioutil.TempDir("", "symtest")
if err != nil {
panic("failed to create temp directory: " + err.Error())
}
defer os.RemoveAll(tmpdir)
err = os.Symlink("target", filepath.Join(tmpdir, "symlink"))
if err == nil {
return
}
err = err.(*os.LinkError).Err
switch err {
case syscall.EWINDOWS, syscall.ERROR_PRIVILEGE_NOT_HELD:
supportsSymlinks = false
}
}
func TestWinSplitListTestsAreValid(t *testing.T) {
comspec := os.Getenv("ComSpec")
if comspec == "" {