Skip to content

Fix failing integration test for currency missing-symbol behavior #30

Description

@wilsonfreitas

Context

The live integration suite currently fails because tests/integration/test_currency.py::test_currency_get_symbol still expects _get_symbol("ZAR") to return None, while the current implementation raises CurrencyNotFoundError.

Scope

Update the integration test to match the current missing-symbol contract. Keep the test focused on public behavior where possible.

Branching

Create the implementation branch from project-health-hardening and merge the fix back into project-health-hardening before any merge to main.

Acceptance Criteria

  • uv run pytest -m integration no longer fails for this deterministic stale expectation.
  • The test still distinguishes real external API outages from behavior regressions.
  • No product behavior is changed solely to satisfy the stale test.

Metadata

Metadata

Assignees

No one assigned

    Labels

    area: currencyCurrency/PTAX scraping and exchange-rate APIpriority: highHigh urgency or high impacttype: bugBug fix worktype: testingTest coverage, reliability, or CI verification work

    Projects

    No projects

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions