diff --git a/utils/reflect_test.go b/utils/reflect_test.go index 21538589a..dd33fc2fa 100644 --- a/utils/reflect_test.go +++ b/utils/reflect_test.go @@ -142,8 +142,10 @@ func TestReflectAsMapStringIface(t *testing.T) { } func TestGreaterThan(t *testing.T) { - if _, err := GreaterThan(1, 1.2, false); err == nil || err.Error() != "incomparable" { + if gte, err := GreaterThan(1, 1.2, false); err != nil { t.Error(err) + } else if gte { + t.Error("should be not greater than") } if _, err := GreaterThan(struct{}{}, map[string]interface{}{"a": "a"}, false); err == nil || err.Error() != "incomparable" { diff --git a/utils/rsrfield_test.go b/utils/rsrfield_test.go index 658e05550..1ddefde16 100644 --- a/utils/rsrfield_test.go +++ b/utils/rsrfield_test.go @@ -490,8 +490,8 @@ func TestRSRFilterPass(t *testing.T) { if fltr.Pass("0s") { t.Error("passing!") } - if fltr.Pass("13") { - t.Error("passing!") + if !fltr.Pass("13") { + t.Error("not passing!") } if !fltr.Pass("12s") { t.Error("not passing!") @@ -523,8 +523,8 @@ func TestRSRFilterPass(t *testing.T) { if !fltr.Pass("0s") { t.Error("not passing!") } - if fltr.Pass("13") { - t.Error("passing!") + if !fltr.Pass("13") { + t.Error("not passing!") } if !fltr.Pass("12s") { t.Error("not passing!") @@ -574,8 +574,8 @@ func TestRSRFilterPass(t *testing.T) { if !fltr.Pass("1ns") { t.Error("not passing!") } - if fltr.Pass("13") { - t.Error("passing!") + if !fltr.Pass("13") { + t.Error("not passing!") } if !fltr.Pass("12s") { t.Error("not passing!") @@ -613,8 +613,8 @@ func TestRSRFilterPass(t *testing.T) { if fltr.Pass("0s") { t.Error("passing!") } - if fltr.Pass("13") { - t.Error("passing!") + if !fltr.Pass("13") { + t.Error("not passing!") } if !fltr.Pass("12s") { t.Error("not passing!")