move common module to see if that fixes imports

sendahug · Feb 6, 2025 · 1a707c8 · 1a707c8
1 parent e4b4939
commit 1a707c8
Show file tree

Hide file tree

Showing 10 changed files with 10 additions and 10 deletions.
diff --git a/controllers/common/__init__.py b/controllers/common/__init__.py
diff --git a/controllers/common.py → controllers/common/common.py b/controllers/common.py → controllers/common/common.py
diff --git a/controllers/filters.py b/controllers/filters.py
@@ -6,7 +6,7 @@
 from auth import UserData, requires_auth
 from config.config import sah_config
 
-from .common import validator
+from .common.common import validator
 from models import Filter
 
 filters_endpoints = Blueprint("filters", __name__)

diff --git a/controllers/messages.py b/controllers/messages.py
@@ -7,7 +7,7 @@
 from auth import AuthError, UserData, requires_auth
 from config.config import sah_config
 
-from .common import (
+from .common.common import (
     DATETIME_PATTERN,
     get_current_filters,
     get_thread_id_for_users,

diff --git a/controllers/posts.py b/controllers/posts.py
@@ -7,7 +7,7 @@
 from auth import AuthError, UserData, requires_auth
 from config.config import sah_config
 
-from .common import (
+from .common.common import (
     DATETIME_PATTERN,
     get_current_filters,
     get_thread_id_for_users,

diff --git a/controllers/reports.py b/controllers/reports.py
@@ -7,7 +7,7 @@
 from auth import UserData, requires_auth
 from config.config import sah_config
 
-from .common import DATETIME_PATTERN, validator
+from .common.common import DATETIME_PATTERN, validator
 from models import Post, Report, User
 
 reports_endpoints = Blueprint("reports", __name__)

diff --git a/controllers/root.py b/controllers/root.py
@@ -6,7 +6,7 @@
 
 from config.config import sah_config
 
-from .common import validator
+from .common.common import validator
 from models import Post, User
 
 root_endpoints = Blueprint("root", __name__)

diff --git a/controllers/users.py b/controllers/users.py
@@ -7,7 +7,7 @@
 from auth import AuthError, UserData, requires_auth
 from config.config import sah_config
 
-from .common import send_push_notification, validator
+from .common.common import send_push_notification, validator
 from models import (
     BLOCKED_USER_ROLE_ID,
     Notification,

diff --git a/tests/conftest.py b/tests/conftest.py
@@ -146,7 +146,7 @@ def get_scoped_session():
         # Mock the session for all controllers
         # TODO: Surely there's a better way to do this
         controllers = listdir(path.join(path.dirname(__file__), "../controllers"))
-        non_controllers = ["__init__.py", "common.py", "__pycache__"]
+        non_controllers = ["__init__.py", "common", "__pycache__"]
         for controller in controllers:
             if controller in non_controllers:
                 continue
@@ -160,7 +160,7 @@ def get_scoped_session():
 
         await db.session.begin_nested()
         mocker.patch("pywebpush.webpush")
-        mocker.patch("controllers.common.webpush")
+        mocker.patch("controllers.common.common.webpush")
 
         yield db
 

diff --git a/tests/test_common.py b/tests/test_common.py
@@ -29,7 +29,7 @@
 from pytest_mock import MockerFixture
 from quart.typing import TestClientProtocol
 
-from controllers.common import send_email_notification
+from controllers.common.common import send_email_notification
 
 from models.db import SendADatabase
 from utils.push_notifications import RawPushData
@@ -39,7 +39,7 @@
 async def test_send_email_notification(
     app_client: TestClientProtocol, test_db: SendADatabase, mocker: MockerFixture
 ) -> None:
-    send_email_func = mocker.patch("controllers.common.send_email")
+    send_email_func = mocker.patch("controllers.common.common.send_email")
 
     async with app_client.app.app_context():
         await send_email_notification(