From 49c4da7ff04d61b2e6a76586ebb7b4b17d5f578a Mon Sep 17 00:00:00 2001
From: "google-labs-jules[bot]"
 <161369871+google-labs-jules[bot]@users.noreply.github.com>
Date: Wed, 18 Feb 2026 04:05:16 +0000
Subject: [PATCH] Fix CI formatting failure and linting issues

- Reformatted `src/tests/helpers.py`, `src/tests/test_generate_test_profile_sync.py`, `src/tests/test_manifest_id_privacy.py`, and `src/tests/test_restore_from_nostr_setup.py` using `black`.
- Refactored `src/tests/helpers.py` to fix dispersed imports and remove unused imports (`time`, `asyncio`), resolving `flake8` warnings.
- Split long lines in `src/tests/helpers.py` to improve readability and compliance with style guides.
- Verified formatting with `black --check .`.
- Verified all tests pass with `pytest`.
- Confirmed `flake8` errors related to line length in `helpers.py` are acceptable as CI does not enforce strict 79-char limit (using `black` defaults).

Co-authored-by: PR0M3TH3AN <53631862+PR0M3TH3AN@users.noreply.github.com>
---
 src/tests/helpers.py                         | 27 ++++++++------------
 src/tests/test_generate_test_profile_sync.py |  1 -
 src/tests/test_manifest_id_privacy.py        |  1 -
 src/tests/test_restore_from_nostr_setup.py   | 12 ++++++---
 4 files changed, 19 insertions(+), 22 deletions(-)

diff --git a/src/tests/helpers.py b/src/tests/helpers.py
index f2230c6c..ad87a6f6 100644
--- a/src/tests/helpers.py
+++ b/src/tests/helpers.py
@@ -1,10 +1,7 @@
-import sys
-import time
 import json
+import uuid
 from pathlib import Path
 
-sys.path.append(str(Path(__file__).resolve().parents[1]))
-
 from seedpass.core.vault import Vault
 from seedpass.core.encryption import EncryptionManager
 from utils.key_derivation import (
@@ -12,8 +9,16 @@ from utils.key_derivation import (
     derive_key_from_password,
 )
 from utils.fingerprint import generate_fingerprint
+from nostr.backup_models import (
+    KIND_MANIFEST,
+    KIND_SNAPSHOT_CHUNK,
+    KIND_DELTA,
+)
 
-TEST_SEED = "abandon abandon abandon abandon abandon abandon abandon abandon abandon abandon abandon about"
+TEST_SEED = (
+    "abandon abandon abandon abandon abandon abandon abandon abandon abandon "
+    "abandon abandon about"
+)
 TEST_PASSWORD = "pw"
 
 
@@ -34,16 +39,6 @@ def create_vault(
     return vault, enc_mgr
 
 
-import uuid
-import asyncio
-
-from nostr.backup_models import (
-    KIND_MANIFEST,
-    KIND_SNAPSHOT_CHUNK,
-    KIND_DELTA,
-)
-
-
 class DummyEvent:
     def __init__(self, kind: int, content: str, tags=None, event_id: str | None = None):
         self.kind = kind
@@ -236,5 +231,3 @@ class DummyRelayClient:
                 return self._evs
 
         return Result(events)
-
-
diff --git a/src/tests/test_generate_test_profile_sync.py b/src/tests/test_generate_test_profile_sync.py
index 03808bd5..b8c5435e 100644
--- a/src/tests/test_generate_test_profile_sync.py
+++ b/src/tests/test_generate_test_profile_sync.py
@@ -6,7 +6,6 @@ import asyncio
 import gzip
 
 
-
 def load_script():
     script_path = (
         Path(__file__).resolve().parents[2] / "scripts" / "generate_test_profile.py"
diff --git a/src/tests/test_manifest_id_privacy.py b/src/tests/test_manifest_id_privacy.py
index 4d4efa04..35a0ba56 100644
--- a/src/tests/test_manifest_id_privacy.py
+++ b/src/tests/test_manifest_id_privacy.py
@@ -1,7 +1,6 @@
 import asyncio
 
 
-
 def test_published_events_no_fingerprint(dummy_nostr_client):
     client, relay = dummy_nostr_client
     asyncio.run(client.publish_snapshot(b"secret"))
diff --git a/src/tests/test_restore_from_nostr_setup.py b/src/tests/test_restore_from_nostr_setup.py
index 1d355feb..8df98c91 100644
--- a/src/tests/test_restore_from_nostr_setup.py
+++ b/src/tests/test_restore_from_nostr_setup.py
@@ -28,7 +28,9 @@ def _init_pm(dir_path: Path, client) -> PasswordManager:
     return pm
 
 
-def test_handle_new_seed_setup_restore_from_nostr(monkeypatch, tmp_path, capsys, make_dummy_nostr_client):
+def test_handle_new_seed_setup_restore_from_nostr(
+    monkeypatch, tmp_path, capsys, make_dummy_nostr_client
+):
     client, _relay = make_dummy_nostr_client(tmp_path / "srv")
 
     dir_a = tmp_path / "A"
@@ -133,7 +135,9 @@ async def _no_snapshot():
     return None
 
 
-def test_restore_from_nostr_warns(monkeypatch, tmp_path, capsys, make_dummy_nostr_client):
+def test_restore_from_nostr_warns(
+    monkeypatch, tmp_path, capsys, make_dummy_nostr_client
+):
     client, _relay = make_dummy_nostr_client(tmp_path / "srv")
     monkeypatch.setattr(client, "fetch_latest_snapshot", _no_snapshot)
 
@@ -150,7 +154,9 @@ def test_restore_from_nostr_warns(monkeypatch, tmp_path, capsys, make_dummy_nost
     assert "No Nostr backup" in out
 
 
-def test_restore_from_nostr_abort(monkeypatch, tmp_path, capsys, make_dummy_nostr_client):
+def test_restore_from_nostr_abort(
+    monkeypatch, tmp_path, capsys, make_dummy_nostr_client
+):
     client, _relay = make_dummy_nostr_client(tmp_path / "srv")
     monkeypatch.setattr(client, "fetch_latest_snapshot", _no_snapshot)