fix failing tests

Author: eadwinCode

Makefile

@@ -5,9 +5,9 @@ help:
 	@fgrep -h "##" $(MAKEFILE_LIST) | fgrep -v fgrep | sort | awk 'BEGIN {FS = ":.*?## "}; {printf "\033[36m%-30s\033[0m %s\n", $$1, $$2}'
 
 clean: ## Removing cached python compiled files
-	find . -name \*pyc  | xargs  rm -fv
-	find . -name \*pyo | xargs  rm -fv
-	find . -name \*~  | xargs  rm -fv
+	find . -name "*.pyc" -type f -delete
+	find . -name "*.pyo" -type f -delete
+	find . -name "*~" -type f -delete
 	find . -name __pycache__  | xargs  rm -rfv
 	find . -name .pytest_cache  | xargs  rm -rfv
 	find . -name .ruff_cache  | xargs  rm -rfv

ellar_sql/factory/base.py

@@ -19,6 +19,10 @@
 
 
 class EllarSQLOptions(SQLAlchemyOptions):
+    # Type hints for SQLAlchemy-specific attributes
+    sqlalchemy_get_or_create: t.Tuple[str, ...]
+    sqlalchemy_session_persistence: str
+
     @staticmethod
     def _check_has_sqlalchemy_session_set(meta, value):
         if value and hasattr(meta, "sqlalchemy_session"):
@@ -31,6 +35,7 @@ class EllarSQLFactory(SQLAlchemyModelFactory):
     """Factory for EllarSQL models."""
 
     _options_class = EllarSQLOptions
+    _meta: EllarSQLOptions
 
     class Meta:
         abstract = True

tests/test_migrations/test_migrations_commands.py

@@ -5,10 +5,7 @@
 def test_migrate_upgrade():
     result = run_command("default.py db init")
     assert result.returncode == 0
-    assert (
-        b"tests/dumbs/default/migrations/alembic.ini' before proceeding."
-        in result.stdout
-    )
+    assert b"default/migrations/alembic.ini before proceeding." in result.stdout
 
     result = run_command("default.py db check")
     assert result.returncode == 1
@@ -36,7 +33,7 @@ def test_migrate_upgrade_custom_directory():
     result = run_command("custom_directory.py db init")
     assert result.returncode == 0
     assert (
-        b"tests/dumbs/custom_directory/temp_migrations/alembic.ini' before proceeding."
+        b"custom_directory/temp_migrations/alembic.ini before proceeding."
         in result.stdout
     )
 
@@ -138,10 +135,7 @@ def test_other_alembic_commands():
 def test_migrate_upgrade_async():
     result = run_command("default_async.py db init")
     assert result.returncode == 0
-    assert (
-        b"tests/dumbs/default_async/migrations/alembic.ini' before proceeding."
-        in result.stdout
-    )
+    assert b"default_async/migrations/alembic.ini before proceeding." in result.stdout
 
     result = run_command("default_async.py db check")
     assert result.returncode == 1

tests/test_migrations/test_multiple_database.py

@@ -6,10 +6,7 @@ def test_migrate_upgrade_for_multiple_database():
     with set_env_variable("multiple_db", "true"):
         result = run_command("multiple_database.py db init -m")
         assert result.returncode == 0
-        assert (
-            b"tests/dumbs/multiple/migrations/alembic.ini' before proceeding."
-            in result.stdout
-        )
+        assert b"multiple/migrations/alembic.ini before proceeding." in result.stdout
 
         result = run_command("multiple_database.py db check")
         assert result.returncode == 1
@@ -62,8 +59,7 @@ def test_migrate_upgrade_for_multiple_database_async():
         result = run_command("multiple_database_async.py db init -m")
         assert result.returncode == 0
         assert (
-            b"tests/dumbs/multiple_async/migrations/alembic.ini' before proceeding."
-            in result.stdout
+            b"multiple_async/migrations/alembic.ini before proceeding." in result.stdout
         )
 
         result = run_command("multiple_database_async.py db check")