speedtest-cli-ng 0.3.0__tar.gz → 1.0.0__tar.gz

{speedtest_cli_ng-0.3.0 → speedtest_cli_ng-1.0.0}/.gitignore

@@ -197,9 +197,9 @@ cython_debug/
 .abstra/
 
 # Visual Studio Code
-#   Visual Studio Code specific template is maintained in a separate VisualStudioCode.gitignore 
+#   Visual Studio Code specific template is maintained in a separate VisualStudioCode.gitignore
 #   that can be found at https://github.com/github/gitignore/blob/main/Global/VisualStudioCode.gitignore
-#   and can be added to the global gitignore or merged into this file. However, if you prefer, 
+#   and can be added to the global gitignore or merged into this file. However, if you prefer,
 #   you could uncomment the following to ignore the entire vscode folder
 # .vscode/
 # Temporary file for partial code execution
@@ -217,4 +217,4 @@ marimo/_lsp/
 __marimo__/
 
 # Streamlit
-.streamlit/secrets.toml
+.streamlit/secrets.toml

{speedtest_cli_ng-0.3.0 → speedtest_cli_ng-1.0.0}/NOTICE

@@ -2,5 +2,5 @@ speedtest-cli
 Copyright (c) 2026 Michał Korczak
 Copyright (c) 2012-2026 Matt Martz
 
