From 9c0f8f053ecaa2a34473dcf4b6b45c2d6812df96 Mon Sep 17 00:00:00 2001
From: Leonidas Spyropoulos <artafinde@archlinux.org>
Date: Fri, 6 May 2022 20:22:07 +0100
Subject: [PATCH] chore: rename logging.py and redis.py to avoid circular
 imports

Signed-off-by: Leonidas Spyropoulos <artafinde@archlinux.org>
---
 aurweb/asgi.py                        |  7 +++----
 aurweb/{logging.py => aur_logging.py} |  0
 aurweb/{redis.py => aur_redis.py}     |  4 ++--
 aurweb/initdb.py                      |  2 +-
 aurweb/models/user.py                 |  4 ++--
 aurweb/packages/util.py               |  2 +-
 aurweb/pkgbase/actions.py             |  4 ++--
 aurweb/prometheus.py                  |  4 ++--
 aurweb/ratelimit.py                   |  6 +++---
 aurweb/routers/accounts.py            |  4 ++--
 aurweb/routers/html.py                |  6 +++---
 aurweb/routers/packages.py            |  4 ++--
 aurweb/routers/pkgbase.py             |  4 ++--
 aurweb/routers/trusted_user.py        |  4 ++--
 aurweb/scripts/mkpkglists.py          |  4 ++--
 aurweb/scripts/notify.py              |  4 ++--
 aurweb/scripts/rendercomment.py       |  4 ++--
 aurweb/testing/alpm.py                |  4 ++--
 aurweb/testing/filelock.py            |  4 ++--
 aurweb/users/validate.py              |  4 ++--
 aurweb/util.py                        |  4 ++--
 test/conftest.py                      |  4 ++--
 test/test_accounts_routes.py          |  4 ++--
 test/test_asgi.py                     |  6 +++---
 test/test_homepage.py                 |  2 +-
 test/test_logging.py                  |  4 ++--
 test/test_packages_util.py            |  2 +-
 test/test_ratelimit.py                |  6 +++---
 test/test_redis.py                    | 24 ++++++++++++------------
 test/test_rendercomment.py            |  4 ++--
 test/test_rpc.py                      |  2 +-
 test/test_rss.py                      |  4 ++--
 32 files changed, 72 insertions(+), 73 deletions(-)
 rename aurweb/{logging.py => aur_logging.py} (100%)
 rename aurweb/{redis.py => aur_redis.py} (95%)

diff --git a/aurweb/asgi.py b/aurweb/asgi.py
index 72b47b4c..b172626f 100644
--- a/aurweb/asgi.py
+++ b/aurweb/asgi.py
@@ -22,19 +22,18 @@ from starlette.middleware.sessions import SessionMiddleware
 import aurweb.captcha  # noqa: F401
 import aurweb.config
 import aurweb.filters  # noqa: F401
-import aurweb.logging
 import aurweb.pkgbase.util as pkgbaseutil
-from aurweb import logging, prometheus, util
+from aurweb import aur_logging, prometheus, util
+from aurweb.aur_redis import redis_connection
 from aurweb.auth import BasicAuthBackend
 from aurweb.db import get_engine, query
 from aurweb.models import AcceptedTerm, Term
 from aurweb.packages.util import get_pkg_or_base
 from aurweb.prometheus import instrumentator
-from aurweb.redis import redis_connection
 from aurweb.routers import APP_ROUTES
 from aurweb.templates import make_context, render_template
 
-logger = logging.get_logger(__name__)
+logger = aur_logging.get_logger(__name__)
 
 # Setup the FastAPI app.
 app = FastAPI()
diff --git a/aurweb/logging.py b/aurweb/aur_logging.py
similarity index 100%
rename from aurweb/logging.py
rename to aurweb/aur_logging.py
diff --git a/aurweb/redis.py b/aurweb/aur_redis.py
similarity index 95%
rename from aurweb/redis.py
rename to aurweb/aur_redis.py
index af179b9b..ec66df19 100644
--- a/aurweb/redis.py
+++ b/aurweb/aur_redis.py
@@ -2,9 +2,9 @@ import fakeredis
 from redis import ConnectionPool, Redis
 
 import aurweb.config
-from aurweb import logging
+from aurweb import aur_logging
 
-logger = logging.get_logger(__name__)
+logger = aur_logging.get_logger(__name__)
 pool = None
 
 
diff --git a/aurweb/initdb.py b/aurweb/initdb.py
index ded4330d..ee59212c 100644
--- a/aurweb/initdb.py
+++ b/aurweb/initdb.py
@@ -3,8 +3,8 @@ import argparse
 import alembic.command
 import alembic.config
 
+import aurweb.aur_logging
 import aurweb.db
-import aurweb.logging
 import aurweb.schema
 
 
diff --git a/aurweb/models/user.py b/aurweb/models/user.py
index 0d638677..9846d996 100644
--- a/aurweb/models/user.py
+++ b/aurweb/models/user.py
@@ -10,12 +10,12 @@ from sqlalchemy.orm import backref, relationship
 import aurweb.config
 import aurweb.models.account_type
 import aurweb.schema
-from aurweb import db, logging, schema, time, util
+from aurweb import aur_logging, db, schema, time, util
 from aurweb.models.account_type import AccountType as _AccountType
 from aurweb.models.ban import is_banned
 from aurweb.models.declarative import Base
 
-logger = logging.get_logger(__name__)
+logger = aur_logging.get_logger(__name__)
 
 SALT_ROUNDS_DEFAULT = 12
 
diff --git a/aurweb/packages/util.py b/aurweb/packages/util.py
index b6ba7e20..cddec0ac 100644
--- a/aurweb/packages/util.py
+++ b/aurweb/packages/util.py
@@ -7,11 +7,11 @@ from fastapi import HTTPException
 from sqlalchemy import orm
 
 from aurweb import config, db, models
+from aurweb.aur_redis import redis_connection
 from aurweb.models import Package
 from aurweb.models.official_provider import OFFICIAL_BASE, OfficialProvider
 from aurweb.models.package_dependency import PackageDependency
 from aurweb.models.package_relation import PackageRelation
-from aurweb.redis import redis_connection
 from aurweb.templates import register_filter
 
 Providers = list[Union[PackageRelation, OfficialProvider]]
diff --git a/aurweb/pkgbase/actions.py b/aurweb/pkgbase/actions.py
index a453cb36..56ba738d 100644
--- a/aurweb/pkgbase/actions.py
+++ b/aurweb/pkgbase/actions.py
@@ -1,6 +1,6 @@
 from fastapi import Request
 
-from aurweb import db, logging, util
+from aurweb import aur_logging, db, util
 from aurweb.auth import creds
 from aurweb.models import PackageBase, User
 from aurweb.models.package_comaintainer import PackageComaintainer
@@ -10,7 +10,7 @@ from aurweb.packages.requests import handle_request, update_closure_comment
 from aurweb.pkgbase import util as pkgbaseutil
 from aurweb.scripts import notify, popupdate
 
-logger = logging.get_logger(__name__)
+logger = aur_logging.get_logger(__name__)
 
 
 @db.retry_deadlock
diff --git a/aurweb/prometheus.py b/aurweb/prometheus.py
index 0bbea4be..b8b7984f 100644
--- a/aurweb/prometheus.py
+++ b/aurweb/prometheus.py
@@ -5,9 +5,9 @@ from prometheus_fastapi_instrumentator import Instrumentator
 from prometheus_fastapi_instrumentator.metrics import Info
 from starlette.routing import Match, Route
 
-from aurweb import logging
+from aurweb import aur_logging
 
-logger = logging.get_logger(__name__)
+logger = aur_logging.get_logger(__name__)
 _instrumentator = Instrumentator()
 
 
diff --git a/aurweb/ratelimit.py b/aurweb/ratelimit.py
index 97923a52..ea191972 100644
--- a/aurweb/ratelimit.py
+++ b/aurweb/ratelimit.py
@@ -1,11 +1,11 @@
 from fastapi import Request
 from redis.client import Pipeline
 
-from aurweb import config, db, logging, time
+from aurweb import aur_logging, config, db, time
+from aurweb.aur_redis import redis_connection
 from aurweb.models import ApiRateLimit
-from aurweb.redis import redis_connection
 
-logger = logging.get_logger(__name__)
+logger = aur_logging.get_logger(__name__)
 
 
 def _update_ratelimit_redis(request: Request, pipeline: Pipeline):
diff --git a/aurweb/routers/accounts.py b/aurweb/routers/accounts.py
index 12e59b30..24aacdf7 100644
--- a/aurweb/routers/accounts.py
+++ b/aurweb/routers/accounts.py
@@ -8,7 +8,7 @@ from fastapi.responses import HTMLResponse, RedirectResponse
 from sqlalchemy import and_, or_
 
 import aurweb.config
-from aurweb import cookies, db, l10n, logging, models, util
+from aurweb import aur_logging, cookies, db, l10n, models, util
 from aurweb.auth import account_type_required, creds, requires_auth, requires_guest
 from aurweb.captcha import get_captcha_salts
 from aurweb.exceptions import ValidationError, handle_form_exceptions
@@ -22,7 +22,7 @@ from aurweb.users import update, validate
 from aurweb.users.util import get_user_by_name
 
 router = APIRouter()
-logger = logging.get_logger(__name__)
+logger = aur_logging.get_logger(__name__)
 
 
 @router.get("/passreset", response_class=HTMLResponse)
diff --git a/aurweb/routers/html.py b/aurweb/routers/html.py
index da1ffd55..f5e6657f 100644
--- a/aurweb/routers/html.py
+++ b/aurweb/routers/html.py
@@ -16,7 +16,7 @@ from sqlalchemy import and_, case, or_
 
 import aurweb.config
 import aurweb.models.package_request
-from aurweb import cookies, db, logging, models, time, util
+from aurweb import aur_logging, cookies, db, models, time, util
 from aurweb.cache import db_count_cache
 from aurweb.exceptions import handle_form_exceptions
 from aurweb.models.account_type import TRUSTED_USER_AND_DEV_ID, TRUSTED_USER_ID
@@ -24,7 +24,7 @@ from aurweb.models.package_request import PENDING_ID
 from aurweb.packages.util import query_notified, query_voted, updated_packages
 from aurweb.templates import make_context, render_template
 
-logger = logging.get_logger(__name__)
+logger = aur_logging.get_logger(__name__)
 router = APIRouter()
 
 
@@ -80,7 +80,7 @@ async def index(request: Request):
 
     bases = db.query(models.PackageBase)
 
-    redis = aurweb.redis.redis_connection()
+    redis = aurweb.aur_redis.redis_connection()
     cache_expire = 300  # Five minutes.
 
     # Package statistics.
diff --git a/aurweb/routers/packages.py b/aurweb/routers/packages.py
index 55d2abf5..0d482521 100644
--- a/aurweb/routers/packages.py
+++ b/aurweb/routers/packages.py
@@ -5,7 +5,7 @@ from typing import Any
 from fastapi import APIRouter, Form, Query, Request, Response
 
 import aurweb.filters  # noqa: F401
-from aurweb import config, db, defaults, logging, models, util
+from aurweb import aur_logging, config, db, defaults, models, util
 from aurweb.auth import creds, requires_auth
 from aurweb.exceptions import InvariantError, handle_form_exceptions
 from aurweb.models.relation_type import CONFLICTS_ID, PROVIDES_ID, REPLACES_ID
@@ -15,7 +15,7 @@ from aurweb.packages.util import get_pkg_or_base
 from aurweb.pkgbase import actions as pkgbase_actions, util as pkgbaseutil
 from aurweb.templates import make_context, make_variable_context, render_template
 
-logger = logging.get_logger(__name__)
+logger = aur_logging.get_logger(__name__)
 router = APIRouter()
 
 
diff --git a/aurweb/routers/pkgbase.py b/aurweb/routers/pkgbase.py
index 3b1ab688..9dab76f8 100644
--- a/aurweb/routers/pkgbase.py
+++ b/aurweb/routers/pkgbase.py
@@ -4,7 +4,7 @@ from fastapi import APIRouter, Form, HTTPException, Query, Request, Response
 from fastapi.responses import JSONResponse, RedirectResponse
 from sqlalchemy import and_
 
-from aurweb import config, db, l10n, logging, templates, time, util
+from aurweb import aur_logging, config, db, l10n, templates, time, util
 from aurweb.auth import creds, requires_auth
 from aurweb.exceptions import InvariantError, ValidationError, handle_form_exceptions
 from aurweb.models import PackageBase
@@ -21,7 +21,7 @@ from aurweb.scripts import notify, popupdate
 from aurweb.scripts.rendercomment import update_comment_render_fastapi
 from aurweb.templates import make_variable_context, render_template
 
-logger = logging.get_logger(__name__)
+logger = aur_logging.get_logger(__name__)
 router = APIRouter()
 
 
diff --git a/aurweb/routers/trusted_user.py b/aurweb/routers/trusted_user.py
index 37edb072..4248347d 100644
--- a/aurweb/routers/trusted_user.py
+++ b/aurweb/routers/trusted_user.py
@@ -7,7 +7,7 @@ from fastapi import APIRouter, Form, HTTPException, Request
 from fastapi.responses import RedirectResponse, Response
 from sqlalchemy import and_, func, or_
 
-from aurweb import db, l10n, logging, models, time
+from aurweb import aur_logging, db, l10n, models, time
 from aurweb.auth import creds, requires_auth
 from aurweb.exceptions import handle_form_exceptions
 from aurweb.models import User
@@ -15,7 +15,7 @@ from aurweb.models.account_type import TRUSTED_USER_AND_DEV_ID, TRUSTED_USER_ID
 from aurweb.templates import make_context, make_variable_context, render_template
 
 router = APIRouter()
-logger = logging.get_logger(__name__)
+logger = aur_logging.get_logger(__name__)
 
 # Some TU route specific constants.
 ITEMS_PER_PAGE = 10  # Paged table size.
diff --git a/aurweb/scripts/mkpkglists.py b/aurweb/scripts/mkpkglists.py
index bfdd12b4..e74bbf25 100755
--- a/aurweb/scripts/mkpkglists.py
+++ b/aurweb/scripts/mkpkglists.py
@@ -32,11 +32,11 @@ import orjson
 from sqlalchemy import literal, orm
 
 import aurweb.config
-from aurweb import db, filters, logging, models, util
+from aurweb import aur_logging, db, filters, models, util
 from aurweb.benchmark import Benchmark
 from aurweb.models import Package, PackageBase, User
 
-logger = logging.get_logger("aurweb.scripts.mkpkglists")
+logger = aur_logging.get_logger("aurweb.scripts.mkpkglists")
 
 
 TYPE_MAP = {
diff --git a/aurweb/scripts/notify.py b/aurweb/scripts/notify.py
index f19438bb..93108cd3 100755
--- a/aurweb/scripts/notify.py
+++ b/aurweb/scripts/notify.py
@@ -13,7 +13,7 @@ import aurweb.config
 import aurweb.db
 import aurweb.filters
 import aurweb.l10n
-from aurweb import db, logging
+from aurweb import aur_logging, db
 from aurweb.models import PackageBase, User
 from aurweb.models.package_comaintainer import PackageComaintainer
 from aurweb.models.package_comment import PackageComment
@@ -22,7 +22,7 @@ from aurweb.models.package_request import PackageRequest
 from aurweb.models.request_type import RequestType
 from aurweb.models.tu_vote import TUVote
 
-logger = logging.get_logger(__name__)
+logger = aur_logging.get_logger(__name__)
 
 aur_location = aurweb.config.get("options", "aur_location")
 
diff --git a/aurweb/scripts/rendercomment.py b/aurweb/scripts/rendercomment.py
index ff6fe09c..4a2c84bd 100755
--- a/aurweb/scripts/rendercomment.py
+++ b/aurweb/scripts/rendercomment.py
@@ -9,10 +9,10 @@ import markdown
 import pygit2
 
 import aurweb.config
-from aurweb import db, logging, util
+from aurweb import aur_logging, db, util
 from aurweb.models import PackageComment
 
-logger = logging.get_logger(__name__)
+logger = aur_logging.get_logger(__name__)
 
 
 class LinkifyExtension(markdown.extensions.Extension):
diff --git a/aurweb/testing/alpm.py b/aurweb/testing/alpm.py
index ddafb710..61a9315f 100644
--- a/aurweb/testing/alpm.py
+++ b/aurweb/testing/alpm.py
@@ -4,10 +4,10 @@ import re
 import shutil
 import subprocess
 
-from aurweb import logging, util
+from aurweb import aur_logging, util
 from aurweb.templates import base_template
 
-logger = logging.get_logger(__name__)
+logger = aur_logging.get_logger(__name__)
 
 
 class AlpmDatabase:
diff --git a/aurweb/testing/filelock.py b/aurweb/testing/filelock.py
index 33b42cb3..d582f0bf 100644
--- a/aurweb/testing/filelock.py
+++ b/aurweb/testing/filelock.py
@@ -4,9 +4,9 @@ from typing import Callable
 
 from posix_ipc import O_CREAT, Semaphore
 
-from aurweb import logging
+from aurweb import aur_logging
 
-logger = logging.get_logger(__name__)
+logger = aur_logging.get_logger(__name__)
 
 
 def default_on_create(path):
diff --git a/aurweb/users/validate.py b/aurweb/users/validate.py
index 6c27a0b7..8fc68864 100644
--- a/aurweb/users/validate.py
+++ b/aurweb/users/validate.py
@@ -9,7 +9,7 @@ when encountering invalid criteria and return silently otherwise.
 from fastapi import Request
 from sqlalchemy import and_
 
-from aurweb import config, db, l10n, logging, models, time, util
+from aurweb import aur_logging, config, db, l10n, models, time, util
 from aurweb.auth import creds
 from aurweb.captcha import get_captcha_answer, get_captcha_salts, get_captcha_token
 from aurweb.exceptions import ValidationError
@@ -17,7 +17,7 @@ from aurweb.models.account_type import ACCOUNT_TYPE_NAME
 from aurweb.models.ssh_pub_key import get_fingerprint
 from aurweb.util import strtobool
 
-logger = logging.get_logger(__name__)
+logger = aur_logging.get_logger(__name__)
 
 
 def invalid_fields(E: str = str(), U: str = str(), **kwargs) -> None:
diff --git a/aurweb/util.py b/aurweb/util.py
index 432b818a..cda12af1 100644
--- a/aurweb/util.py
+++ b/aurweb/util.py
@@ -15,9 +15,9 @@ from email_validator import EmailSyntaxError, validate_email
 from fastapi.responses import JSONResponse
 
 import aurweb.config
-from aurweb import defaults, logging
+from aurweb import aur_logging, defaults
 
-logger = logging.get_logger(__name__)
+logger = aur_logging.get_logger(__name__)
 
 
 def make_random_string(length: int) -> str:
diff --git a/test/conftest.py b/test/conftest.py
index aac221f7..15a982aa 100644
--- a/test/conftest.py
+++ b/test/conftest.py
@@ -52,12 +52,12 @@ from sqlalchemy.orm import scoped_session
 
 import aurweb.config
 import aurweb.db
-from aurweb import initdb, logging, testing
+from aurweb import aur_logging, initdb, testing
 from aurweb.testing.email import Email
 from aurweb.testing.filelock import FileLock
 from aurweb.testing.git import GitRepository
 
-logger = logging.get_logger(__name__)
+logger = aur_logging.get_logger(__name__)
 
 # Synchronization lock for database setup.
 setup_lock = Lock()
diff --git a/test/test_accounts_routes.py b/test/test_accounts_routes.py
index f4034a9a..33baa0ea 100644
--- a/test/test_accounts_routes.py
+++ b/test/test_accounts_routes.py
@@ -11,7 +11,7 @@ from fastapi.testclient import TestClient
 
 import aurweb.config
 import aurweb.models.account_type as at
-from aurweb import captcha, db, logging, time
+from aurweb import aur_logging, captcha, db, time
 from aurweb.asgi import app
 from aurweb.db import create, query
 from aurweb.models.accepted_term import AcceptedTerm
@@ -31,7 +31,7 @@ from aurweb.models.user import User
 from aurweb.testing.html import get_errors
 from aurweb.testing.requests import Request
 
-logger = logging.get_logger(__name__)
+logger = aur_logging.get_logger(__name__)
 
 # Some test global constants.
 TEST_USERNAME = "test"
diff --git a/test/test_asgi.py b/test/test_asgi.py
index 6ff80fa3..3b794c76 100644
--- a/test/test_asgi.py
+++ b/test/test_asgi.py
@@ -10,17 +10,17 @@ from fastapi import HTTPException
 from fastapi.testclient import TestClient
 
 import aurweb.asgi
+import aurweb.aur_redis
 import aurweb.config
-import aurweb.redis
 from aurweb.exceptions import handle_form_exceptions
 from aurweb.testing.requests import Request
 
 
 @pytest.fixture
 def setup(db_test, email_test):
-    aurweb.redis.redis_connection().flushall()
+    aurweb.aur_redis.redis_connection().flushall()
     yield
-    aurweb.redis.redis_connection().flushall()
+    aurweb.aur_redis.redis_connection().flushall()
 
 
 @pytest.fixture
diff --git a/test/test_homepage.py b/test/test_homepage.py
index 5490a244..521f71c4 100644
--- a/test/test_homepage.py
+++ b/test/test_homepage.py
@@ -7,6 +7,7 @@ from fastapi.testclient import TestClient
 
 from aurweb import db, time
 from aurweb.asgi import app
+from aurweb.aur_redis import redis_connection
 from aurweb.models.account_type import USER_ID
 from aurweb.models.package import Package
 from aurweb.models.package_base import PackageBase
@@ -14,7 +15,6 @@ from aurweb.models.package_comaintainer import PackageComaintainer
 from aurweb.models.package_request import PackageRequest
 from aurweb.models.request_type import DELETION_ID, RequestType
 from aurweb.models.user import User
-from aurweb.redis import redis_connection
 from aurweb.testing.html import parse_root
 from aurweb.testing.requests import Request
 
diff --git a/test/test_logging.py b/test/test_logging.py
index 63092d07..90d13c93 100644
--- a/test/test_logging.py
+++ b/test/test_logging.py
@@ -1,6 +1,6 @@
-from aurweb import logging
+from aurweb import aur_logging
 
-logger = logging.get_logger(__name__)
+logger = aur_logging.get_logger(__name__)
 
 
 def test_logging(caplog):
diff --git a/test/test_packages_util.py b/test/test_packages_util.py
index 0042cd71..a5273b68 100644
--- a/test/test_packages_util.py
+++ b/test/test_packages_util.py
@@ -2,6 +2,7 @@ import pytest
 from fastapi.testclient import TestClient
 
 from aurweb import asgi, config, db, time
+from aurweb.aur_redis import kill_redis
 from aurweb.models.account_type import USER_ID
 from aurweb.models.official_provider import OFFICIAL_BASE, OfficialProvider
 from aurweb.models.package import Package
@@ -11,7 +12,6 @@ from aurweb.models.package_source import PackageSource
 from aurweb.models.package_vote import PackageVote
 from aurweb.models.user import User
 from aurweb.packages import util
-from aurweb.redis import kill_redis
 
 
 @pytest.fixture(autouse=True)
diff --git a/test/test_ratelimit.py b/test/test_ratelimit.py
index 20528847..b7cd7e7d 100644
--- a/test/test_ratelimit.py
+++ b/test/test_ratelimit.py
@@ -3,13 +3,13 @@ from unittest import mock
 import pytest
 from redis.client import Pipeline
 
-from aurweb import config, db, logging
+from aurweb import aur_logging, config, db
+from aurweb.aur_redis import redis_connection
 from aurweb.models import ApiRateLimit
 from aurweb.ratelimit import check_ratelimit
-from aurweb.redis import redis_connection
 from aurweb.testing.requests import Request
 
-logger = logging.get_logger(__name__)
+logger = aur_logging.get_logger(__name__)
 
 
 @pytest.fixture(autouse=True)
diff --git a/test/test_redis.py b/test/test_redis.py
index a66cd204..6f9bdb40 100644
--- a/test/test_redis.py
+++ b/test/test_redis.py
@@ -3,11 +3,11 @@ from unittest import mock
 import pytest
 
 import aurweb.config
-from aurweb.redis import redis_connection
+from aurweb.aur_redis import redis_connection
 
 
 @pytest.fixture
-def rediss():
+def redis():
     """Create a RedisStub."""
 
     def mock_get(section, key):
@@ -21,20 +21,20 @@ def rediss():
     yield redis
 
 
-def test_redis_stub(rediss):
+def test_redis_stub(redis):
     # We don't yet have a test key set.
-    assert rediss.get("test") is None
+    assert redis.get("test") is None
 
     # Set the test key to abc.
-    rediss.set("test", "abc")
-    assert rediss.get("test").decode() == "abc"
+    redis.set("test", "abc")
+    assert redis.get("test").decode() == "abc"
 
     # Test expire.
-    rediss.expire("test", 0)
-    assert rediss.get("test") is None
+    redis.expire("test", 0)
+    assert redis.get("test") is None
 
     # Now, set the test key again and use delete() on it.
-    rediss.set("test", "abc")
-    assert rediss.get("test").decode() == "abc"
-    rediss.delete("test")
-    assert rediss.get("test") is None
+    redis.set("test", "abc")
+    assert redis.get("test").decode() == "abc"
+    redis.delete("test")
+    assert redis.get("test") is None
diff --git a/test/test_rendercomment.py b/test/test_rendercomment.py
index 5b7ff5ac..59eb7191 100644
--- a/test/test_rendercomment.py
+++ b/test/test_rendercomment.py
@@ -2,14 +2,14 @@ from unittest import mock
 
 import pytest
 
-from aurweb import config, db, logging, time
+from aurweb import aur_logging, config, db, time
 from aurweb.models import Package, PackageBase, PackageComment, User
 from aurweb.models.account_type import USER_ID
 from aurweb.scripts import rendercomment
 from aurweb.scripts.rendercomment import update_comment_render
 from aurweb.testing.git import GitRepository
 
-logger = logging.get_logger(__name__)
+logger = aur_logging.get_logger(__name__)
 aur_location = config.get("options", "aur_location")
 
 
diff --git a/test/test_rpc.py b/test/test_rpc.py
index 84ddd8d7..f417d379 100644
--- a/test/test_rpc.py
+++ b/test/test_rpc.py
@@ -10,6 +10,7 @@ from redis.client import Pipeline
 import aurweb.models.dependency_type as dt
 import aurweb.models.relation_type as rt
 from aurweb import asgi, config, db, rpc, scripts, time
+from aurweb.aur_redis import redis_connection
 from aurweb.models.account_type import USER_ID
 from aurweb.models.dependency_type import DEPENDS_ID
 from aurweb.models.license import License
@@ -22,7 +23,6 @@ from aurweb.models.package_relation import PackageRelation
 from aurweb.models.package_vote import PackageVote
 from aurweb.models.relation_type import PROVIDES_ID
 from aurweb.models.user import User
-from aurweb.redis import redis_connection
 
 
 @pytest.fixture
diff --git a/test/test_rss.py b/test/test_rss.py
index 8526caa1..d227a183 100644
--- a/test/test_rss.py
+++ b/test/test_rss.py
@@ -4,14 +4,14 @@ import lxml.etree
 import pytest
 from fastapi.testclient import TestClient
 
-from aurweb import db, logging, time
+from aurweb import aur_logging, db, time
 from aurweb.asgi import app
 from aurweb.models.account_type import AccountType
 from aurweb.models.package import Package
 from aurweb.models.package_base import PackageBase
 from aurweb.models.user import User
 
-logger = logging.get_logger(__name__)
+logger = aur_logging.get_logger(__name__)
 
 
 @pytest.fixture(autouse=True)