refactor(ramcache): Rename internal functions and variables

ramcache/ramcache.go

@@ -148,10 +148,10 @@ func (r *ramcache) cleanupExpiredItems() {
 
 // removeExpiredItems removes expired items from the store.
 func (r *ramcache) removeExpiredItems() {
-	items := r.store.GetItemsSortedByExpiry()
-	for _, item := range items {
-		if item.Item.IsExpired() {
-			r.store.Delete(item.Key)
+	keyItems := r.store.KeyItemsSortedByExpiry()
+	for _, ki := range keyItems {
+		if ki.Item.IsExpired() {
+			r.store.Delete(ki.Key)
 		} else {
 			// Items are sorted by expiry time, so we can break early
 			break

ramcache/store.go

@@ -61,8 +61,8 @@ type keyItem struct {
 	Item Item   // Item is the item.
 }
 
-// GetItemsSortedByExpiry returns all items sorted by expiry time (closest to expiry first).
-func (s *store) GetItemsSortedByExpiry() []keyItem {
+// KeyItemsSortedByExpiry returns all key items sorted by expiry time (closest to expiry first).
+func (s *store) KeyItemsSortedByExpiry() []keyItem {
 	s.mu.RLock()
 	defer s.mu.RUnlock()
 	items := make([]keyItem, 0, len(s.items))

ramcache/store_test.go

@@ -46,13 +46,13 @@ func TestStore(t *testing.T) {
 		}
 	})
 
-	t.Run("GetItemsSortedByExpiry", func(t *testing.T) {
+	t.Run("KeyItemsSortedByExpiry", func(t *testing.T) {
 		s := newStore()
 		s.Set("key1", Item{Value: []byte("value1"), Expiry: time.Now().Add(20 * time.Minute)})
 		s.Set("key2", Item{Value: []byte("value2"), Expiry: time.Now().Add(10 * time.Minute)})
-		items := s.GetItemsSortedByExpiry()
+		items := s.KeyItemsSortedByExpiry()
 		if len(items) != 2 || items[0].Key != "key2" || items[1].Key != "key1" {
-			t.Errorf("GetItemsSortedByExpiry failed. Expected [key2, key1], got [%v, %v]", items[0].Key, items[1].Key)
+			t.Errorf("KeyItemsSortedByExpiry failed. Expected [key2, key1], got [%v, %v]", items[0].Key, items[1].Key)
 		}
 	})
 }