equity.estimates.price_target - Integrate with Benzinga (#5946)

* deprecation message

* fix lints

* adding equity.estimates.ratings and fmp as its provider

* unit tests for fmp

* removing unused file

* typos

* add benzinga to AnalystRatings

* revert changes to estimates router

* moving it to price target instead

* reverting changes

* fmp support for multiple ticker

* benzinga price target

* ruffy

* unused argument

* fix type hints

* tests for the new fetcher

* new integration tests

* fixing tests

* missing std arg

---------

Co-authored-by: Igor Radovanovic <74266147+IgorWounds@users.noreply.github.com>
This commit is contained in:
Henrique Joaquim
2024-01-17 18:39:40 +00:00
committed by GitHub
parent 37c7b14605
commit 31d03eb5f1
11 changed files with 470 additions and 31 deletions

View File

@@ -568,7 +568,24 @@ def test_equity_ownership_major_holders(params, obb):
@parametrize(
"params",
[
({"symbol": "AAPL", "provider": "fmp"}),
({"symbol": "AAPL", "limit": 10, "provider": "fmp"}),
(
{
"symbol": "AAPL",
"limit": 10,
"provider": "benzinga",
# optional provider params
"fields": None,
"date": None,
"date_from": None,
"date_to": None,
"importance": None,
"updated": None,
"action": None,
"analyst": None,
"firm": None,
}
),
],
)
@pytest.mark.integration