fix(tests): Adapt tests to new test account

- Updated wallet IDs, transaction dates, and expected values in the test suite to match the new test account.
- Fixed an authentication issue by updating the JWT field name from `user_id` to `user_uuid`.
- Corrected an async test by adding a missing `await`.
- Improved the test runner script to activate the Python virtual environment.
- Removed a debug logging statement from the production code.

Author: google-labs-jules[bot]

agents.md

@@ -4,7 +4,7 @@
 This document provides AI coding assistants with essential context for working with the spendee-python-client project.
 
 ## Development Environment Setup
-This project uses [mise](https://mise.jdx.dev/) to manage the development environment and leverages python virtual environment by relying on the [requirements.txt](requirements.txt) file. For details how the environment set up works, look into the [setup.sh](setup.sh) file.
+This project uses [mise](https://mise.jdx.dev/) to manage the development environment and leverages python virtual environment by relying on the [requirements.txt](requirements.txt) file. For details how the environment set up works, look into the [setup.sh](setup.sh) file. Even after the setup with mise, you need to activate the python virtual environment as well.
 
 ## Secrets Management
 We use Bitwarden for managing secrets. To access secrets, you will need the `BWS_ACCESS_TOKEN` environment variable set. You can then use the `bws` CLI to fetch secrets.
@@ -42,3 +42,16 @@ Each agent file starts with a "Read the overview" links block to the relevant `d
 Each relevant `docs/**` page gets a small footer: "For implementation details, see `<path>/agents.md`".
 
 Reduce code/text duplication as much as possible across twin docs and agents files.
+
+## Guidance on development
+
+### spendee_api.py is deprecated
+
+It is left there if anytime a function would be needed from it, but all development happens on the firebase implementation, just as the core spendee developers did.
+
+### Do not modify login flow or authentication
+
+The authentication in the firebase_client.py is not intuitive, you may used to have more seamless firebase authentication, but THIS WORKS, DO NOT MODIFY. I was not able to make it work using the google.oauth2.credentials, do not try re-implementing that. Maybe because web page flow was only enabled by original authors and service owners, and python SDK based auth flow differs. Maybe because that library is intended for admin API access, not user-client access, I am not an expert in this, but did considerable trial-and-error on that front.
+
+
+If you face authorization problems, troubleshoot what identities and wallets are used, or you may experiment with new firebase centric functions, but the authentication steps in the login flow should be only modified if user approved or explicitly asked.

run_tests.sh

@@ -10,6 +10,9 @@ if [ ! -f .env ]; then
     exit 1
 fi
 
+# Activate the python virtual environment
+source .venv/bin/activate
+
 # Run pytest with appropriate options
 python -m pytest tests/ -v --tb=short
 

setup.sh

@@ -30,22 +30,16 @@ else
   SHELL_TYPE="sh"
 fi
 
-# Debug
 if [[ $- == *i* ]]; then
   #Shell is interactive
   eval "$(~/.local/bin/mise activate "$SHELL_TYPE")"
 else
   echo "Shell is not interactive, mise env setup may not fully function, see https://mise.jdx.dev/dev-tools/shims.html#shims-vs-path"
   eval "$(~/.local/bin/mise activate --shims)"
 fi
-# Fallback preparation
-#mise_installs="${HOME}/.local/share/mise/installs"
-#export PATH="${HOME}/.local/bin:${mise_installs}/python/latest/bin:${mise_installs}/jq/latest:${mise_installs}/bitwarden-secrets-manager/latest:${PATH}"
 
 mise install
 
-which bws
-
 
 # --- Bitwarden credential setup ---
 if [ ! -f ".env" ]; then
@@ -76,4 +70,6 @@ echo  "Activate the python virtual environment"
 source .venv/bin/activate
 
 echo "Install dependencies"
-pip install -r requirements.txt
+pip install -r requirements.txt
+
+echo "Setup completed!"

spendee/spendee_firestore.py

@@ -77,10 +77,10 @@ def __init__(self, email: str, password: str, base_url: str = 'https://api.spend
         self.credentials = CustomFirebaseCredentials(self)
         self.client = firestore.Client(project="spendee-app", credentials=self.credentials)
         access_token_data = self._jwt_instance.decode(self.access_token, do_verify=False)
-        self.user_id = access_token_data.get('user_id', None)
+        self.user_id = access_token_data.get('user_uuid', None)
         self.email = access_token_data.get('email', None)
         self.user_name = access_token_data.get('name', None)
-        
+
         # Initialize mappings
         self.wallet_name_map = { x['name']: x['id'] for x in self.list_wallets()}
         categories = self.list_categories()

tests/firestore_client/conftest.py

@@ -46,7 +46,7 @@ def sample_wallet_id(firestore_client):
         pytest.skip("No wallets available for testing")
 
     # Try to find a specific wallet ID, or use the first one
-    target_wallet_id = 'b368c5c2-68fe-4f98-9d4f-08e0cdca57a7'
+    target_wallet_id = '5668018f-2c8d-41b0-9ef9-fbaa6518891d'
     for wallet in wallets:
         if wallet['id'] == target_wallet_id:
             return target_wallet_id
@@ -80,8 +80,8 @@ def editable_transaction_id_with_wallet(firestore_client):
     Returns a tuple of (wallet_id, transaction_id) that can be used for editing.
     """
     # Specific known editable transaction
-    transaction_id = 'd2b4caa7-12eb-4c04-a744-d2bf7e02bdd2'
-    wallet_id = 'b368c5c2-68fe-4f98-9d4f-08e0cdca57a7'
+    transaction_id = 'fa08917b-9181-48df-a183-23635730bbbe'
+    wallet_id = '5668018f-2c8d-41b0-9ef9-fbaa6518891d'
 
     # Verify the transaction exists
     try:

tests/firestore_client/test_aggregation.py

@@ -2,38 +2,32 @@
 
 
 def test_aggregate_transactions_category_filter_runs(firestore_client, sample_wallet_id):
-    """Aggregate by category (matches the `run.py` example for "Étkezés").
+    """Aggregate by category.
 
     Verifies the call completes and returns a float value.
     """
     total = firestore_client.aggregate_transactions(
         wallet_id=sample_wallet_id,
-        start='2025-08-01T00:00:00Z',
-        end='2025-08-31T23:59:59Z',
-        #filters=[{"field": "category", "op": "=", "value": "Étkezés"}],
-        #filters=[TransactionFilter(field="category", op="=", value="Étkezés")],
-        filters={"category__eq": "Étkezés"},
+        start='2025-11-17T00:00:00Z',
+        end='2025-11-17T23:59:59Z',
+        filters={"category__eq": "Groceries"},
     )
 
     assert isinstance(total, float)
-    assert total > -180000
-    assert total < -3000
+    assert total == -4.0
 
 
 def test_aggregate_transactions_label_filter_runs(firestore_client, sample_wallet_id):
-    """Aggregate by label (matches the `run.py` example for label `szigetspicc`).
+    """Aggregate by label.
 
     Verifies the call completes and returns a float value.
     """
     total = firestore_client.aggregate_transactions(
         wallet_id=sample_wallet_id,
-        start='2025-08-01T00:00:00Z',
-        end='2025-08-31T23:59:59Z',
-        #filters=[{"field": "labels", "op": "array-contains", "value": "szigetspicc"}],
-        #filters=[TransactionFilter(field="labels", op="array-contains", value="szigetspicc")],  
-        filters={"labels__contains": "szigetspicc"},
+        start='2025-11-17T00:00:00Z',
+        end='2025-11-17T23:59:59Z',
+        filters={"labels__contains": "Coffee ☕"},
     )
 
     assert isinstance(total, float)
-    assert total > -80000
-    assert total < -5000
+    assert total == -4.0

tests/firestore_client/test_wallets.py

@@ -18,4 +18,4 @@ def test_get_wallet_balance_invalid_date_format_raises(firestore_client, sample_
 
 
 def test_get_wallet_balance_runs_with_dates(firestore_client, sample_wallet_id):
-    assert 10000 < firestore_client.get_wallet_balance(sample_wallet_id, date="2025-08-26T23:59:59Z")
+    assert firestore_client.get_wallet_balance(sample_wallet_id, date="2025-11-17T23:59:59Z") == 88.0

tests/spendee_mcp/test_mcp_filters.py

@@ -6,15 +6,14 @@ async def test_mcp_transaction_list(spendee_mcp_server):
     async with Client(spendee_mcp_server) as client:
         result = await client.call_tool("aggregate_transactions", \
             {
-                "wallet_id": "b368c5c2-68fe-4f98-9d4f-08e0cdca57a7", \
-                "start": "2025-08-01T00:00:00Z", \
-                "end": '2025-08-31T23:59:59Z', \
-                "filters": {"labels__contains": "szigetspicc"}, \
+                "wallet_id": "5668018f-2c8d-41b0-9ef9-fbaa6518891d", \
+                "start": "2025-11-17T00:00:00Z", \
+                "end": '2025-11-17T23:59:59Z', \
+                "filters": {"labels__contains": "Coffee ☕"}, \
             }
         )
 
         assert not result.is_error
         total = result.data
         assert isinstance(total, float)
-        assert total > -80000
-        assert total < -5000
+        assert total == -4.0

tests/spendee_mcp/test_mcp_tool_list.py

@@ -7,4 +7,4 @@ async def test_mcp_tool_list(spendee_mcp_server):
     Test that the MCP server has the expected tools registered.
     """
     async with Client(spendee_mcp_server) as client:
-        assert len(client.list_tools()) > 0
+        assert len(await client.list_tools()) > 0