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

Commit cf6ed06

Browse files
committed
fix: set async: false for tests that manipulate environment variables
Tests that modify global state like environment variables must run synchronously to prevent race conditions. Fixed: - groq_test.exs: Was missing async option (defaulted to true) - ex_llm_pipeline_api_test.exs: Was explicitly set to async: true This prevents tests from interfering with each other's API keys.
1 parent 6641890 commit cf6ed06

File tree

2 files changed

+2
-2
lines changed

2 files changed

+2
-2
lines changed

test/ex_llm/providers/groq_test.exs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
defmodule ExLLM.Providers.GroqTest do
2-
use ExUnit.Case
2+
use ExUnit.Case, async: false
33
alias ExLLM.Providers.Groq
44
alias ExLLM.Testing.ConfigProviderHelper
55

test/ex_llm_pipeline_api_test.exs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
defmodule ExLLM.PipelineAPITest do
2-
use ExUnit.Case, async: true
2+
use ExUnit.Case, async: false
33

44
alias ExLLM.Pipeline.Request
55
alias ExLLM.Plugs

0 commit comments

Comments
 (0)