fix(cli): update how settings are retrieved for cli

need to add tests for cli since that is where most people will use the code

Author: Karl Wooster

src/ynamazon/cli/cli.py

@@ -19,6 +19,7 @@
 from ynamazon.settings import ConfigFile, SecretApiKey, SecretBudgetId, get_settings
 from ynamazon.ynab_transactions import (
     TempYnabTransaction,
+    get_default_ynab_config,
     get_ynab_transactions,
     update_ynab_transaction,
 )
@@ -227,15 +228,14 @@ def new_ynamazon(  # noqa: C901
             }
         )
 
-    ynab_config = Configuration(
-        access_token=cli_settings.ynab_api_key.get_secret_value()
-    )
+    ynab_config = get_default_ynab_config()
 
     try:
         ynab_trans, amazon_with_memo_payee = get_ynab_transactions(
             configuration=ynab_config,
-            budget_id=cli_settings.ynab_budget_id.get_secret_value(),
+            budget_id=cli_settings.get_secret_value("ynab_budget_id"),
         )
+        console.print(f"[bold green]Found {len(ynab_trans)} transactions.[/]")
     except YnabSetupError as e:
         console.print(f"[bold red]get_settings() error: {e}[/]")
         console.print(

src/ynamazon/main.py

@@ -15,8 +15,8 @@
 
 from .exceptions import YnabSetupError
 from .settings import get_settings
-from .ynab_transactions import default_configuration as ynab_configuration
 from .ynab_transactions import (
+    get_default_ynab_config,
     get_ynab_transactions,
     markdown_formatted_link,
     markdown_formatted_title,
@@ -113,7 +113,7 @@ def process_transactions(
 ) -> None:
     """Match YNAB transactions to Amazon Transactions and optionally update YNAB Memos."""
     amazon_config = amazon_config or AmazonConfig()
-    ynab_config = ynab_config or ynab_configuration
+    ynab_config = ynab_config or get_default_ynab_config()
     budget_id = budget_id or get_settings().ynab_budget_id.get_secret_value()
 
     console = Console()