diff options
author | kuba-- <kuba@sourced.tech> | 2018-09-17 22:20:50 +0200 |
---|---|---|
committer | kuba-- <kuba@sourced.tech> | 2018-09-17 22:20:50 +0200 |
commit | 4896974b4daf86f53d782c868d408f830f84c294 (patch) | |
tree | c0668f6aeb4f95c5d75bef0e2aa469b563d10810 /plumbing/cache/object_test.go | |
parent | 2fb32d2a8601213b6db109d3e9028c6b64af1874 (diff) | |
download | go-git-4896974b4daf86f53d782c868d408f830f84c294.tar.gz |
Fix potential LRU cache size issue.
Signed-off-by: kuba-- <kuba@sourced.tech>
Diffstat (limited to 'plumbing/cache/object_test.go')
-rw-r--r-- | plumbing/cache/object_test.go | 19 |
1 files changed, 19 insertions, 0 deletions
diff --git a/plumbing/cache/object_test.go b/plumbing/cache/object_test.go index ac3f0a3..b3e5f79 100644 --- a/plumbing/cache/object_test.go +++ b/plumbing/cache/object_test.go @@ -45,6 +45,25 @@ func (s *ObjectSuite) TestPutSameObject(c *C) { } } +func (s *ObjectSuite) TestPutSameObjectWithDifferentSize(c *C) { + const hash = "aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa" + + cache := NewObjectLRU(7 * Byte) + cache.Put(newObject(hash, 1*Byte)) + cache.Put(newObject(hash, 3*Byte)) + cache.Put(newObject(hash, 5*Byte)) + cache.Put(newObject(hash, 7*Byte)) + + c.Assert(cache.MaxSize, Equals, 7*Byte) + c.Assert(cache.actualSize, Equals, 7*Byte) + c.Assert(cache.ll.Len(), Equals, 1) + + obj, ok := cache.Get(plumbing.NewHash(hash)) + c.Assert(obj.Hash(), Equals, plumbing.NewHash(hash)) + c.Assert(FileSize(obj.Size()), Equals, 7*Byte) + c.Assert(ok, Equals, true) +} + func (s *ObjectSuite) TestPutBigObject(c *C) { for _, o := range s.c { o.Put(s.bObject) |