Merge pull request #4 from Senpos:fix-duplicated-tests

add MaxKeySize option test, remove duplicated MaxCacheSize option test
go-pkgz · Oct 13, 2019 · 51598f3 · 51598f3
2 parents 1bf0f67 + 4ed8e74
commit 51598f3
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/expirable_cache_test.go b/expirable_cache_test.go
@@ -88,8 +88,8 @@ func TestExpirableCache_BadOptions(t *testing.T) {
 	_, err := NewExpirableCache(MaxCacheSize(-1))
 	assert.EqualError(t, err, "failed to set cache option: negative max cache size")
 
-	_, err = NewExpirableCache(MaxCacheSize(-1))
-	assert.EqualError(t, err, "failed to set cache option: negative max cache size")
+	_, err = NewExpirableCache(MaxKeySize(-1))
+	assert.EqualError(t, err, "failed to set cache option: negative max key size")
 
 	_, err = NewExpirableCache(MaxKeys(-1))
 	assert.EqualError(t, err, "failed to set cache option: negative max keys")

diff --git a/lru_cache_test.go b/lru_cache_test.go
@@ -62,8 +62,8 @@ func TestLruCache_BadOptions(t *testing.T) {
 	_, err := NewLruCache(MaxCacheSize(-1))
 	assert.EqualError(t, err, "failed to set cache option: negative max cache size")
 
-	_, err = NewLruCache(MaxCacheSize(-1))
-	assert.EqualError(t, err, "failed to set cache option: negative max cache size")
+	_, err = NewLruCache(MaxKeySize(-1))
+	assert.EqualError(t, err, "failed to set cache option: negative max key size")
 
 	_, err = NewLruCache(MaxKeys(-1))
 	assert.EqualError(t, err, "failed to set cache option: negative max keys")