From f77e10fb2e1cb79b27727908fa081c890ecf118f Mon Sep 17 00:00:00 2001 From: Mikio Hara Date: Fri, 1 May 2015 12:38:42 +0900 Subject: 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 Run-TryBot: Mikio Hara --- src/net/interface_linux_test.go | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) (limited to 'src/net/interface_linux_test.go') 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) } } -- cgit v1.3