[autofix.ci] apply automated fixes

This commit is contained in:
autofix-ci[bot]
2026-03-26 21:15:20 +00:00
committed by GitHub
parent 945a81936c
commit 852fb037b2
4 changed files with 442 additions and 368 deletions

View File

@ -1,7 +1,6 @@
"""Tests for download_translations.py."""
import json
import sys
from unittest.mock import patch
import pytest
@ -12,7 +11,7 @@ def _run_main(output_dir: str):
with patch("sys.argv", ["download_translations.py", "--output", output_dir]):
import importlib
import scripts.gp.download_translations as dl_mod # noqa: PLC0415
import scripts.gp.download_translations as dl_mod
importlib.reload(dl_mod)
dl_mod.main()
@ -64,9 +63,9 @@ class TestDownloadTranslations:
with (
patch("scripts.gp.download_translations.get_strings", side_effect=_get_strings),
patch("scripts.gp.download_translations.TARGET_LANGS", ["fr", "de"]),
pytest.raises(SystemExit) as exc_info,
):
with pytest.raises(SystemExit) as exc_info:
_run_main(str(tmp_path))
_run_main(str(tmp_path))
assert exc_info.value.code == 1
# Successful language should still have been written

View File

@ -3,16 +3,15 @@
import base64
import hashlib
import hmac
import json
from unittest.mock import MagicMock, patch
import pytest
# ---------------------------------------------------------------------------
# Helpers
# ---------------------------------------------------------------------------
def _expected_signature(password: str, message: str) -> str:
sig = hmac.new(
bytes(password, "ISO-8859-1"),
@ -26,6 +25,7 @@ def _expected_signature(password: str, message: str) -> str:
# get_headers
# ---------------------------------------------------------------------------
class TestGetHeaders:
def test_get_request_authorization_format(self):
with patch.dict(
@ -33,7 +33,9 @@ class TestGetHeaders:
{"GP_ADMIN_USER_ID": "user123", "GP_ADMIN_PASSWORD": "secret"},
):
import importlib
import scripts.gp.gp_client as gp_client # noqa: PLC0415
from scripts.gp import gp_client
importlib.reload(gp_client)
headers = gp_client.get_headers("https://example.com/api", "GET")
@ -49,7 +51,9 @@ class TestGetHeaders:
{"GP_ADMIN_USER_ID": "user123", "GP_ADMIN_PASSWORD": "secret"},
):
import importlib
import scripts.gp.gp_client as gp_client # noqa: PLC0415
from scripts.gp import gp_client
importlib.reload(gp_client)
headers = gp_client.get_headers("https://example.com/api", "PUT", {"key": "val"})
@ -62,7 +66,9 @@ class TestGetHeaders:
{"GP_ADMIN_USER_ID": "user123", "GP_ADMIN_PASSWORD": "secret"},
):
import importlib
import scripts.gp.gp_client as gp_client # noqa: PLC0415
from scripts.gp import gp_client
importlib.reload(gp_client)
headers = gp_client.get_headers("https://example.com/api", "PATCH", {})
@ -75,7 +81,9 @@ class TestGetHeaders:
{"GP_ADMIN_USER_ID": "user123", "GP_ADMIN_PASSWORD": "secret"},
):
import importlib
import scripts.gp.gp_client as gp_client # noqa: PLC0415
from scripts.gp import gp_client
importlib.reload(gp_client)
with patch("scripts.gp.gp_client.datetime") as mock_dt:
@ -91,17 +99,23 @@ class TestGetHeaders:
# list_bundles
# ---------------------------------------------------------------------------
class TestListBundles:
def test_returns_parsed_json_on_success(self):
mock_response = MagicMock()
mock_response.json.return_value = {"bundleIds": ["langflow-ui"]}
with patch.dict(
"os.environ",
{"GP_ADMIN_USER_ID": "u", "GP_ADMIN_PASSWORD": "p"},
), patch("scripts.gp.gp_client.requests.get", return_value=mock_response) as mock_get:
with (
patch.dict(
"os.environ",
{"GP_ADMIN_USER_ID": "u", "GP_ADMIN_PASSWORD": "p"},
),
patch("scripts.gp.gp_client.requests.get", return_value=mock_response) as mock_get,
):
import importlib
import scripts.gp.gp_client as gp_client # noqa: PLC0415
from scripts.gp import gp_client
importlib.reload(gp_client)
result = gp_client.list_bundles()
@ -116,12 +130,17 @@ class TestListBundles:
mock_response = MagicMock()
mock_response.raise_for_status.side_effect = req.exceptions.HTTPError("401 Unauthorized")
with patch.dict(
"os.environ",
{"GP_ADMIN_USER_ID": "u", "GP_ADMIN_PASSWORD": "p"},
), patch("scripts.gp.gp_client.requests.get", return_value=mock_response):
with (
patch.dict(
"os.environ",
{"GP_ADMIN_USER_ID": "u", "GP_ADMIN_PASSWORD": "p"},
),
patch("scripts.gp.gp_client.requests.get", return_value=mock_response),
):
import importlib
import scripts.gp.gp_client as gp_client # noqa: PLC0415
from scripts.gp import gp_client
importlib.reload(gp_client)
with pytest.raises(req.exceptions.HTTPError):
@ -130,15 +149,20 @@ class TestListBundles:
def test_raises_on_timeout(self):
import requests as req
with patch.dict(
"os.environ",
{"GP_ADMIN_USER_ID": "u", "GP_ADMIN_PASSWORD": "p"},
), patch(
"scripts.gp.gp_client.requests.get",
side_effect=req.exceptions.Timeout,
with (
patch.dict(
"os.environ",
{"GP_ADMIN_USER_ID": "u", "GP_ADMIN_PASSWORD": "p"},
),
patch(
"scripts.gp.gp_client.requests.get",
side_effect=req.exceptions.Timeout,
),
):
import importlib
import scripts.gp.gp_client as gp_client # noqa: PLC0415
from scripts.gp import gp_client
importlib.reload(gp_client)
with pytest.raises(req.exceptions.Timeout):
@ -149,17 +173,23 @@ class TestListBundles:
# upload_strings
# ---------------------------------------------------------------------------
class TestUploadStrings:
def test_successful_upload(self):
mock_response = MagicMock()
mock_response.json.return_value = {"status": "ok"}
with patch.dict(
"os.environ",
{"GP_ADMIN_USER_ID": "u", "GP_ADMIN_PASSWORD": "p"},
), patch("scripts.gp.gp_client.requests.put", return_value=mock_response) as mock_put:
with (
patch.dict(
"os.environ",
{"GP_ADMIN_USER_ID": "u", "GP_ADMIN_PASSWORD": "p"},
),
patch("scripts.gp.gp_client.requests.put", return_value=mock_response) as mock_put,
):
import importlib
import scripts.gp.gp_client as gp_client # noqa: PLC0415
from scripts.gp import gp_client
importlib.reload(gp_client)
result = gp_client.upload_strings({"hello": "Hello"})
@ -173,12 +203,17 @@ class TestUploadStrings:
mock_response = MagicMock()
mock_response.raise_for_status.side_effect = req.exceptions.HTTPError("500")
with patch.dict(
"os.environ",
{"GP_ADMIN_USER_ID": "u", "GP_ADMIN_PASSWORD": "p"},
), patch("scripts.gp.gp_client.requests.put", return_value=mock_response):
with (
patch.dict(
"os.environ",
{"GP_ADMIN_USER_ID": "u", "GP_ADMIN_PASSWORD": "p"},
),
patch("scripts.gp.gp_client.requests.put", return_value=mock_response),
):
import importlib
import scripts.gp.gp_client as gp_client # noqa: PLC0415
from scripts.gp import gp_client
importlib.reload(gp_client)
with pytest.raises(req.exceptions.HTTPError):
@ -189,19 +224,23 @@ class TestUploadStrings:
# get_strings
# ---------------------------------------------------------------------------
class TestGetStrings:
def test_returns_parsed_json(self):
mock_response = MagicMock()
mock_response.json.return_value = {
"resourceStrings": {"hello": {"value": "Bonjour"}}
}
mock_response.json.return_value = {"resourceStrings": {"hello": {"value": "Bonjour"}}}
with patch.dict(
"os.environ",
{"GP_ADMIN_USER_ID": "u", "GP_ADMIN_PASSWORD": "p"},
), patch("scripts.gp.gp_client.requests.get", return_value=mock_response):
with (
patch.dict(
"os.environ",
{"GP_ADMIN_USER_ID": "u", "GP_ADMIN_PASSWORD": "p"},
),
patch("scripts.gp.gp_client.requests.get", return_value=mock_response),
):
import importlib
import scripts.gp.gp_client as gp_client # noqa: PLC0415
from scripts.gp import gp_client
importlib.reload(gp_client)
result = gp_client.get_strings("fr")
@ -214,12 +253,17 @@ class TestGetStrings:
mock_response = MagicMock()
mock_response.raise_for_status.side_effect = req.exceptions.HTTPError("403 Forbidden")
with patch.dict(
"os.environ",
{"GP_ADMIN_USER_ID": "u", "GP_ADMIN_PASSWORD": "p"},
), patch("scripts.gp.gp_client.requests.get", return_value=mock_response):
with (
patch.dict(
"os.environ",
{"GP_ADMIN_USER_ID": "u", "GP_ADMIN_PASSWORD": "p"},
),
patch("scripts.gp.gp_client.requests.get", return_value=mock_response),
):
import importlib
import scripts.gp.gp_client as gp_client # noqa: PLC0415
from scripts.gp import gp_client
importlib.reload(gp_client)
with pytest.raises(req.exceptions.HTTPError):

View File

@ -1,9 +1,7 @@
"""Tests for upload_strings.py."""
import json
import sys
from pathlib import Path
from unittest.mock import MagicMock, call, patch
from unittest.mock import patch
import pytest
@ -13,7 +11,7 @@ def _run_main(source_path: str):
with patch("sys.argv", ["upload_strings.py", "--source", source_path]):
import importlib
import scripts.gp.upload_strings as upload_mod # noqa: PLC0415
import scripts.gp.upload_strings as upload_mod
importlib.reload(upload_mod)
upload_mod.main()
@ -71,6 +69,6 @@ class TestUploadStrings:
patch("scripts.gp.upload_strings.list_bundles", return_value={"bundleIds": []}),
patch("scripts.gp.upload_strings.create_bundle"),
patch("scripts.gp.upload_strings.upload_strings"),
pytest.raises(FileNotFoundError),
):
with pytest.raises(FileNotFoundError):
_run_main(missing)
_run_main(missing)

657
uv.lock generated

File diff suppressed because it is too large Load Diff