Refactor: changed the errors.Contains function

nyagamunene · nyagamunene · commit 1a5997a9793e · 2024-06-12T17:54:02.000+03:00
Signed-off-by: nyagamunene &lt;stevenyaga2014@gmail.com&gt;
diff --git a/pkg/errors/errors.go b/pkg/errors/errors.go
@@ -5,6 +5,7 @@ package errors
 
 import (
 	"encoding/json"
+	"errors"
 )
 
 // Error specifies an API that must be fullfiled by error type.
@@ -80,9 +81,9 @@ func Contains(e1, e2 error) bool {
 		if ce.Msg() == e2.Error() {
 			return true
 		}
-		return Contains(ce.Err(), e2)
+		return errors.Is(ce.Err(), e2)
 	}
-	return e1.Error() == e2.Error()
+	return errors.Is(e1, e2)
 }
 
 // Wrap returns an Error that wrap err with wrapper.

Original file line number	Diff line number	Diff line change
`@@ -5,6 +5,7 @@ package errors`
`5`	`5`
`6`	`6`	`import (`
`7`	`7`	`"encoding/json"`
	`8`	`+ "errors"`
`8`	`9`	`)`
`9`	`10`
`10`	`11`	`// Error specifies an API that must be fullfiled by error type.`
`@@ -80,9 +81,9 @@ func Contains(e1, e2 error) bool {`
`80`	`81`	`if ce.Msg() == e2.Error() {`
`81`	`82`	`return true`
`82`	`83`	`}`
`83`		`- return Contains(ce.Err(), e2)`
	`84`	`+ return errors.Is(ce.Err(), e2)`
`84`	`85`	`}`
`85`		`- return e1.Error() == e2.Error()`
	`86`	`+ return errors.Is(e1, e2)`
`86`	`87`	`}`
`87`	`88`
`88`	`89`	`// Wrap returns an Error that wrap err with wrapper.`