thorstenalpers / Finance.NET / 35493165529

91%

push

github

Keep an Alpha Vantage rate-limit notice retryable (#64)

* Accept derived exceptions in the Alpha Vantage integration tests

Assert.ThrowsAsync matches the exact type, so the negative cases turned
red once #63 made a refused request throw FinanceNetAccessDeniedException.

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>

* Keep an Alpha Vantage rate-limit notice retryable

The notice advertises "premium endpoints", so the "premium endpoint"
marker classified a temporary rate limit as a permanent refusal. The
marker now matches the full sentence. The notice also lacks the old
"higher API call volume" wording, so GetOverviewAsync deserialized it
to an empty object and reported no data without retrying. "rate limit"
is now recognised as a retryable limit in all four methods.

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>

---------

Co-authored-by: thorsten <thorsten@PC>
Co-authored-by: Claude Fable 5.1 <noreply@anthropic.com>

468 of 633 branches covered (73.93%)

Branch coverage included in aggregate %.

6 of 6 new or added lines in 1 file covered. (100.0%)

1810 of 1870 relevant lines covered (96.79%)

93.78 hits per line