-This product includes software originally developed by Matt Martz 
+This product includes software originally developed by Matt Martz
 (https://github.com/sivel/speedtest-cli).

{speedtest_cli_ng-0.3.0 → speedtest_cli_ng-1.0.0}/PKG-INFO

@@ -1,16 +1,16 @@
-Metadata-Version: 2.4
+Metadata-Version: 2.5
 Name: speedtest-cli-ng
-Version: 0.3.0
+Version: 1.0.0
 Summary: Next generation CLI for testing internet bandwidth using speedtest.net
 Project-URL: Homepage, https://github.com/Omikorin/speedtest-cli
 Project-URL: Source, https://github.com/Omikorin/speedtest-cli
 Project-URL: Issues, https://github.com/Omikorin/speedtest-cli/issues
-Author-email: Michał Korczak <git@omikor.in>
+Author: Michał Korczak
 License-Expression: Apache-2.0
 License-File: LICENSE
 License-File: NOTICE
 Keywords: bandwidth,cli,internet,speedtest,speedtest.net
-Classifier: Development Status :: 4 - Beta
+Classifier: Development Status :: 5 - Production/Stable
 Classifier: Environment :: Console
 Classifier: Intended Audience :: Developers
 Classifier: Intended Audience :: System Administrators
@@ -22,7 +22,7 @@ Classifier: Topic :: System :: Networking
 Classifier: Topic :: Terminals
 Classifier: Topic :: Utilities
 Requires-Python: >=3.14
-Requires-Dist: httpx2<3,>=2.5.0
+Requires-Dist: httpx2<3,>=2.12.0
 Requires-Dist: rich<16,>=15.0.0
 Description-Content-Type: text/markdown
 
@@ -34,12 +34,8 @@ Description-Content-Type: text/markdown
 [![CI](https://github.com/Omikorin/speedtest-cli/actions/workflows/ci.yml/badge.svg?branch=main)](https://github.com/Omikorin/speedtest-cli/actions/workflows/ci.yml)
 [![Quality Gate Status](https://sonarcloud.io/api/project_badges/measure?project=Omikorin_speedtest-cli&metric=alert_status)](https://sonarcloud.io/summary/new_code?id=Omikorin_speedtest-cli)
 
----
-
 Next generation CLI for testing internet bandwidth using speedtest.net
 
-Rebuilt for modern Python 3.14+
-
 ## Getting started
 
 ### Installation
@@ -115,11 +111,21 @@ Miscellaneous Options:
 ### Requirements
 
 - Python 3.14+
-- uv 0.11+
+- uv 0.13+
+- prek 0.5+
 
-### Setup project
+### Setup pre-commit hooks
 
-This will install project's dependencies. Keep in mind that there are no runtime dependencies in production.
+You can install hooks to automatically run the validation checks:
+
+```shell
+uv tool install prek
+prek install
+```
+
+### Install dependencies
+
+This will install project's dependencies
 
 ```bash
 uv sync
@@ -128,6 +134,12 @@ uv sync
 ### Basic workflow
 
 ```bash
+# Run pre-commit checks for changed files
+prek
+
+# Run all pre-commit checks
+prek --all-files
+
 # Run format
 uv run ruff format --check .
 

{speedtest_cli_ng-0.3.0 → speedtest_cli_ng-1.0.0}/README.md

@@ -6,12 +6,8 @@
 [![CI](https://github.com/Omikorin/speedtest-cli/actions/workflows/ci.yml/badge.svg?branch=main)](https://github.com/Omikorin/speedtest-cli/actions/workflows/ci.yml)
 [![Quality Gate Status](https://sonarcloud.io/api/project_badges/measure?project=Omikorin_speedtest-cli&metric=alert_status)](https://sonarcloud.io/summary/new_code?id=Omikorin_speedtest-cli)
 
----
-
 Next generation CLI for testing internet bandwidth using speedtest.net
 
-Rebuilt for modern Python 3.14+
-
 ## Getting started
 
 ### Installation
@@ -87,11 +83,21 @@ Miscellaneous Options:
 ### Requirements
 
 - Python 3.14+
-- uv 0.11+
+- uv 0.13+
+- prek 0.5+
 
-### Setup project
+### Setup pre-commit hooks
 
-This will install project's dependencies. Keep in mind that there are no runtime dependencies in production.
+You can install hooks to automatically run the validation checks:
+
+```shell
+uv tool install prek
+prek install
+```
+
+### Install dependencies
+
+This will install project's dependencies
 
 ```bash
 uv sync
@@ -100,6 +106,12 @@ uv sync
 ### Basic workflow
 
 ```bash
+# Run pre-commit checks for changed files
+prek
+
+# Run all pre-commit checks
+prek --all-files
+
 # Run format
 uv run ruff format --check .
 

speedtest_cli_ng-1.0.0/prek.toml

@@ -0,0 +1,69 @@
+minimum_prek_version = "0.5.10"
+
+[update]
+cooldown_days = 7
+freeze = true
+include_tags = "v*"
+exclude_tags = ["*-{alpha,beta,rc}*"]
+repos = {
+  "https://github.com/python-jsonschema/check-jsonschema" = { include_tags = "*" },
+  "https://github.com/astral-sh/uv-pre-commit" = { include_tags = "*" }
+}
+
+[[repos]]
+repo = "builtin"
+hooks = [
+  { id = "check-merge-conflict" },
+  { id = "trailing-whitespace" },
+  { id = "end-of-file-fixer" },
+  { id = "mixed-line-ending" },
+  { id = "check-json" },
+  { id = "check-json5" },
+  { id = "check-toml" },
+  { id = "check-yaml" },
+]
+
+[[repos]]
+repo = "https://github.com/abravalheri/validate-pyproject"
+rev = "82d21874c3242e4dc27bb47641c996902ef64eef"  # frozen: v0.27
+hooks = [
+  { id = "validate-pyproject" },
+]
+
+[[repos]]
+repo = "https://github.com/python-jsonschema/check-jsonschema"
+rev = "a76b92851fc9eede9f9bc4073eb55202f34eb0df"  # frozen: 0.39.0
+hooks = [
+  { id = "check-github-workflows" },
+  { id = "check-renovate", additional_dependencies = ['json5'] },
+]
+
+# zizmor detects security vulnerabilities in GitHub Actions workflows
+[[repos]]
+repo = "https://github.com/zizmorcore/zizmor-pre-commit"
+rev = "c92afdf2c6ca336aaa8f39e2bd8d48f62dbd8e1f"  # frozen: v1.31.1
+hooks = [
+  { id = "zizmor", args = ["--no-progress", "--fix"] },
+]
+
+[[repos]]
+repo = "https://github.com/astral-sh/uv-pre-commit"
+rev = "46787482b9fb2003c56e0f7592e312ef381829d9"  # frozen: 0.13.6
+hooks = [
+  { id = "uv-lock" },
+]
+
+[[repos]]
+repo = "https://github.com/astral-sh/ruff-pre-commit"
+rev = "25bcf95026b96e2dd3c0aec3090c3d6ef6ca2e6c"  # frozen: v0.17.3
+hooks = [
+  { id = "ruff-check", args = ["--fix", "--exit-non-zero-on-fix"] },
+  { id = "ruff-format", types_or = ["python", "pyi", "jupyter", "markdown"] },
+]
+
+[[repos]]
+repo = "https://github.com/astral-sh/ty-pre-commit"
+rev = "18b2b8bb1b9b90d2d21283756e89b07fc9c747bf"  # frozen: v0.0.95
+hooks = [
+  { id = "ty" },
+]

{speedtest_cli_ng-0.3.0 → speedtest_cli_ng-1.0.0}/pyproject.toml

@@ -7,13 +7,13 @@ name = "speedtest-cli-ng"
 description = "Next generation CLI for testing internet bandwidth using speedtest.net"
 keywords = ["speedtest", "speedtest.net", "bandwidth", "internet", "cli"]
 license = "Apache-2.0"
-version = "0.3.0"
+version = "1.0.0"
 requires-python = ">=3.14"
-authors = [{ name = "Michał Korczak", email = "git@omikor.in" }]
+authors = [{ name = "Michał Korczak" }]
 license-files = ["LICENSE", "NOTICE"]
 readme = "README.md"
 classifiers = [
-    "Development Status :: 4 - Beta",
+    "Development Status :: 5 - Production/Stable",
     "Environment :: Console",
     "Intended Audience :: Developers",
     "Intended Audience :: System Administrators",
@@ -26,20 +26,10 @@ classifiers = [
     "Topic :: Utilities",
 ]
 dependencies = [
-    "httpx2>=2.5.0,<3",
+    "httpx2>=2.12.0,<3",
     "rich>=15.0.0,<16",
 ]
 
-[dependency-groups]
-dev = [
-    # Tests
-    "pytest>=9.0.0,<10",
-    "pytest-cov>=7.0.0,<8",
-    # Linting
-    "ruff>=0.15.0,<1",
-    "ty>=0.0.55,<1",
-]
-
 [project.urls]
 Homepage = "https://github.com/Omikorin/speedtest-cli"
 Source = "https://github.com/Omikorin/speedtest-cli"
@@ -50,6 +40,16 @@ speedtest = "speedtest.__main__:main"
 speedtest-cli = "speedtest.__main__:main"
 speedtest-cli-ng = "speedtest.__main__:main"
 
+[dependency-groups]
+dev = [
+    # Tests
+    "pytest>=9.1.1,<10",
+    "pytest-cov>=7.1.0,<8",
+    # Linting
+    "ruff>=0.16.4,<1",
+    "ty>=0.0.73,<1",
+]
+
 [tool.hatch.build.targets.sdist]
 exclude = [
     ".github",
@@ -61,7 +61,7 @@ exclude = [
 packages = ["src/speedtest"]
 
 [tool.uv]
-required-version = ">=0.11.0"
+required-version = ">=0.13.0"
 exclude-newer = "7 days"
 
 [[tool.uv.index]]
@@ -75,7 +75,7 @@ line-length = 120
 src = ["speedtest", "tests"]
 
 [tool.ruff.lint]
-select = [
+extend-select = [
     "ANN", # flake8 annotations
     "B",   # flake8 bugbear
     "PYI", # flake8 pyi
@@ -102,6 +102,5 @@ possibly-unresolved-reference = "warn"
 [tool.pytest]
 minversion = "9.0"
 # Automatically show extra test info, fail if coverage drops, and output a terminal coverage report
-addopts = ["-v", "--cov=speedtest", "--cov-report=term-missing"]
+addopts = ["-v", "--cov=speedtest", "--cov-branch", "--cov-report=term-missing"]
 testpaths = ["tests"]
-

{speedtest_cli_ng-0.3.0 → speedtest_cli_ng-1.0.0}/src/speedtest/__main__.py

@@ -28,7 +28,7 @@ def main() -> int:
 
         return int(code)
 
-    except Exception:
+    except OSError, RuntimeError, ValueError:
         logger.exception("An unexpected error occurred.")
         return ExitStatus.ERROR.value
 

{speedtest_cli_ng-0.3.0 → speedtest_cli_ng-1.0.0}/src/speedtest/cli/runner.py

@@ -91,7 +91,7 @@ def shell() -> int:
 
         if ctx.share:
             with console.status("[bold magenta]Generating share link..."):
-                results.share_url = client.generate_share_link(results)
+                results.share_url = client.generate_share_link(results, ctx.api_config)
 
             logger.info(f"Share results: {results.share_url}")
 

{speedtest_cli_ng-0.3.0 → speedtest_cli_ng-1.0.0}/src/speedtest/client.py

@@ -1,10 +1,11 @@
-import urllib.error
-import urllib.parse
-import urllib.request
+import json
 from hashlib import md5
 from operator import attrgetter
 
+import httpx2
+
 from speedtest.engine import get_best_server, run_download_test, run_upload_test
+from speedtest.engine.network import build_user_agent
 from speedtest.exceptions import CLIError, NoMatchedServerError
 from speedtest.models import ApiConfig, RunContext, Server, SpeedtestResult
 
@@ -55,59 +56,65 @@ class SpeedtestClient:
 
         return run_upload_test(best_server_url=server.url, ctx=ctx)
 
-    def generate_share_link(self, results: SpeedtestResult) -> str:
+    def generate_share_link(self, results: SpeedtestResult, config: ApiConfig) -> str:
         """POST data to the speedtest.net API to obtain a share results link."""
 
-        if not results.is_complete or not results.server:
+        if results.ping_ms is None or not results.server:
             raise CLIError("Cannot generate share link: missing test results.")
 
-        # The legacy Ookla API expects speeds in kilobits per second (kbps)
+        # The web client reports kbps and signs ping-upload-download with a fixed salt.
         download_kbps = round((results.download_bps or 0) / 1000.0)
         upload_kbps = round((results.upload_bps or 0) / 1000.0)
-        ping = round(results.ping_ms or 0)
+        ping = round(results.ping_ms)
 
-        hash_str = f"{ping}-{upload_kbps}-{download_kbps}-297aae72"
-        hash_val = md5(hash_str.encode()).hexdigest()
+        hash_val = md5(f"{ping}-{upload_kbps}-{download_kbps}-817d699764d33f89c".encode()).hexdigest()
+        host, _, port = results.server.host.partition(":")
 
-        api_parameters = {
-            "recommendedserverid": results.server.id,
+        payload: dict[str, object] = {
+            "app": {"sdk": {"commit": "a36027299986af441deb58cd730f432fcad58871", "version": "3.1.1"}},
+            "serverid": results.server.id,
+            "testmethod": "http,http,http",
+            "source": "st4-js",
+            "configs": {"host": host, "port": int(port) if port else 8080},
             "ping": ping,
-            "screenresolution": "",
-            "promo": "",
-            "download": download_kbps,
-            "screendpi": "",
-            "upload": upload_kbps,
-            "testmethod": "http",
+            "ispName": config.isp_name,
+            "guid": config.guid,
             "hash": hash_val,
-            "touchscreen": "none",
-            "startmode": "pingselect",
-            "accuracy": 1,
-            "bytesreceived": results.download_bytes or 0,
-            "bytessent": results.upload_bytes or 0,
-            "serverid": results.server.id,
+            "clientip": config.ip_address,
+        }
+        if download_kbps:
+            payload["download"] = download_kbps
+        if upload_kbps:
+            payload["upload"] = upload_kbps
+
+        headers = {
+            "User-Agent": build_user_agent(),
+            "Accept": "application/json",
+            "Origin": "https://www.speedtest.net",
+            "Referer": "https://www.speedtest.net/",
+            "Content-Type": "application/json;charset=UTF-8",
         }
 
-        api_data = urllib.parse.urlencode(api_parameters).encode()
-        headers = {"Referer": "https://c.speedtest.net/flash/speedtest.swf"}
-
-        req = urllib.request.Request(
-            "https://www.speedtest.net/api/api.php", data=api_data, headers=headers, method="POST"
-        )
-
+        body = ""
         try:
-            with urllib.request.urlopen(req) as response:
-                if response.status != 200:
-                    raise CLIError(f"Could not submit results. HTTP {response.status}")
-
-                body = response.read().decode(errors="ignore")
-
-        except urllib.error.URLError as e:
+            response = httpx2.post(
+                "https://www.speedtest.net/api/results.php",
+                content=json.dumps(payload),
+                headers=headers,
+                timeout=10.0,
+            )
+            response.raise_for_status()
+            body = response.text
+            resultid = response.json().get("resultid")
+
+        except httpx2.HTTPStatusError as e:
+            raise CLIError(f"Could not submit results. HTTP {e.response.status_code}") from e
+        except httpx2.RequestError as e:
             raise CLIError(f"Failed to connect to Share API: {e}") from e
+        except ValueError, AttributeError:
+            resultid = None
 
-        qsargs = urllib.parse.parse_qs(body)
-        resultid = qsargs.get("resultid")
-
-        if not resultid or len(resultid) != 1:
-            raise CLIError("Could not parse result ID from API response.")
+        if not resultid:
+            raise CLIError(f"Could not parse result ID from API response: {body[:200]!r}")
 
-        return f"https://www.speedtest.net/result/{resultid[0]}.png"
+        return f"https://www.speedtest.net/result/{resultid}.png"

{speedtest_cli_ng-0.3.0 → speedtest_cli_ng-1.0.0}/src/speedtest/engine/config.py

@@ -7,6 +7,7 @@ from json import JSONDecodeError
 
 import httpx2
 
+from speedtest.exceptions import CLIError
 from speedtest.models import ApiConfig
 from speedtest.utils import CONFIG_URL
 
@@ -15,11 +16,9 @@ from .network import build_user_agent
 __all__ = ["ConfigFetchError", "fetch_raw_config", "get_config", "parse_config"]
 
 
-class ConfigFetchError(Exception):
+class ConfigFetchError(CLIError):
     """Raised when the configuration cannot be retrieved or parsed."""
 
-    pass
-
 
 def fetch_raw_config(url: str = CONFIG_URL) -> str:
     """

{speedtest_cli_ng-0.3.0 → speedtest_cli_ng-1.0.0}/src/speedtest/engine/network.py

@@ -15,11 +15,13 @@ import httpx2
 
 from speedtest import __version__
 from speedtest.exceptions import UploadTimeoutError
+from speedtest.utils import logger
 
 # --- Constants ---
-CHUNK_SIZE_BYTES = 10240
 UPLOAD_RESPONSE_TRUNCATION = 11
-DUMMY_CHUNK = b"0123456789ABCDEFGHIJKLMNOPQRSTUVWXYZ" * 300
+DUMMY_CHUNK = b"0123456789ABCDEFGHIJKLMNOPQRSTUVWXYZ" * 1820  # ~64 KiB
+CHUNK_SIZE_BYTES = len(DUMMY_CHUNK)
+UPLOAD_BODY_PREFIX = b"content1="  # upload.php expects a form-encoded body
 
 __all__ = [
     "HTTPUploaderData",
@@ -89,7 +91,7 @@ class HTTPUploaderData:
         if remaining <= 0:
             return b""
 
-        max_alloc = len(DUMMY_CHUNK)
+        max_alloc = CHUNK_SIZE_BYTES
 
         if n < 0 or n > remaining:
             read_size = min(remaining, max_alloc)
@@ -97,6 +99,8 @@ class HTTPUploaderData:
             read_size = min(n, remaining, max_alloc)
 
         chunk = DUMMY_CHUNK[:read_size]
+        if self.total_bytes_read == 0 and read_size > len(UPLOAD_BODY_PREFIX):
+            chunk = UPLOAD_BODY_PREFIX + chunk[len(UPLOAD_BODY_PREFIX) :]
 
         self.total_bytes_read += read_size
         return chunk
@@ -159,8 +163,10 @@ def upload_worker(
         return 0
 
     try:
-        client.send(request)
+        response = client.send(request)
+        logger.debug(f"Upload {payload_data.length} B -> HTTP {response.status_code}")
         return payload_data.total_bytes_read
 
-    except httpx2.RequestError, UploadTimeoutError:
+    except (httpx2.RequestError, UploadTimeoutError) as e:
+        logger.debug(f"Upload ended early at {payload_data.total_bytes_read}/{payload_data.length} B: {e!r}")
         return payload_data.total_bytes_read

{speedtest_cli_ng-0.3.0 → speedtest_cli_ng-1.0.0}/src/speedtest/engine/servers.py

@@ -65,7 +65,7 @@ def get_best_server(closest_servers: list[Server]) -> tuple[Server, float]:
             for future in as_completed(future_to_server):
                 try:
                     yield future.result()
-                except Exception as e:
+                except OSError as e:
                     logger.debug(f"Server TCP ping generated an exception: {e}")
 
     best_server, lowest_latency = find_fastest_server(_execute_pings())

{speedtest_cli_ng-0.3.0 → speedtest_cli_ng-1.0.0}/src/speedtest/engine/transfer.py

@@ -68,6 +68,7 @@ def _generate_upload_payloads(
         headers = {
             "User-Agent": user_agent,
             "Content-Length": str(size),
+            "Content-Type": "application/x-www-form-urlencoded",
             "Cache-Control": "no-cache",
         }
 
@@ -75,6 +76,16 @@ def _generate_upload_payloads(
         yield req, data
 
 
+def _resolve_upload_url(url: str) -> str:
+    """Follow redirects with an empty POST; streamed bodies can't be replayed after a 307."""
+
+    try:
+        with httpx2.Client(timeout=5.0, follow_redirects=True, verify=False) as client:
+            return str(client.post(url, content=b"").url)
+    except httpx2.HTTPError:
+        return url
+
+
 def run_download_test(best_server_url: str, ctx: RunContext) -> tuple[int, float]:
     """Execute a multi-threaded download speed test leveraging connection pooling."""
 
@@ -104,7 +115,7 @@ def run_download_test(best_server_url: str, ctx: RunContext) -> tuple[int, float
         for future in as_completed(futures):
             try:
                 bytes_received += future.result()
-            except Exception as e:
+            except (OSError, httpx2.HTTPError) as e:
                 logger.debug(f"Download thread failed: {e}")
 
             if ctx.shutdown_event and ctx.shutdown_event.is_set():
@@ -123,6 +134,7 @@ def run_upload_test(best_server_url: str, ctx: RunContext) -> tuple[int, float]:
     test_length = 10.0
     bytes_sent = 0
 
+    best_server_url = _resolve_upload_url(best_server_url)
     tasks = list(_generate_upload_payloads(best_server_url, test_length, ctx.shutdown_event))
     start_time = time.monotonic()
 
@@ -143,7 +155,7 @@ def run_upload_test(best_server_url: str, ctx: RunContext) -> tuple[int, float]:
         for future in as_completed(futures):
             try:
                 bytes_sent += future.result()
-            except Exception as e:
+            except (OSError, httpx2.HTTPError) as e:
                 logger.debug(f"Upload thread failed: {e}")
 
             if ctx.shutdown_event and ctx.shutdown_event.is_set():

{speedtest_cli_ng-0.3.0 → speedtest_cli_ng-1.0.0}/src/speedtest/utils/logger.py

@@ -11,6 +11,7 @@ __all__ = ["console", "logger", "setup_logging"]
 
 logger = logging.getLogger("speedtest")
 console = Console()
+error_console = Console(stderr=True)
 
 
 def setup_logging(debug: bool = False, quiet: bool = False) -> None:
@@ -29,8 +30,9 @@ def setup_logging(debug: bool = False, quiet: bool = False) -> None:
 
     logger.setLevel(level)
 
+    # Quiet mode reserves stdout for JSON, so log records move to stderr instead of vanishing.
     handler = RichHandler(
-        console=console,
+        console=error_console if quiet else console,
         show_time=debug,
         show_level=True,
         show_path=debug,

{speedtest_cli_ng-0.3.0 → speedtest_cli_ng-1.0.0}/tests/cli/test_display.py

@@ -24,7 +24,10 @@ def test_print_server_list_success(mock_server: Server) -> None:
 
         table_arg = mock_print.call_args[0][0]
         assert table_arg.title == "Available Target Servers"
-        assert len(table_arg.columns) == 5
+        assert [c.header for c in table_arg.columns] == ["ID", "Distance", "Name", "Country", "Sponsor"]
+
+        row = [next(iter(c.cells)) for c in table_arg.columns]
+        assert row == ["1234", "17 km", "New York", "United States", "Acme Corp"]
 
 
 def test_print_server_list_broken_pipe(mock_server: Server) -> None:
@@ -91,3 +94,16 @@ def test_print_json_tty_indenting(capsys: CaptureFixture[str]) -> None:
     captured = capsys.readouterr()
 
     assert '{\n  "timestamp":' in captured.out
+    assert json.loads(captured.out)["ping"] == pytest.approx(10.0)
+    assert json.loads(captured.out)["server"] is None
+
+
+def test_print_json_compact_when_piped(capsys: CaptureFixture[str], mock_result: SpeedtestResult) -> None:
+    """Piped output must be a single line so other tools can read it."""
+
+    with patch.object(sys.stdout, "isatty", return_value=False):
+        print_json(mock_result)
+
+    out = capsys.readouterr().out
+    assert out.count("\n") == 1
+    assert json.loads(out)["bytes_sent"] == 6_250_000

{speedtest_cli_ng-0.3.0 → speedtest_cli_ng-1.0.0}/tests/cli/test_parser.py

@@ -67,9 +67,8 @@ def test_parser_thread_options() -> None:
         assert args.threads is None
 
     # Test mutual exclusivity of threads and single
-    with patch.object(sys, "argv", ["speedtest-cli", "--threads", "4", "--single"]):
-        with pytest.raises(SystemExit):
-            parse_args()
+    with patch.object(sys, "argv", ["speedtest-cli", "--threads", "4", "--single"]), pytest.raises(SystemExit):
+        parse_args()
 
 
 def test_parser_output_options() -> None: