diff options
| author | Mikio Hara <mikioh.mikioh@gmail.com> | 2015-05-01 12:38:42 +0900 |
|---|---|---|
| committer | Mikio Hara <mikioh.mikioh@gmail.com> | 2015-05-06 09:25:08 +0000 |
| commit | f77e10fb2e1cb79b27727908fa081c890ecf118f (patch) | |
| tree | c77dc4c460670fdd1644be8c58d6bd8136a291a7 /src/net/interface_linux_test.go | |
| parent | 9b184fd23c47a8eae3587b247d05b25f8cab04ed (diff) | |
| download | go-git-f77e10fb2e1cb79b27727908fa081c890ecf118f.tar.gz | |
net: simplify error messages in tests
This change simplifies unnecessarily redundant error messages in tests.
There's no need to worry any more because package APIs now return
consistent, self-descriptive error values.
Alos renames ambiguous test functions and makes use of test tables.
Change-Id: I7b61027607c4ae2a3cf605d08d58cf449fa27eb2
Reviewed-on: https://go-review.googlesource.com/9662
Reviewed-by: Ian Lance Taylor <iant@golang.org>
Run-TryBot: Mikio Hara <mikioh.mikioh@gmail.com>
Diffstat (limited to 'src/net/interface_linux_test.go')
| -rw-r--r-- | src/net/interface_linux_test.go | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/src/net/interface_linux_test.go b/src/net/interface_linux_test.go index d8800bd0ce..059bde11c6 100644 --- a/src/net/interface_linux_test.go +++ b/src/net/interface_linux_test.go @@ -78,7 +78,7 @@ var ( func TestParseProcNet(t *testing.T) { defer func() { if p := recover(); p != nil { - t.Fatalf("parseProcNetIGMP or parseProtNetIGMP6 panicked: %v", p) + t.Fatalf("panicked: %v", p) } }() @@ -88,7 +88,7 @@ func TestParseProcNet(t *testing.T) { ifmat4 = append(ifmat4, ifmat...) } if len(ifmat4) != numOfTestIPv4MCAddrs { - t.Fatalf("parseProcNetIGMP returns %v addresses, expected %v", len(ifmat4), numOfTestIPv4MCAddrs) + t.Fatalf("got %d; want %d", len(ifmat4), numOfTestIPv4MCAddrs) } var ifmat6 []Addr @@ -97,6 +97,6 @@ func TestParseProcNet(t *testing.T) { ifmat6 = append(ifmat6, ifmat...) } if len(ifmat6) != numOfTestIPv6MCAddrs { - t.Fatalf("parseProcNetIGMP6 returns %v addresses, expected %v", len(ifmat6), numOfTestIPv6MCAddrs) + t.Fatalf("got %d; want %d", len(ifmat6), numOfTestIPv6MCAddrs) } } |