[FIX] fix testcase, add DB type, delete double check(profile, import)

Author: bnbong

backend/migrations/versions/001_initial_schema.py

@@ -93,6 +93,7 @@ def downgrade() -> None:
     op.drop_table("profiles")
     op.drop_index("ix_users_email", "users")
     op.drop_table("users")
-    # Drop enum types (PostgreSQL only; SQLite ignores)
-    op.execute("DROP TYPE IF EXISTS accounttype")
-    op.execute("DROP TYPE IF EXISTS userstatus")
+    # Drop enum types (PostgreSQL only; SQLite has no user-defined types).
+    if op.get_bind().dialect.name == "postgresql":
+        op.execute("DROP TYPE IF EXISTS accounttype")
+        op.execute("DROP TYPE IF EXISTS userstatus")

backend/migrations/versions/002_add_reports.py

@@ -75,6 +75,7 @@ def downgrade() -> None:
     op.drop_index("ix_reports_reported_user_id", "reports")
     op.drop_index("ix_reports_reporter_user_id", "reports")
     op.drop_table("reports")
-    # Drop enum types (PostgreSQL only; SQLite ignores)
-    op.execute("DROP TYPE IF EXISTS reportreasoncode")
-    op.execute("DROP TYPE IF EXISTS reportstatus")
+    # Drop enum types (PostgreSQL only; SQLite has no user-defined types).
+    if op.get_bind().dialect.name == "postgresql":
+        op.execute("DROP TYPE IF EXISTS reportreasoncode")
+        op.execute("DROP TYPE IF EXISTS reportstatus")

src/fall_in/core/game_manager.py

@@ -278,6 +278,7 @@ def bootstrap_authenticated_account(self, sync_local_progress: bool) -> dict:
 
         from fall_in.net.backend_api import get_json, post_json
 
+        profile: dict = {}
         if self.has_registered_session() and sync_local_progress:
             merged = post_json(
                 "/me/progress/merge",
@@ -300,7 +301,8 @@ def bootstrap_authenticated_account(self, sync_local_progress: bool) -> dict:
                 ],
             )
 
-        profile = get_json("/me/profile", self.access_token)
+        if not profile:
+            profile = get_json("/me/profile", self.access_token)
         self.user_id = profile.get("user_id")
         self.nickname = profile.get("nickname", self.nickname)
         self.account_type = profile.get("account_type", self.account_type)

src/fall_in/net/messages.py

@@ -42,6 +42,7 @@ class ClientMessageType(str, Enum):
 
     # Connection management
     PING = "PING"
+    PONG = "PONG"
     RECONNECT = "RECONNECT"
 
 

src/fall_in/scenes/title_scene.py

@@ -267,7 +267,6 @@ def _open_debug_menu(self) -> None:
         if not DEBUG_MODE or GameManager().has_auth_session():
             return
 
-        from fall_in.core.game_manager import GameManager
         from fall_in.scenes.title_debug_scene import DebugScene
 
         GameManager().change_scene(DebugScene())

tests/multiplayer/test_game_scene_multiplayer.py

@@ -303,10 +303,11 @@ def test_remote_match_result_transitions_to_game_over_scene(monkeypatch):
     import fall_in.scenes.game_over_scene as game_over_module
 
     class _FakeGameOverScene:
-        def __init__(self, winner, players, round_number):
+        def __init__(self, winner, players, round_number, multiplayer_reward=None):
             self.winner = winner
             self.players = players
             self.round_number = round_number
+            self.multiplayer_reward = multiplayer_reward
 
     monkeypatch.setattr(game_over_module, "GameOverScene", _FakeGameOverScene)
     monkeypatch.setattr(