diff options
| author | Robert Griesemer <gri@golang.org> | 2009-12-15 15:40:16 -0800 |
|---|---|---|
| committer | Robert Griesemer <gri@golang.org> | 2009-12-15 15:40:16 -0800 |
| commit | d65a5cce89596e0256280ab8318548d3fa5bb09b (patch) | |
| tree | a6ffc3d3f824cca65bc931bf0f045c8af78bc62b /src/pkg/path/path_test.go | |
| parent | a3d1045fb7b34355c9859aea2781e0d77004cc82 (diff) | |
| download | go-d65a5cce89596e0256280ab8318548d3fa5bb09b.tar.xz | |
1) Change default gofmt default settings for
parsing and printing to new syntax.
Use -oldparser to parse the old syntax,
use -oldprinter to print the old syntax.
2) Change default gofmt formatting settings
to use tabs for indentation only and to use
spaces for alignment. This will make the code
alignment insensitive to an editor's tabwidth.
Use -spaces=false to use tabs for alignment.
3) Manually changed src/exp/parser/parser_test.go
so that it doesn't try to parse the parser's
source files using the old syntax (they have
new syntax now).
4) gofmt -w src misc test/bench
4th set of files.
R=rsc
CC=golang-dev
https://golang.org/cl/180049
Diffstat (limited to 'src/pkg/path/path_test.go')
| -rw-r--r-- | src/pkg/path/path_test.go | 74 |
1 files changed, 37 insertions, 37 deletions
diff --git a/src/pkg/path/path_test.go b/src/pkg/path/path_test.go index 74520fa315..296712e522 100644 --- a/src/pkg/path/path_test.go +++ b/src/pkg/path/path_test.go @@ -5,12 +5,12 @@ package path import ( - "os"; - "testing"; + "os" + "testing" ) type CleanTest struct { - path, clean string; + path, clean string } var cleantests = []CleanTest{ @@ -72,7 +72,7 @@ func TestClean(t *testing.T) { } type SplitTest struct { - path, dir, file string; + path, dir, file string } var splittests = []SplitTest{ @@ -92,7 +92,7 @@ func TestSplit(t *testing.T) { } type JoinTest struct { - dir, file, path string; + dir, file, path string } var jointests = []JoinTest{ @@ -114,7 +114,7 @@ func TestJoin(t *testing.T) { } type ExtTest struct { - path, ext string; + path, ext string } var exttests = []ExtTest{ @@ -134,9 +134,9 @@ func TestExt(t *testing.T) { } type Node struct { - name string; - entries []*Node; // nil if the entry is a file - mark int; + name string + entries []*Node // nil if the entry is a file + mark int } var tree = &Node{ @@ -166,7 +166,7 @@ var tree = &Node{ } func walkTree(n *Node, path string, f func(path string, n *Node)) { - f(path, n); + f(path, n) for _, e := range n.entries { walkTree(e, Join(path, e.name), f) } @@ -175,25 +175,25 @@ func walkTree(n *Node, path string, f func(path string, n *Node)) { func makeTree(t *testing.T) { walkTree(tree, tree.name, func(path string, n *Node) { if n.entries == nil { - fd, err := os.Open(path, os.O_CREAT, 0660); + fd, err := os.Open(path, os.O_CREAT, 0660) if err != nil { t.Errorf("makeTree: %v", err) } - fd.Close(); + fd.Close() } else { os.Mkdir(path, 0770) } }) } -func markTree(n *Node) { walkTree(n, "", func(path string, n *Node) { n.mark++ }) } +func markTree(n *Node) { walkTree(n, "", func(path string, n *Node) { n.mark++ }) } func checkMarks(t *testing.T) { walkTree(tree, tree.name, func(path string, n *Node) { if n.mark != 1 { t.Errorf("node %s mark = %d; expected 1", path, n.mark) } - n.mark = 0; + n.mark = 0 }) } @@ -209,8 +209,8 @@ func mark(name string) { type TestVisitor struct{} func (v *TestVisitor) VisitDir(path string, d *os.Dir) bool { - mark(d.Name); - return true; + mark(d.Name) + return true } func (v *TestVisitor) VisitFile(path string, d *os.Dir) { @@ -218,52 +218,52 @@ func (v *TestVisitor) VisitFile(path string, d *os.Dir) { } func TestWalk(t *testing.T) { - makeTree(t); + makeTree(t) // 1) ignore error handling, expect none - v := &TestVisitor{}; - Walk(tree.name, v, nil); - checkMarks(t); + v := &TestVisitor{} + Walk(tree.name, v, nil) + checkMarks(t) // 2) handle errors, expect none - errors := make(chan os.Error, 64); - Walk(tree.name, v, errors); + errors := make(chan os.Error, 64) + Walk(tree.name, v, errors) if err, ok := <-errors; ok { t.Errorf("no error expected, found: s", err) } - checkMarks(t); + checkMarks(t) if os.Getuid() != 0 { // introduce 2 errors: chmod top-level directories to 0 - os.Chmod(Join(tree.name, tree.entries[1].name), 0); - os.Chmod(Join(tree.name, tree.entries[3].name), 0); + os.Chmod(Join(tree.name, tree.entries[1].name), 0) + os.Chmod(Join(tree.name, tree.entries[3].name), 0) // mark respective subtrees manually - markTree(tree.entries[1]); - markTree(tree.entries[3]); + markTree(tree.entries[1]) + markTree(tree.entries[3]) // correct double-marking of directory itself - tree.entries[1].mark--; - tree.entries[3].mark--; + tree.entries[1].mark-- + tree.entries[3].mark-- // 3) handle errors, expect two - errors = make(chan os.Error, 64); - os.Chmod(Join(tree.name, tree.entries[1].name), 0); - Walk(tree.name, v, errors); + errors = make(chan os.Error, 64) + os.Chmod(Join(tree.name, tree.entries[1].name), 0) + Walk(tree.name, v, errors) for i := 1; i <= 2; i++ { if _, ok := <-errors; !ok { - t.Errorf("%d. error expected, none found", i); - break; + t.Errorf("%d. error expected, none found", i) + break } } if err, ok := <-errors; ok { t.Errorf("only two errors expected, found 3rd: %v", err) } // the inaccessible subtrees were marked manually - checkMarks(t); + checkMarks(t) } // cleanup - os.Chmod(Join(tree.name, tree.entries[1].name), 0770); - os.Chmod(Join(tree.name, tree.entries[3].name), 0770); + os.Chmod(Join(tree.name, tree.entries[1].name), 0770) + os.Chmod(Join(tree.name, tree.entries[3].name), 0770) if err := os.RemoveAll(tree.name); err != nil { t.Errorf("removeTree: %v", err) } |
