Skip to content

Commit ff0ba5b

Browse files
committed
test(alibaba): cover sk-ws- plan key in token-plan routing test
The docstring claims sk-sp-/sk-tok-/sk-ws- all route to the Token Plan endpoint, but the parametrization only exercised sk-sp- and sk-tok-. Add sk-ws-test-key so the consolidated assertion matches the docstring (addresses CodeRabbit nitpick on #105).
1 parent 2d7183b commit ff0ba5b

1 file changed

Lines changed: 1 addition & 1 deletion

File tree

tests/auth/test_alibaba_auth.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -738,7 +738,7 @@ async def test_logout_alibaba_rejects_non_default_config_location():
738738

739739

740740
@pytest.mark.asyncio
741-
@pytest.mark.parametrize("key", ["sk-sp-test-key", "sk-tok-test-key"])
741+
@pytest.mark.parametrize("key", ["sk-sp-test-key", "sk-tok-test-key", "sk-ws-test-key"])
742742
async def test_login_alibaba_plan_key_routes_to_token_plan(monkeypatch, tmp_path, key):
743743
"""Subscription plan keys (sk-sp-/sk-tok-/sk-ws-) all use the Token Plan endpoint."""
744744
from pythinker_code.auth.alibaba import ALIBABA_TOKEN_PLAN_BASE_URL, login_alibaba_api_key

0 commit comments

Comments
 (0)