refactor(models): Add _resolve_url method to Executor
Extracted URL resolution logic into separate method for better testability
This commit is contained in:
1 parent
130d06351d
commit
805b28e344
1 file changed
+5
-1
+5
-1
@@ -14,10 +14,14 @@ class Executor:
|
|||||||
def __init__(self, context: Context):
|
def __init__(self, context: Context):
|
||||||
self.context = context
|
self.context = context
|
||||||
|
|
||||||
def execute(self, request: RequestModel) -> Response:
|
def _resolve_url(self, request: RequestModel) -> str:
|
||||||
url = self.context.resolve(request.path)
|
url = self.context.resolve(request.path)
|
||||||
if not url.startswith("http"):
|
if not url.startswith("http"):
|
||||||
url = f"{self.context.config.base_url}{url}"
|
url = f"{self.context.config.base_url}{url}"
|
||||||
|
return url
|
||||||
|
|
||||||
|
def execute(self, request: RequestModel) -> Response:
|
||||||
|
url = self._resolve_url(request)
|
||||||
|
|
||||||
with httpx.Client() as client:
|
with httpx.Client() as client:
|
||||||
resp = client.request(
|
resp = client.request(
|
||||||
|
|||||||
Reference in new issue
Block a user