Bump version to 0.2.21 and improve test coverage

Updated README to reflect version 0.2.21 and clarified remote storage functionality. Incremented versions in `__init__.py` and `game_storage.py`.

Enhanced `test_download_game_settings.py`:
- Added Hugging Face API token import and environment setup.
- Improved test for resolving short IDs with detailed success/failure output.
- Updated assertions for new fields in `settings.json`.
- Added `__main__` block for direct test execution with `pytest`.

README.md

@@ -121,7 +121,10 @@ docker run -p8501:8501 battlewords
  - Persistent Storage: all game results saved locally for personal statistics without accounts.
  - Challenge Mode: remote storage of challenge results, multi-user leaderboard, and shareable links.
 
--0.2.20 (development)
+-0.2.21
+ - fix tests 
+ 
+-0.2.20 
  - Remote Storage game_id:
    - Per-game JSON settings uploaded to a storage server (Hugging Face repo) under unique `games/{uid}/settings.json`
    - A shortened URL id (sid) is generated; shareable link: `?game_id=<sid>`

battlewords/__init__.py

@@ -1,2 +1,2 @@
-__version__ = "0.2.20"
+__version__ = "0.2.21"
 __all__ = ["models", "generator", "logic", "ui", "game_storage"]

battlewords/game_storage.py

