Fix urljoin in query.py and add path-prefix base URL tests

- Apply the same urljoin → string concatenation fix to query.py
  (CloudRecoService had the identical bug)
- Add TestCustomBaseVWSURL::test_custom_base_url_with_path_prefix
  and TestCustomBaseVWQURL::test_custom_base_url_with_path_prefix
  to assert that a base URL containing a path prefix is preserved
  in the outgoing request URL; these tests would fail with the old
  urljoin approach

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

Author: adamtheturtle

src/vws/query.py

@@ -5,7 +5,6 @@
 import json
 from http import HTTPMethod, HTTPStatus
 from typing import Any, BinaryIO
-from urllib.parse import urljoin
 
 import requests
 from beartype import BeartypeConf, beartype
@@ -146,7 +145,7 @@ def query(
 
         requests_response = requests.request(
             method=method,
-            url=urljoin(base=self._base_vwq_url, url=request_path),
+            url=self._base_vwq_url.rstrip("/") + request_path,
             headers=headers,
             data=content,
             timeout=self._request_timeout_seconds,

tests/test_query.py

@@ -2,11 +2,13 @@
 
 import datetime
 import io
+import secrets
 import uuid
 from typing import BinaryIO
 
 import pytest
 import requests
+import responses as responses_mock
 from freezegun import freeze_time
 from mock_vws import MockVWS
 from mock_vws.database import CloudDatabase
@@ -190,6 +192,40 @@ def test_custom_base_url(image: io.BytesIO | BinaryIO) -> None:
             match = matches[0]
             assert match.target_id == target_id
 
+    @staticmethod
+    @responses_mock.activate
+    def test_custom_base_url_with_path_prefix(
+        image: io.BytesIO | BinaryIO,
+    ) -> None:
+        """
+        A base VWQ URL with a path prefix is used as-is, without the
+        prefix
+        being dropped.
+        """
+        base_vwq_url = "http://example.com/prefix"
+        responses_mock.add(
+            method=responses_mock.POST,
+            url="http://example.com/prefix/v1/query",
+            json={
+                "result_code": "Success",
+                "results": [],
+                "query_id": "abc",
+            },
+            status=200,
+        )
+        cloud_reco_client = CloudRecoService(
+            client_access_key=secrets.token_hex(),
+            client_secret_key=secrets.token_hex(),
+            base_vwq_url=base_vwq_url,
+        )
+
+        cloud_reco_client.query(image=image)
+
+        assert len(responses_mock.calls) == 1
+        assert responses_mock.calls[0].request.url == (
+            "http://example.com/prefix/v1/query"
+        )
+
 
 class TestMaxNumResults:
     """Tests for the ``max_num_results`` parameter of ``query``."""

tests/test_vws.py

@@ -9,6 +9,7 @@
 
 import pytest
 import requests
+import responses as responses_mock
 from freezegun import freeze_time
 from mock_vws import MockVWS
 from mock_vws.database import CloudDatabase
@@ -246,6 +247,38 @@ def test_custom_base_url(image: io.BytesIO | BinaryIO) -> None:
                 application_metadata=None,
             )
 
+    @staticmethod
+    @responses_mock.activate
+    def test_custom_base_url_with_path_prefix() -> None:
+        """
+        A base VWS URL with a path prefix is used as-is, without the
+        prefix
+        being dropped.
+        """
+        base_vws_url = "http://example.com/prefix"
+        responses_mock.add(
+            method=responses_mock.GET,
+            url="http://example.com/prefix/targets",
+            json={
+                "result_code": "Success",
+                "results": [],
+                "transaction_id": "abc",
+            },
+            status=200,
+        )
+        vws_client = VWS(
+            server_access_key=secrets.token_hex(),
+            server_secret_key=secrets.token_hex(),
+            base_vws_url=base_vws_url,
+        )
+
+        vws_client.list_targets()
+
+        assert len(responses_mock.calls) == 1
+        assert responses_mock.calls[0].request.url == (
+            "http://example.com/prefix/targets"
+        )
+
 
 class TestListTargets:
     """Tests for listing targets."""