Skip to content

Commit ed89a7c

Browse files
committed
fix: LLM routes not registered, and formatting in sdk tests
1 parent 7a865e8 commit ed89a7c

File tree

8 files changed

+139
-117
lines changed

8 files changed

+139
-117
lines changed

README.md

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -33,11 +33,13 @@ The patching and unpatching is in `task patch` and `task unpatch` targets.
3333

3434
## Contributing
3535

36-
Contributions to httpbun are welcome, for the most part. However, I strongly urge you to open an issue to discuss
37-
whatever you're working to contribute *before* you start working on it. This will ensure we are on the same page and
36+
Bug fixes, yes. New features, less so please. Either way, consider opening an issue to discuss
37+
it *before* you start working. This will ensure we are on the same page and
3838
your work would be in the right place to be merged in. It'll also ensure we don't end up working on the same thing,
3939
duplicating efforts. Thanks!
4040

41+
New features are added mostly as _I_ need them, and in the form that I want to use them.
42+
4143
## License
4244

4345
[Apache-2.0 License](https://github.com/sharat87/httpbun/blob/main/LICENSE). Project includes a

Taskfile.dist.yml

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -53,9 +53,13 @@ tasks:
5353
- mv x "$(go env GOROOT)/src/net/http/server.go"
5454

5555
llm-sdk-tests:
56+
aliases: [st]
5657
dir: llm-sdk-tests
5758
env:
5859
BASE_URL: http://localhost:3090/llm
60+
HTTP_PROXY: http://localhost:9020
5961
cmds:
6062
- curl -fsS "$(dirname "$BASE_URL")" >/dev/null
63+
- uv run ruff check --fix .
64+
- uv run ruff format .
6165
- uv run pytest tests

llm-sdk-tests/pyproject.toml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,13 +1,13 @@
11
[project]
22
name = "llm-sdk-tests"
33
version = "0.1.0"
4-
description = "Add your description here"
5-
readme = "README.md"
4+
description = "Tests that use LLM SDKs to hit the mock `/llm` APIs to verify compliance"
65
requires-python = ">=3.13"
76
dependencies = [
87
"openai>=2.14.0",
98
"pytest>=9.0.2",
109
"pytest-asyncio>=1.3.0",
10+
"ruff>=0.14.10",
1111
]
1212

1313
[tool.pytest.ini_options]

llm-sdk-tests/tests/conftest.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4,4 +4,4 @@
44

55
@pytest.fixture
66
def base_url() -> str:
7-
return os.getenv("BASE_URL")
7+
return os.getenv("BASE_URL")

0 commit comments

Comments
 (0)