refactor: remove unused imports and reorganize module-level constants across multiple routers

tests/conftest.py

@@ -1,4 +1,10 @@
-import os, sys, tempfile, subprocess, time, asyncio, threading
+import os
+import sys
+import tempfile
+import subprocess
+import time
+import asyncio
+import threading
 from pathlib import Path
 import pytest
 

tests/test_admin.py

@@ -1,4 +1,3 @@
-import pytest
 from uuid import uuid4
 from datetime import datetime, timezone
 from tests.conftest import BASE_URL
@@ -65,7 +64,8 @@ def test_admin_users_loads(alice):
 
 def test_admin_users_pagination(alice):
     page, _ = alice
-    import urllib.request, urllib.parse
+    import urllib.request
+    import urllib.parse
 
     for i in range(30):
         data = urllib.parse.urlencode(

tests/test_cli.py

@@ -1,5 +1,5 @@
 import argparse
-from datetime import datetime, timezone, timedelta
+from datetime import datetime, timezone
 
 import pytest
 

tests/test_containers.py

@@ -1,10 +1,9 @@
-import asyncio
 from datetime import timedelta
 
 import pytest
 import requests
 
-from tests.conftest import BASE_URL, login_user, run_async
+from tests.conftest import BASE_URL, run_async
 from devplacepy.database import db, get_table, init_db, refresh_snapshot
 from devplacepy import config, project_files
 from devplacepy.services.containers import api, store, runtime
@@ -13,7 +12,7 @@ from devplacepy.services.containers.backend.docker_cli import build_run_argv, pa
 from devplacepy.services.containers.backend.fake import FakeBackend
 from devplacepy.services.containers.service import ContainerService
 from devplacepy.services.containers.api import INSTANCE_LABEL
-from devplacepy.services.devii.tasks.schedule import Schedule, now_utc, to_iso
+from devplacepy.services.devii.tasks.schedule import Schedule, now_utc
 
 _CONTAINER_TABLES = (
     "instances",

tests/test_landing.py

@@ -1,9 +1,5 @@
 import pytest
-from uuid import uuid4
-from datetime import datetime
 from tests.conftest import BASE_URL
-from devplacepy.database import get_table
-from devplacepy.utils import make_combined_slug
 
 
 def test_landing_page_loads(page, app_server):

tests/test_seo.py

@@ -1,4 +1,3 @@
-import pytest
 import requests
 from tests.conftest import BASE_URL
 

tests/test_utils.py

@@ -1,11 +1,19 @@
+from devplacepy.database import get_table
 from devplacepy.utils import (
-    hash_password,
-    verify_password,
-    generate_uid,
-    slugify,
-    time_ago,
-    level_for_xp,
+    award_badge,
+    award_xp,
     badge_info,
+    check_milestone_badges,
+    create_mention_notifications,
+    extract_mentions,
+    generate_uid,
+    hash_password,
+    level_for_xp,
+    safe_next,
+    slugify,
+    strip_html,
+    time_ago,
+    verify_password,
 )
 from datetime import datetime, timedelta, timezone
 
@@ -114,18 +122,6 @@ def test_badge_info_unknown_fallback():
     assert meta["description"] == "Nonexistent Badge"
 
 
-from devplacepy.utils import (
-    safe_next,
-    strip_html,
-    extract_mentions,
-    award_badge,
-    award_xp,
-    check_milestone_badges,
-    create_mention_notifications,
-)
-from devplacepy.database import get_table
-
-
 def _seed_user(role="Member", xp=0, level=1):
     uid = generate_uid()
     username = f"ut_{uid[:8]}"

tests/test_xss.py

@@ -1,4 +1,3 @@
-from tests.conftest import BASE_URL
 from tests.test_post import create_post
 
 POST_PAYLOAD = (