Removed held argument as non-functional in price

Author: Romamo

.github/workflows/ci.yml

@@ -13,10 +13,6 @@ jobs:
       matrix:
         python-version:
           - "3.10"
-          - "3.11"
-          - "3.12"
-          - "3.13"
-          - "3.14"
       fail-fast: false
     steps:
       - name: Checkout

src/beancount_cli/commands/price.py

@@ -75,9 +75,6 @@ def price_fetch(
         False, "--inactive", "-i", help="Include commodities with no balance"
     ),
     fill_gaps: bool = typer.Option(False, "--fill-gaps", help="Fill gaps in price history"),
-    held: bool = typer.Option(
-        False, "--held", help="Include commodities with non-zero balance in any account"
-    ),
     dry_run: bool = False,
 ):
     """Fetch and update prices using bean-price library.
@@ -128,12 +125,11 @@ def price_fetch(
             jobs = bp_price.get_price_jobs_up_to_date(
                 entries,
                 date_last=datetime.now().date(),
-                inactive=inactive,
-                held=held,
+                inactive=inactive
             )
         else:
             jobs = bp_price.get_price_jobs_at_date(
-                entries, date=None, inactive=inactive, held=held
+                entries, date=None, inactive=inactive
             )
 
         if not jobs:

tests/test_cli.py

@@ -219,20 +219,3 @@ def test_report_audit_help_shows_audit_only_flags():
     assert "--limit" in (out + err)
     assert "--all" in (out + err)
 
-def test_price_fetch_held(temp_beancount_file):
-    with patch("beancount_cli.commands.price.bp_price.get_price_jobs_at_date") as mock_get_jobs:
-        mock_get_jobs.return_value = []
-        code, out, err = run_cli("price", "fetch", "--held", str(temp_beancount_file))
-        assert code in (0, None)
-        mock_get_jobs.assert_called_once()
-        assert mock_get_jobs.call_args[1].get("held") is True
-
-def test_price_fetch_update(temp_beancount_file):
-    with patch("beancount_cli.commands.price.bp_price.get_price_jobs_up_to_date") as mock_get_jobs:
-        mock_get_jobs.return_value = []
-        code, out, err = run_cli("price", "fetch", "--update", str(temp_beancount_file))
-        assert code in (0, None)
-        mock_get_jobs.assert_called_once()
-        # Verify that held is passed as False by default
-        assert mock_get_jobs.call_args[1].get("held") is False
-