remove NotEmpty (#54)

nil-go · Nov 13, 2023 · e4eb38c · e4eb38c
1 parent 1147575
commit e4eb38c
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 11 deletions.
diff --git a/internal/assert/assert.go b/internal/assert/assert.go
@@ -27,7 +27,10 @@ func NoError(tb testing.TB, err error) {
 func EqualError(tb testing.TB, err error, message string) {
 	tb.Helper()
 
-	if err.Error() != message {
+	switch {
+	case err == nil:
+		tb.Errorf("expected: %v; actual: <nil>", message)
+	case err.Error() != message:
 		tb.Errorf("expected: %v; actual: %v", message, err.Error())
 	}
 }
@@ -39,11 +42,3 @@ func True(tb testing.TB, value bool) {
 		tb.Errorf("expected True")
 	}
 }
-
-func NotEmpty(tb testing.TB, value any) {
-	tb.Helper()
-
-	if reflect.ValueOf(value).IsZero() {
-		tb.Errorf("expected not empty")
-	}
-}
diff --git a/provider/env/benchmark_test.go b/provider/env/benchmark_test.go
@@ -19,7 +19,7 @@ func BenchmarkNew(b *testing.B) {
 
 	values, err := loader.Load()
 	assert.NoError(b, err)
-	assert.NotEmpty(b, values["USER"])
+	assert.True(b, values["USER"] != "")
 }
 
 func BenchmarkLoad(b *testing.B) {
@@ -36,5 +36,5 @@ func BenchmarkLoad(b *testing.B) {
 	b.StopTimer()
 
 	assert.NoError(b, err)
-	assert.NotEmpty(b, values["USER"])
+	assert.True(b, values["USER"] != "")
 }