github/ping
1
0
Fork 0
mirror of https://github.com/go-ping/ping.git synced 2025-08-12 18:25:58 +00:00
Code Issues Packages Projects Releases Wiki Activity

test: improve assertions and remove stack traces

Browse Source 
Seems like these stack traces are not necessary, as the information which assertion actually failed is already figured out by "go test" in a less convoluted way.
This commit is contained in:
Florian Loch 2022-02-28 12:47:34 +01:00
parent bca0870d92
commit dbd1b5c8a8
1 changed files with 7 additions and 11 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

18
ping_test.go
View File

@ -5,7 +5,6 @@ import (
"errors"
"fmt"
"net"
"runtime/debug"
"sync"
"sync/atomic"
"testing"
@ -492,46 +491,43 @@ func makeTestPinger() *Pinger {
func AssertNoError(t *testing.T, err error) {
t.Helper()
if err != nil {
t.Errorf("Expected No Error but got %s, Stack:\n%s",
err, string(debug.Stack()))
t.Errorf("Expected No Error but got %q", err)
}
}
func AssertError(t *testing.T, err error, info string) {
t.Helper()
if err == nil {
t.Errorf("Expected Error but got %s, %s, Stack:\n%s",
err, info, string(debug.Stack()))
t.Errorf("Expected Error %q but got %q",
err, info)
}
}
func AssertEqualStrings(t *testing.T, expected, actual string) {
t.Helper()
if expected != actual {
t.Errorf("Expected %s, got %s, Stack:\n%s",
expected, actual, string(debug.Stack()))
t.Errorf("Expected %s, got %s", expected, actual)
}
}
func AssertNotEqualStrings(t *testing.T, expected, actual string) {
t.Helper()
if expected == actual {
t.Errorf("Expected %s, got %s, Stack:\n%s",
expected, actual, string(debug.Stack()))
t.Errorf("Expected %q, got %q", expected, actual)
}
}
func AssertTrue(t *testing.T, b bool) {
t.Helper()
if !b {
t.Errorf("Expected True, got False, Stack:\n%s", string(debug.Stack()))
t.Errorf("Expected True, got False")
}
}
func AssertFalse(t *testing.T, b bool) {
t.Helper()
if b {
t.Errorf("Expected False, got True, Stack:\n%s", string(debug.Stack()))
t.Errorf("Expected False, got True")
}
}