Skip to content

chore(internal): update client tests #197

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Feb 13, 2025
Merged

chore(internal): update client tests #197

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
110 changes: 62 additions & 48 deletions tests/test_client.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -23,10 +23,12 @@

from groq import Groq, AsyncGroq, APIResponseValidationError
from groq._types import Omit
from groq._utils import maybe_transform
from groq._models import BaseModel, FinalRequestOptions
from groq._constants import RAW_RESPONSE_HEADER
from groq._exceptions import GroqError, APIStatusError, APITimeoutError, APIResponseValidationError
from groq._base_client import DEFAULT_TIMEOUT, HTTPX_DEFAULT_TIMEOUT, BaseClient, make_request_options
from groq.types.chat.completion_create_params import CompletionCreateParams

from .utils import update_env

Expand Down Expand Up @@ -706,18 +708,21 @@ def test_retrying_timeout_errors_doesnt_leak(self, respx_mock: MockRouter) -> No
"/openai/v1/chat/completions",
body=cast(
object,
dict(
messages=[
{
"role": "system",
"content": "You are a helpful assistant.",
},
{
"role": "user",
"content": "Explain the importance of low latency LLMs",
},
],
model="llama3-8b-8192",
maybe_transform(
dict(
messages=[
{
"role": "system",
"content": "You are a helpful assistant.",
},
{
"role": "user",
"content": "Explain the importance of low latency LLMs",
},
],
model="llama3-8b-8192",
),
CompletionCreateParams,
),
),
cast_to=httpx.Response,
Expand All @@ -736,18 +741,21 @@ def test_retrying_status_errors_doesnt_leak(self, respx_mock: MockRouter) -> Non
"/openai/v1/chat/completions",
body=cast(
object,
dict(
messages=[
{
"role": "system",
"content": "You are a helpful assistant.",
},
{
"role": "user",
"content": "Explain the importance of low latency LLMs",
},
],
model="llama3-8b-8192",
maybe_transform(
dict(
messages=[
{
"role": "system",
"content": "You are a helpful assistant.",
},
{
"role": "user",
"content": "Explain the importance of low latency LLMs",
},
],
model="llama3-8b-8192",
),
CompletionCreateParams,
),
),
cast_to=httpx.Response,
Expand Down Expand Up @@ -1526,18 +1534,21 @@ async def test_retrying_timeout_errors_doesnt_leak(self, respx_mock: MockRouter)
"/openai/v1/chat/completions",
body=cast(
object,
dict(
messages=[
{
"role": "system",
"content": "You are a helpful assistant.",
},
{
"role": "user",
"content": "Explain the importance of low latency LLMs",
},
],
model="llama3-8b-8192",
maybe_transform(
dict(
messages=[
{
"role": "system",
"content": "You are a helpful assistant.",
},
{
"role": "user",
"content": "Explain the importance of low latency LLMs",
},
],
model="llama3-8b-8192",
),
CompletionCreateParams,
),
),
cast_to=httpx.Response,
Expand All @@ -1556,18 +1567,21 @@ async def test_retrying_status_errors_doesnt_leak(self, respx_mock: MockRouter)
"/openai/v1/chat/completions",
body=cast(
object,
dict(
messages=[
{
"role": "system",
"content": "You are a helpful assistant.",
},
{
"role": "user",
"content": "Explain the importance of low latency LLMs",
},
],
model="llama3-8b-8192",
maybe_transform(
dict(
messages=[
{
"role": "system",
"content": "You are a helpful assistant.",
},
{
"role": "user",
"content": "Explain the importance of low latency LLMs",
},
],
model="llama3-8b-8192",
),
CompletionCreateParams,
),
),
cast_to=httpx.Response,
Expand Down