From 890a5c3458b43e6104ff5da8dfa139d013d77544 Mon Sep 17 00:00:00 2001 From: Dragos Harabor Date: Tue, 4 Jul 2017 19:17:15 -0700 Subject: [PATCH] Issue #469 fix reflect.Value.Len() can cover more types, per godoc: > Len returns v's length. It panics if v's Kind is not Array, Chan, Map, Slice, or String. --- assert/assertions.go | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/assert/assertions.go b/assert/assertions.go index 6d92506..8259050 100644 --- a/assert/assertions.go +++ b/assert/assertions.go @@ -449,9 +449,7 @@ func isEmpty(object interface{}) bool { objValue := reflect.ValueOf(object) switch objValue.Kind() { - case reflect.Map: - fallthrough - case reflect.Slice, reflect.Chan: + case reflect.Array, reflect.Chan, reflect.Map, reflect.Slice, reflect.String: { return (objValue.Len() == 0) }