search

ooples / OoplesFinance.YahooFinanceAPI

A .NET API wrapper for Yahoo Finance that allows users to get top trending stocks, historical stock data, stock splits, dividends, capital gains, and much much more!
Apache License 2.0
47 stars 10 forks source link

Unit tests don't build due to recent merge #56

Closed ooples closed 6 months ago

ooples commented 6 months ago

I should have done my due diligence before I did the recent pull request but the merge is now causing unit tests to fail due to the mocked HTTP client containing no headers.

ooples commented 6 months ago

@oscar811 Please run the unit tests with test explorer on Visual Studio so you can address the issues with your recent unit tests. There are no headers being mocked so your code to check for headers fails and since the crumb is empty, an exception gets thrown