[PR #14090] [CLOSED] test: add comprehensive tests for EncoderCache #45755

Closed
opened 2026-04-25 01:24:23 -05:00 by GiteaMirror · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/ollama/ollama/pull/14090
Author: @aturzone
Created: 2/5/2026
Status: Closed

Base: mainHead: test/add-encoder-cache-tests


📝 Commits (1)

  • ba6ddb1 test: add comprehensive tests for EncoderCache

📊 Changes

1 file changed (+435 additions, -0 deletions)

View changed files

kvcache/encoder_test.go (+435 -0)

📄 Description

Add test coverage for kvcache/encoder.go which previously had no tests. Tests cover:

  • Cache initialization and configuration
  • Layer management
  • Put/Get operations
  • Reserve mode behavior
  • Remove operations
  • Multi-layer caching
  • Panic conditions (multiple sequences, config changes, CopyPrefix)

🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/ollama/ollama/pull/14090 **Author:** [@aturzone](https://github.com/aturzone) **Created:** 2/5/2026 **Status:** ❌ Closed **Base:** `main` ← **Head:** `test/add-encoder-cache-tests` --- ### 📝 Commits (1) - [`ba6ddb1`](https://github.com/ollama/ollama/commit/ba6ddb1776fa5a30264c0385242ccc37ab27e661) test: add comprehensive tests for EncoderCache ### 📊 Changes **1 file changed** (+435 additions, -0 deletions) <details> <summary>View changed files</summary> ➕ `kvcache/encoder_test.go` (+435 -0) </details> ### 📄 Description Add test coverage for kvcache/encoder.go which previously had no tests. Tests cover: - Cache initialization and configuration - Layer management - Put/Get operations - Reserve mode behavior - Remove operations - Multi-layer caching - Panic conditions (multiple sequences, config changes, CopyPrefix) --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
GiteaMirror added the pull-request label 2026-04-25 01:24:23 -05:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: github-starred/ollama#45755