@@ -5,7 +5,7 @@ BattleWords-specific storage wrapper for HuggingFace storage operations.
 This module provides high-level functions for saving and loading BattleWords games
 using the shared storage module from battlewords.modules.
 """
-__version__ = "0.1.1"
+__version__ = "0.1.2"
 
 import json
 import tempfile

battlewords/ui.py

@@ -255,6 +255,10 @@ def inject_styles() -> None:
 
           /* Final score style */
           .bw-final-score { color: #1ca41c !important; font-weight: 800; }
+          .stExpander {z-index: 10;}
+          div[data-testid="stToastContainer"], div[data-testid="stToast"] {
+            margin: 0 auto;
+          }
 
           /* Make grid buttons square and fill their column */
           div[data-testid="stButton"]{
@@ -490,75 +494,6 @@ def _sync_back(state: GameState) -> None:
 def _render_header():
     st.title(f"Battlewords v{version}")
 
-    # Show Challenge Mode banner if loading a shared game
-    shared_settings = st.session_state.get("shared_game_settings")
-    if shared_settings:
-        users = shared_settings.get("users", [])
-
-        if users:
-            # Sort users by score (descending), then by time (ascending)
-            sorted_users = sorted(users, key=lambda u: (-u["score"], u["time"]))
-            best_user = sorted_users[0]
-            best_score = best_user["score"]
-            best_time = best_user["time"]
-            mins, secs = divmod(best_time, 60)
-            best_time_str = f"{mins:02d}:{secs:02d}"
-
-            # Build leaderboard HTML
-            leaderboard_rows = []
-            for i, user in enumerate(sorted_users[:5], 1):  # Top 5
-                u_mins, u_secs = divmod(user["time"], 60)
-                u_time_str = f"{u_mins:02d}:{u_secs:02d}"
-                medal = ["🥇", "🥈", "🥉"][i-1] if i <= 3 else f"{i}."
-                leaderboard_rows.append(
-                    f"<div style='padding:0.25rem; font-size:0.85rem;'>{medal} {user['username']}: {user['score']} pts in {u_time_str}</div>"
-                )
-            leaderboard_html = "".join(leaderboard_rows)
-
-            st.markdown(
-                f"""
-                <div style="
-                    background: linear-gradient(90deg, #1d64c8 0%, #165ba8 100%);
-                    color: white;
-                    padding: 1rem;
-                    border-radius: 0.5rem;
-                    margin-bottom: 1rem;
-                    text-align: center;
-                    box-shadow: 0 4px 6px rgba(0, 0, 0, 0.1);
-                ">
-                    🎯 <strong>CHALLENGE MODE</strong> 🎯<br/>
-                    <span style="font-size: 0.9rem;">
-                        Beat the best: <strong>{best_score} points</strong> in <strong>{best_time_str}</strong> by <strong>{best_user['username']}</strong>
-                    </span>
-                    <div style="margin-top:0.75rem; border-top: 1px solid rgba(255,255,255,0.3); padding-top:0.5rem;">
-                        <strong style="font-size:0.9rem;">🏆 Leaderboard</strong>
-                        {leaderboard_html}
-                    </div>
-                </div>
-                """,
-                unsafe_allow_html=True
-            )
-        else:
-            st.markdown(
-                """
-                <div style="
-                    background: linear-gradient(90deg, #1d64c8 0%, #165ba8 100%);
-                    color: white;
-                    padding: 1rem;
-                    border-radius: 0.5rem;
-                    margin-bottom: 1rem;
-                    text-align: center;
-                    box-shadow: 0 4px 6px rgba(0, 0, 0, 0.1);
-                ">
-                    🎯 <strong>CHALLENGE MODE</strong> 🎯<br/>
-                    <span style="font-size: 0.9rem;">
-                        Be the first to complete this challenge!
-                    </span>
-                </div>
-                """,
-                unsafe_allow_html=True
-            )
-
     st.subheader("Reveal letters in cells, then guess the words!")
     # st.markdown(
     #     "- Grid is 12×12 with 6 words (two 4-letter, two 5-letter, two 6-letter).\n"
@@ -566,6 +501,76 @@ def _render_header():
     #     "- Scoring: length + unrevealed letters of that word at guess time.\n"
     #     "- Score Board: radar of last letter of word, score and status.\n"
     #     "- Words do not overlap, but may be touching.")
+    with st.expander("🎯 Challenge Mode (click to expand/collapse)", expanded=True):
+        # Show Challenge Mode banner if loading a shared game
+        shared_settings = st.session_state.get("shared_game_settings")
+        if shared_settings:
+            users = shared_settings.get("users", [])
+
+            if users:
+                # Sort users by score (descending), then by time (ascending)
+                sorted_users = sorted(users, key=lambda u: (-u["score"], u["time"]))
+                best_user = sorted_users[0]
+                best_score = best_user["score"]
+                best_time = best_user["time"]
+                mins, secs = divmod(best_time, 60)
+                best_time_str = f"{mins:02d}:{secs:02d}"
+
+                # Build leaderboard HTML
+                leaderboard_rows = []
+                for i, user in enumerate(sorted_users[:5], 1):  # Top 5
+                    u_mins, u_secs = divmod(user["time"], 60)
+                    u_time_str = f"{u_mins:02d}:{u_secs:02d}"
+                    medal = ["🥇", "🥈", "🥉"][i-1] if i <= 3 else f"{i}."
+                    leaderboard_rows.append(
+                        f"<div style='padding:0.25rem; font-size:0.85rem;'>{medal} {user['username']}: {user['score']} pts in {u_time_str}</div>"
+                    )
+                leaderboard_html = "".join(leaderboard_rows)
+
+                st.markdown(
+                    f"""
+                    <div style="
+                        background: linear-gradient(90deg, #1d64c8 0%, #165ba8 100%);
+                        color: white;
+                        padding: 1rem;
+                        border-radius: 0.5rem;
+                        margin-bottom: 1rem;
+                        text-align: center;
+                        box-shadow: 0 4px 6px rgba(0, 0, 0, 0.1);
+                    ">
+                        🎯 <strong>CHALLENGE MODE</strong> 🎯<br/>
+                        <span style="font-size: 0.9rem;">
+                            Beat the best: <strong>{best_score} points</strong> in <strong>{best_time_str}</strong> by <strong>{best_user['username']}</strong>
+                        </span>
+                        <div style="margin-top:0.75rem; border-top: 1px solid rgba(255,255,255,0.3); padding-top:0.5rem;">
+                            <strong style="font-size:0.9rem;">🏆 Leaderboard</strong>
+                            {leaderboard_html}
+                        </div>
+                    </div>
+                    """,
+                    unsafe_allow_html=True
+                )
+            else:
+                st.markdown(
+                    """
+                    <div style="
+                        background: linear-gradient(90deg, #1d64c8 0%, #165ba8 100%);
+                        color: white;
+                        padding: 1rem;
+                        border-radius: 0.5rem;
+                        margin-bottom: 1rem;
+                        text-align: center;
+                        box-shadow: 0 4px 6px rgba(0, 0, 0, 0.1);
+                    ">
+                        🎯 <strong>CHALLENGE MODE</strong> 🎯<br/>
+                        <span style="font-size: 0.9rem;">
+                            Be the first to complete this challenge!
+                        </span>
+                    </div>
+                    """,
+                    unsafe_allow_html=True
+                )
+
     inject_styles()
 
     st.markdown(
@@ -1673,9 +1678,12 @@ def run_app():
                 if users:
                     best_score = max(u["score"] for u in users)
                     best_time = min(u["time"] for u in users)
-                    st.info(f"🎯 Loading shared challenge (Best: {best_score} pts in {best_time}s by {len(users)} player(s))")
+                    st.toast(
+                        f"🎯 Loading shared challenge (Best: {best_score} pts in {best_time}s by {len(users)} player(s))",
+                        icon="ℹ️"
+                    )
                 else:
-                    st.info(f"🎯 Loading shared challenge")
+                    st.toast("🎯 Loading shared challenge", icon="ℹ️")
             else:
                 st.warning(f"No shared game found for ID: {sid}. Starting a normal game.")
                 st.session_state["shared_game_loaded"] = True  # Prevent repeated attempts

tests/test_download_game_settings.py

@@ -6,11 +6,16 @@ import pytest
 # Ensure the modules path is available
 sys.path.insert(0, os.path.abspath(os.path.join(os.path.dirname(__file__), "..")))
 
+from battlewords.modules.constants import HF_API_TOKEN  # <-- Import the token
 from battlewords.modules.storage import gen_full_url, _get_json_from_repo, HF_REPO_ID, SHORTENER_JSON_FILE
 
+# Ensure the token is set for Hugging Face Hub
+if HF_API_TOKEN:
+    os.environ["HF_API_TOKEN"] = HF_API_TOKEN
+
 def test_download_settings_by_short_id_handles_both(capsys):
     # Use a fixed short id for testing
-    short_id = "hDjsB_dl1"
+    short_id = "hDjsB_dl"
 
     # Step 1: Resolve short ID to full URL
     status, full_url = gen_full_url(
@@ -30,7 +35,10 @@ def test_download_settings_by_short_id_handles_both(capsys):
         assert "Could not resolve short id" in captured.out
         # Ensure failure shape is consistent
         assert not full_url, "full_url should be empty/None on failure"
+        print("settings.json was not found or could not be resolved.")
         return
+    else:
+        print(f"Resolved short id '{short_id}' to full URL: {full_url}")
 
     # Success branch
     assert status == "success_retrieved_full", f"Failed to resolve short ID: {status}"
@@ -45,8 +53,11 @@ def test_download_settings_by_short_id_handles_both(capsys):
     settings = _get_json_from_repo(HF_REPO_ID, file_path, repo_type="dataset")
     assert settings, "Failed to download or parse settings.json"
 
-    print("Downloaded settings.json:", settings)
+    print("\nDownloaded settings.json contents:", settings)
     # Optionally, add more assertions about the settings structure
-    assert "uid" in settings
-    assert "word_list" in settings
-    assert "score" in settings
+    assert "challenge_id" in settings
+    assert "wordlist_source" in settings
+    assert "users" in settings
+
+if __name__ == "__main__":
+    pytest.main(["-s", __file__])