[PR #12863] [MERGED] embeddings: removed redundant test #45231

Closed
opened 2026-04-25 00:55:50 -05:00 by GiteaMirror · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/ollama/ollama/pull/12863
Author: @npardal
Created: 10/30/2025
Status: Merged
Merged: 10/31/2025
Merged by: @npardal

Base: mainHead: nicole/test-removal


📝 Commits (2)

📊 Changes

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

View changed files

📝 integration/api_test.go (+0 -27)

📄 Description

This PR removes a redundant test from TestAPIEmbeddings
Contents of this test already exists in

  • embed_test.go (vector equality/token counts)
  • model_arch_test.go

🔄 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/12863 **Author:** [@npardal](https://github.com/npardal) **Created:** 10/30/2025 **Status:** ✅ Merged **Merged:** 10/31/2025 **Merged by:** [@npardal](https://github.com/npardal) **Base:** `main` ← **Head:** `nicole/test-removal` --- ### 📝 Commits (2) - [`c829fa4`](https://github.com/ollama/ollama/commit/c829fa46c51f7540b8c48ba9c38a9cfb3fd39f0c) removed redundant test - [`e49ad81`](https://github.com/ollama/ollama/commit/e49ad8116a5252232e9cf2f819a15553833556a0) testing api tool calling ### 📊 Changes **1 file changed** (+0 additions, -27 deletions) <details> <summary>View changed files</summary> 📝 `integration/api_test.go` (+0 -27) </details> ### 📄 Description This PR removes a redundant test from **TestAPIEmbeddings** Contents of this test already exists in - **embed_test.go** (vector equality/token counts) - **model_arch_test.go** --- <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 00:55:50 -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#45231