[owl] Fix various bug (#909)

- fix anthropic reasoning test
- fix owl-config.yaml vlm_model_id naming
- set inference provider properly for vllm cloud (and related test)
- fix string_normalisation test

Author: noobHappylife

services/api/src/owl/utils/lm.py

@@ -377,6 +377,8 @@ def batch(seq, n):
     def _inference_provider(self, provider: str, owned_by: str) -> str:
         if provider == CloudProvider.ELLM:
             return CloudProvider.ELLM
+        elif provider == CloudProvider.VLLM_CLOUD:
+            return CloudProvider.VLLM_CLOUD
         # this check for provider like azure/bedrock that provides other cloud providers model (openai/anthropic)
         if provider in ModelProvider:
             return ModelProvider(provider)

services/api/tests/routers/test_serving.py

@@ -818,7 +818,7 @@ def test_chat_reasoning_anthropic(setup: ServingContext, stream: bool):
     # Test default params
     response = _test_chat_reasoning_cloud(**kwargs)
     assert len(response.content) > 0
-    kwargs["routing_id"] = "claude-4-6-sonnet-latest"
+    kwargs["routing_id"] = "claude-sonnet-4-6"
     response = _test_chat_reasoning_cloud(**kwargs)
     assert len(response.content) > 0
     # Test disabling reasoning

services/api/tests/test_types.py

@@ -69,18 +69,18 @@ def test_id_string_bad(value: str):
 
 
 def test_string_normalisation():
-    # ---
-    #
-    # Zalgo text
-    assert IdTest(id="H̵̛͕̞̦̰̜͍̰̥̟͆̏͂̌͑ͅä̷͔̟͓̬̯̟͍̭͉͈̮͙̣̯̬͚̞̭̍̀̾͠m̴̡̧̛̝̯̹̗̹̤̲̺̟̥̈̏͊̔̑̍͆̌̀̚͝͝b̴̢̢̫̝̠̗̼̬̻̮̺̭͔̘͑̆̎̚ư̵̧̡̥̙̭̿̈̀̒̐̊͒͑r̷̡̡̲̼̖͎̫̮̜͇̬͌͘g̷̹͍͎̬͕͓͕̐̃̈́̓̆̚͝ẻ̵̡̼̬̥̹͇̭͔̯̉͛̈́̕r̸̮̖̻̮̣̗͚͖̝̂͌̾̓̀̿̔̀͋̈́͌̈́̋͜").id == "Hämbưrgẻr"
-    #
-    #
-    # ---
-    # Arabic
-    assert IdTest(id="مَرْحَبًا بِكُمْ").id == "مرحبا بكم"
-    # ---
-    # Thai
-    assert IdTest(id="สวัสดีครับ คามุย อิอิ").id == "สวสดครบ คามย ออ"
+    with pytest.raises(ValidationError, match="Text contains excessive combining marks"):
+        IdTest(
+            id=(
+                "H̵̛͕̞̦̰̜͍̰̥̟͆̏͂̌͑ͅä̷͔̟͓̬̯̟͍̭͉͈̮͙̣̯̬͚̞̭̍̀̾͠"
+                "m̴̡̧̛̝̯̹̗̹̤̲̺̟̥̈̏͊̔̑̍͆̌̀̚͝͝b̴̢̢̫̝̠̗̼̬̻̮̺̭͔̘͑̆̎̚"
+                "ư̵̧̡̥̙̭̿̈̀̒̐̊͒͑r̷̡̡̲̼̖͎̫̮̜͇̬͌͘g̷̹͍͎̬͕͓͕̐̃̈́̓̆̚͝"
+                "ẻ̵̡̼̬̥̹͇̭͔̯̉͛̈́̕r̸̮̖̻̮̣̗͚͖̝̂͌̾̓̀̿̔̀͋̈́͌̈́̋͜"
+            )
+        )
+
+    assert IdTest(id="مَرْحَبًا بِكُمْ").id == "مَرْحَبًا بِكُمْ"
+    assert IdTest(id="สวัสดีครับ คามุย อิอิ").id == "สวัสดีครับ คามุย อิอิ"
 
 
 def test_datetime_utc():

services/api/tests/utils/test_lm.py

@@ -0,0 +1,31 @@
+from types import SimpleNamespace
+
+from owl.types import CloudProvider, ModelProvider
+from owl.utils.lm import DeploymentRouter
+
+
+def _make_router(*, owned_by: str = "openai") -> DeploymentRouter:
+    router = DeploymentRouter.__new__(DeploymentRouter)
+    router.config = SimpleNamespace(owned_by=owned_by)
+    return router
+
+
+def test_inference_provider_should_prefer_vllm_cloud_over_owned_by() -> None:
+    router = _make_router()
+
+    assert (
+        router._inference_provider(CloudProvider.VLLM_CLOUD, "openai")
+        == CloudProvider.VLLM_CLOUD
+    )
+
+
+def test_inference_provider_should_prefer_ellm_over_owned_by() -> None:
+    router = _make_router()
+
+    assert router._inference_provider(CloudProvider.ELLM, "openai") == CloudProvider.ELLM
+
+
+def test_inference_provider_should_use_owned_by_for_azure_openai() -> None:
+    router = _make_router()
+
+    assert router._inference_provider(CloudProvider.AZURE, "openai") == ModelProvider.OPENAI