From 61d221293ddabc30c8ee48f94701e66fdd7282a7 Mon Sep 17 00:00:00 2001 From: Ian Schwartz <168640+studiozeroseven@users.noreply.github.com> Date: Sun, 21 Jun 2026 22:58:24 -0500 Subject: [PATCH] style: fix lint violations in test_manual_import_fallback MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit CI lint job (pre-commit: black/isort/flake8, line-length=79) failed on this file. - Collapse overly-nested expression breaks to one-per-arg - Trim a redundant word from a docstring to fit 79 chars - Re-run pre-commit-equivalent locally — black, isort, flake8 all pass --- .../test_src/test_manual_import_fallback.py | 23 ++++++++++++++----- 1 file changed, 17 insertions(+), 6 deletions(-) diff --git a/backend/appsettings/tests/test_src/test_manual_import_fallback.py b/backend/appsettings/tests/test_src/test_manual_import_fallback.py index 1da75830..8903adca 100644 --- a/backend/appsettings/tests/test_src/test_manual_import_fallback.py +++ b/backend/appsettings/tests/test_src/test_manual_import_fallback.py @@ -10,10 +10,8 @@ ValueError fallback shape. from unittest.mock import patch import pytest - from appsettings.src.manual import ManualImport - SAMPLE_VIDEO = { "media": "/cache/import/dQw4w9WgXcQ.mp4", "video_id": "dQw4w9WgXcQ", @@ -24,7 +22,12 @@ SAMPLE_VIDEO = { EMBED_JSON_DATA = {"video_id": "dQw4w9WgXcQ", "media_url": "channel/test.mp4"} -CONFIG = {"downloads": {"integrate_ryd": False, "integrate_sponsorblock": False}} +CONFIG = { + "downloads": { + "integrate_ryd": False, + "integrate_sponsorblock": False, + } +} def _make_import(ignore_error: bool = False) -> ManualImport: @@ -80,9 +83,13 @@ def test_value_error_still_falls_back_to_embedded(): def test_keyerror_with_no_embed_and_strict_mode_raises(): - """strict mode (ignore_error=False) must still raise when both paths fail.""" + """strict mode (ignore_error=False) raises when both paths fail.""" with ( - patch.object(ManualImport, "index_metadata", side_effect=KeyError("id")), + patch.object( + ManualImport, + "index_metadata", + side_effect=KeyError("id"), + ), patch("appsettings.src.manual.IndexFromEmbed") as mock_embed, ): mock_embed.return_value.run_index.return_value = None @@ -93,7 +100,11 @@ def test_keyerror_with_no_embed_and_strict_mode_raises(): def test_keyerror_with_no_embed_and_lenient_mode_silent(): """ignore_error=True must swallow the failure when fallback also empty.""" with ( - patch.object(ManualImport, "index_metadata", side_effect=KeyError("id")), + patch.object( + ManualImport, + "index_metadata", + side_effect=KeyError("id"), + ), patch("appsettings.src.manual.IndexFromEmbed") as mock_embed, ): mock_embed.return_value.run_index.return_value = None