From 0e4c09ccd17ad35b846196b669ac13fae2d15905 Mon Sep 17 00:00:00 2001 From: XiaolongZhang Date: Fri, 7 Aug 2026 18:23:02 +0800 Subject: [PATCH] fix: route invalid_input order requests through the discovered endpoint The four order GET/PUT calls in invalid_input_test.py used a raw "/orders/{order_id}" path against the httpx base_url, while every call in order_test.py goes through self.get_order_url(order_id), which resolves the shopping service endpoint discovered from /.well-known/ucp. When the shopping service is not hosted at the server root, the raw path 404s before validation runs, so these tests would fail (or pass) for the wrong reason instead of exercising the intended 422 validation path. Use get_order_url to match the rest of the suite; behavior is unchanged for the sample server, whose shopping endpoint is at the root. --- invalid_input_test.py | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/invalid_input_test.py b/invalid_input_test.py index d0d9e81..404e14d 100644 --- a/invalid_input_test.py +++ b/invalid_input_test.py @@ -49,7 +49,7 @@ def test_invalid_adjustment_status(self): # Get Order response = self.client.get( - f"/orders/{order_id}", headers=self.get_headers() + self.get_order_url(order_id), headers=self.get_headers() ) order_obj = order.Order(**response.json()) order_dict = order_obj.model_dump( @@ -71,7 +71,7 @@ def test_invalid_adjustment_status(self): # Update Order resp = self.client.put( - f"/orders/{order_id}", + self.get_order_url(order_id), json=order_dict, headers=self.get_headers(), ) @@ -116,7 +116,7 @@ def test_malformed_adjustment_payload(self): # Get Order response = self.client.get( - f"/orders/{order_id}", headers=self.get_headers() + self.get_order_url(order_id), headers=self.get_headers() ) order_obj = order.Order(**response.json()) order_dict = order_obj.model_dump( @@ -128,7 +128,7 @@ def test_malformed_adjustment_payload(self): # Update Order resp = self.client.put( - f"/orders/{order_id}", + self.get_order_url(order_id), json=order_dict, headers=self.get_headers(), )