Skip to content
This repository was archived by the owner on Dec 20, 2025. It is now read-only.

Commit e40f954

Browse files
azmavethclaude
andcommitted
fix: add missing test tags to prevent CI failures
- Add @moduletag :requires_service and :local_only to ModelManagementTest as it calls Models.list_all() which attempts to connect to all providers - This prevents Ollama connection attempts during CI pipeline tests 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>
1 parent 5063dc8 commit e40f954

File tree

3 files changed

+10
-0
lines changed

3 files changed

+10
-0
lines changed

test/ex_llm/infrastructure/ollama_model_registry_test.exs

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -3,6 +3,9 @@ defmodule ExLLM.Infrastructure.OllamaModelRegistryTest do
33

44
alias ExLLM.Infrastructure.OllamaModelRegistry
55

6+
@moduletag :requires_service
7+
@moduletag provider: :ollama
8+
69
setup do
710
# Clear cache before each test
811
OllamaModelRegistry.clear_cache()

test/ex_llm/model_management_test.exs

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -8,6 +8,10 @@ defmodule ExLLM.ModelManagementTest do
88

99
use ExUnit.Case, async: true
1010

11+
# This test module calls Models.list_all() which attempts to connect to all providers
12+
@moduletag :requires_service
13+
@moduletag :local_only
14+
1115
alias ExLLM.Core.Models
1216

1317
describe "model listing and discovery" do

test/ex_llm/oauth2_test.exs

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -11,6 +11,9 @@ defmodule ExLLM.OAuth2Test do
1111
providers and handles various error scenarios appropriately.
1212
"""
1313

14+
@moduletag :oauth2
15+
@moduletag :requires_oauth
16+
1417
describe "OAuth2 provider support" do
1518
test "lists supported providers" do
1619
providers = Helper.supported_providers()

0 commit comments

Comments
 (0)