smart-tests-cli 2.15.1__tar.gz → 2.15.2__tar.gz

{smart_tests_cli-2.15.1 → smart_tests_cli-2.15.2}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: smart-tests-cli
-Version: 2.15.1
+Version: 2.15.2
 Summary: Smart Tests CLI
 Author-email: CloudBees <support@cloudbees.com>
 License: Apache Software License v2
@@ -20,6 +20,7 @@ Requires-Dist: python-dateutil>=2.9
 Requires-Dist: tabulate>=0.9
 Requires-Dist: click<8.2,>=8.1
 Requires-Dist: InquirerPy>=0.3.4
+Requires-Dist: requests-toolbelt>=1.0.0
 Dynamic: license-file
 
 This is a CLI to interact with CloudBees Smart Tests.

{smart_tests_cli-2.15.1 → smart_tests_cli-2.15.2}/pyproject.toml

@@ -19,6 +19,7 @@ dependencies = [
     "tabulate>=0.9",
     "click>=8.1,<8.2",
     "InquirerPy>=0.3.4",
+    "requests-toolbelt>=1.0.0",
 ]
 dynamic = ["version"]
 

{smart_tests_cli-2.15.1 → smart_tests_cli-2.15.2}/smart_tests/commands/record/session.py

@@ -97,8 +97,8 @@ def session(
         res = client.request("post", sub_path, payload=payload)
 
         if res.status_code == HTTPStatus.NOT_FOUND:
-            msg = f"Build {build_name} was not found." \
-                f"Make sure to run `launchable record build --build {build_name}` before you run this command."
+            msg = f"Build {build_name} was not found. " \
+                f"Make sure to run `smart-tests record build --build {build_name}` before you run this command."
             tracking_client.send_error_event(
                 event_name=Tracking.ErrorEvent.INTERNAL_CLI_ERROR,
                 stack_trace=msg,

{smart_tests_cli-2.15.1 → smart_tests_cli-2.15.2}/smart_tests/commands/record/tests.py

@@ -12,6 +12,7 @@ import click
 from dateutil.parser import ParserError, parse
 from junitparser import JUnitXml, TestCase, TestSuite  # type: ignore  # noqa: F401
 from more_itertools import ichunked
+from requests_toolbelt import MultipartEncoder  # type: ignore
 from tabulate import tabulate
 
 import smart_tests.args4p.converters as converters
@@ -33,6 +34,13 @@ from ...utils.logger import Logger
 from ...utils.smart_tests_client import SmartTestsClient
 from .case_event import CaseEvent, CaseEventGenerator, CaseEventType, DataBuilder, TestPathBuilder
 
+# Together with the workers sending events, this stays within urllib3's default pool size (10 connections per host)
+MAX_RAW_FILE_UPLOAD_WORKERS = 5
+# (connect timeout, read timeout)
+# urllib3 applies the connect timeout to each write of the request body, too. The read timeout is long because
+# the server may store the file before responding.
+RAW_FILE_UPLOAD_TIMEOUT = (30, 300)
+
 GROUP_NAME_RULE = re.compile("^[a-zA-Z0-9][a-zA-Z0-9_-]*$")
 RESERVED_GROUP_NAMES = ["group", "groups", "nogroup", "nogroups"]
 
@@ -272,17 +280,19 @@ class RecordTests:
             url = f"{self.client.base_url()}{path}"
 
             with open(file_path, 'rb') as f:
-                files = {'file': (os.path.basename(file_path), f)}
+                # Stream the multipart body instead of using `files=`. With `files=`, requests builds the whole body
+                # in memory and writes it in one go, and urllib3 applies the connect timeout to that write, so large
+                # files fail with "The write operation timed out".
+                body = MultipartEncoder(fields={'file': (os.path.basename(file_path), f)})
                 headers = self.client.http_client._headers(compress=False)
-                # Remove Content-Type header to let requests set it for multipart
-                headers.pop('Content-Type', None)
+                headers['Content-Type'] = body.content_type
 
                 response = self.client.http_client.session.request(
                     'POST',
                     url,
-                    files=files,
+                    data=body,
                     headers=headers,
-                    timeout=(5, 60),
+                    timeout=RAW_FILE_UPLOAD_TIMEOUT,
                     verify=(not self.client.http_client.skip_cert_verification)
                 )
 
@@ -300,17 +310,21 @@ class RecordTests:
             self.logger.warning(f"Error uploading raw test result file {file_path}: {str(e)}")
             return False
 
-    def upload_raw_files(self) -> None:
+    def upload_raw_files(self, executor: ThreadPoolExecutor) -> None:
 
         if self.reports and not self.dry_run:
             self.logger.debug(f"Uploading {len(self.reports)} raw test result file(s)")
             for report_file in self.reports:
-                self.upload_raw_file(report_file)
+                executor.submit(self.upload_raw_file, report_file)
 
     def run(self):
-        # Upload raw test result files before parsing
-        self.upload_raw_files()
+        # Upload raw test result files in the background while parsing and recording test results,
+        # so that large files don't delay recording. Leaving the `with` block waits for the uploads to finish.
+        with ThreadPoolExecutor(max_workers=MAX_RAW_FILE_UPLOAD_WORKERS) as executor:
+            self.upload_raw_files(executor)
+            self._record_tests()
 
+    def _record_tests(self):
         count = 0  # count number of test cases sent
         is_observation = False
 
@@ -487,7 +501,7 @@ class RecordTests:
         url = self._get_test_session_url()
         click.echo(
             f"\nVisit {url} to view uploaded test results "
-            f"(or run `launchable inspect tests --test-session-id {self.test_session_id}`)")
+            f"(or run `smart-tests inspect tests --test-session-id {self.test_session_id}`)")
 
     def _get_test_session_url(self) -> str:
         cbp_workspace = self.client.get_cbp_workspace()

{smart_tests_cli-2.15.1 → smart_tests_cli-2.15.2}/smart_tests/test_runners/karma.py

@@ -26,10 +26,10 @@ def subset(client, _with: Annotated[str | None, typer.Option(
     """
     Usage:
         find src -name "*.spec.ts" -o -name "*.spec.js" > test-list.txt
-        cat test-list.txt | launchable subset --target 10% karma
+        cat test-list.txt | smart-tests subset karma --session @session.txt --target 10%
 
         # Output in ng test format
-        find src -name "*.spec.ts" | launchable subset --target 10% karma --with ng
+        find src -name "*.spec.ts" | smart-tests subset karma --session @session.txt --target 10% --with ng
     """
     for t in client.stdin():
         path = t.strip()

{smart_tests_cli-2.15.1 → smart_tests_cli-2.15.2}/smart_tests_cli.egg-info/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: smart-tests-cli
-Version: 2.15.1
+Version: 2.15.2
 Summary: Smart Tests CLI
 Author-email: CloudBees <support@cloudbees.com>
 License: Apache Software License v2
@@ -20,6 +20,7 @@ Requires-Dist: python-dateutil>=2.9
 Requires-Dist: tabulate>=0.9
 Requires-Dist: click<8.2,>=8.1
 Requires-Dist: InquirerPy>=0.3.4
+Requires-Dist: requests-toolbelt>=1.0.0
 Dynamic: license-file
 
 This is a CLI to interact with CloudBees Smart Tests.

{smart_tests_cli-2.15.1 → smart_tests_cli-2.15.2}/smart_tests_cli.egg-info/requires.txt

@@ -6,3 +6,4 @@ python-dateutil>=2.9
 tabulate>=0.9
 click<8.2,>=8.1
 InquirerPy>=0.3.4
+requests-toolbelt>=1.0.0

smart_tests_cli-2.15.2/smart_tests_cli.egg-info/scm_version.json

@@ -0,0 +1,8 @@
+{
+  "tag": "2.15.2",
+  "distance": 0,
+  "node": "g17521872e945f52bba869b47c72b0b7157bf5789",
+  "dirty": false,
+  "branch": "main",
+  "node_date": "2026-10-05"
+}

{smart_tests_cli-2.15.1 → smart_tests_cli-2.15.2}/tests/commands/record/test_tests.py

@@ -9,6 +9,7 @@ from unittest import mock
 import responses  # type: ignore
 
 from smart_tests.commands.record.tests import INVALID_TIMESTAMP, parse_launchable_timeformat
+from smart_tests.utils.http_client import get_base_url
 from tests.cli_test_case import CliTestCase
 
 
@@ -62,6 +63,37 @@ class TestsTest(CliTestCase):
         # normal.xml
         self.assertIn('open_class_user_test.rb', gzip.decompress(self.find_request('/events').request.body).decode())
 
+    @responses.activate
+    @mock.patch.dict(os.environ, {"SMART_TESTS_TOKEN": CliTestCase.smart_tests_token})
+    def test_upload_raw_files(self):
+        uploads = []
+
+        def callback(request):
+            body = request.body
+            if hasattr(body, 'read'):
+                body = body.read()
+            self.assertEqual(int(request.headers['Content-Length']), len(body))
+            self.assertTrue(request.headers['Content-Type'].startswith('multipart/form-data; boundary='))
+            uploads.append(body)
+            return 200, {}, ''
+
+        responses.add_callback(
+            responses.POST,
+            f"{get_base_url()}/intake/organizations/{self.organization}/workspaces/{self.workspace}"
+            f"/builds/{self.build_name}/test_sessions/{self.session_id}/test_result_file",
+            callback=callback)
+
+        xml_a = Path(__file__).parent.joinpath('../../data/googletest/output_a.xml').resolve()
+        xml_b = Path(__file__).parent.joinpath('../../data/googletest/output_b.xml').resolve()
+        result = self.cli('record', 'tests', 'googletest', '--session', self.session, str(xml_a), str(xml_b))
+
+        self.assert_success(result)
+        self.assertNotIn("Error uploading raw test result file", result.output)
+        self.assertEqual(len(uploads), 2)
+        for path in (xml_a, xml_b):
+            body = next(b for b in uploads if f'filename="{path.name}"'.encode() in b)
+            self.assertIn(path.read_bytes(), body)
+
     def test_parse_launchable_timeformat(self):
         t1 = "2021-04-01T09:35:47.934+00:00"  # 1617269747.934
         t2 = "2021-05-24T18:29:04.285+00:00"  # 1621880944.285

{smart_tests_cli-2.15.1 → smart_tests_cli-2.15.2}/uv.lock

@@ -428,6 +428,18 @@ wheels = [
     { url = "https://files.pythonhosted.org/packages/1e/db/4254e3eabe8020b458f1a747140d32277ec7a271daf1d235b70dc0b4e6e3/requests-2.32.5-py3-none-any.whl", hash = "sha256:2462f94637a34fd532264295e186976db0f5d453d1cdd31473c85a6a161affb6", size = 64738, upload-time = "2025-08-18T20:46:00.542Z" },
 ]
 
+[[package]]
+name = "requests-toolbelt"
+version = "1.0.0"
+source = { registry = "https://pypi.org/simple" }
+dependencies = [
+    { name = "requests" },
+]
+sdist = { url = "https://files.pythonhosted.org/packages/f3/61/d7545dafb7ac2230c70d38d31cbfe4cc64f7144dc41f6e4e4b78ecd9f5bb/requests-toolbelt-1.0.0.tar.gz", hash = "sha256:7681a0a3d047012b5bdc0ee37d7f8f07ebe76ab08caeccfc3921ce23c88d5bc6", size = 206888, upload-time = "2023-05-01T04:11:33.229Z" }
+wheels = [
+    { url = "https://files.pythonhosted.org/packages/3f/51/d4db610ef29373b879047326cbf6fa98b6c1969d6f6dc423279de2b1be2c/requests_toolbelt-1.0.0-py2.py3-none-any.whl", hash = "sha256:cccfdd665f0a24fcf4726e690f65639d272bb0637b9b92dfd91a5568ccf6bd06", size = 54481, upload-time = "2023-05-01T04:11:28.427Z" },
+]
+
 [[package]]
 name = "responses"
 version = "0.25.8"
@@ -461,6 +473,7 @@ dependencies = [
     { name = "more-itertools" },
     { name = "python-dateutil" },
     { name = "requests" },
+    { name = "requests-toolbelt" },
     { name = "tabulate" },
     { name = "urllib3" },
 ]
@@ -490,6 +503,7 @@ requires-dist = [
     { name = "more-itertools", specifier = ">=7.1.0" },
     { name = "python-dateutil", specifier = ">=2.9" },
     { name = "requests", specifier = ">=2.32.5" },
+    { name = "requests-toolbelt", specifier = ">=1.0.0" },
     { name = "tabulate", specifier = ">=0.9" },
     { name = "urllib3", specifier = ">=2.5" },
 ]

smart_tests_cli-2.15.1/smart_tests_cli.egg-info/scm_version.json

@@ -1,8 +0,0 @@
-{
-  "tag": "2.15.1",
-  "distance": 0,
-  "node": "ga228846b9d3b9fa01d2612f6094402c40af16362",
-  "dirty": false,
-  "branch": "main",
-  "node_date": "2026-09-29"
-}