test: Use error.Is(errorType, errorInstance) to check for error equality (TheAlgorithms#666)

himanshu-patel-dev · web-flow · commit c129115f3397 · 2023-10-05T15:47:56.000+05:30
Checking error with == operator might fail on wrapped errors
diff --git a/graph/bellmanford_test.go b/graph/bellmanford_test.go
@@ -125,7 +125,7 @@ func TestBellmanford(t *testing.T) {
 			if resIsReachable != test.isReachable {
 				t.Errorf("Reachable, Expected: %t, Computed: %t", test.isReachable, resIsReachable)
 			}
-			if resError != test.err {
+			if !errors.Is(test.err, resError) {
 				if resError == nil || test.err == nil {
 					t.Errorf("Reachable, Expected: %s, Computed: %s", test.err, resError)
 				} else if resError.Error() != test.err.Error() {

Original file line number	Diff line number	Diff line change
`@@ -125,7 +125,7 @@ func TestBellmanford(t *testing.T) {`
`125`	`125`	`if resIsReachable != test.isReachable {`
`126`	`126`	`t.Errorf("Reachable, Expected: %t, Computed: %t", test.isReachable, resIsReachable)`
`127`	`127`	`}`
`128`		`- if resError != test.err {`
	`128`	`+ if !errors.Is(test.err, resError) {`
`129`	`129`	`if resError == nil \|\| test.err == nil {`
`130`	`130`	`t.Errorf("Reachable, Expected: %s, Computed: %s", test.err, resError)`
`131`	`131`	`} else if resError.Error() != test.err.Error() {`