summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authormontezdesousa <79287829+montezdesousa@users.noreply.github.com>2024-06-17 10:09:28 +0100
committerGitHub <noreply@github.com>2024-06-17 09:09:28 +0000
commit5ec2db7643864b6a08d844126314377d35d812cc (patch)
tree2eb3961f76b4c0e50f29e2a6c0deeddea392f562
parentc5e16684baab63ce93deeb7641169ae4d3a1cf4a (diff)
[BugFix] Replace some base exceptions by OpenBBError (#6502)
* fix: replace base exceptions by openbberrors * Update yield_curve.py * fix typo --------- Co-authored-by: Danglewood <85772166+deeleeramone@users.noreply.github.com>
-rw-r--r--.pre-commit-config.yaml2
-rw-r--r--openbb_platform/providers/biztoc/openbb_biztoc/utils/helpers.py7
-rw-r--r--openbb_platform/providers/cboe/openbb_cboe/models/futures_curve.py3
-rw-r--r--openbb_platform/providers/cboe/openbb_cboe/models/options_chains.py3
-rw-r--r--openbb_platform/providers/ecb/openbb_ecb/models/currency_reference_rates.py5
-rw-r--r--openbb_platform/providers/ecb/openbb_ecb/models/yield_curve.py3
-rw-r--r--openbb_platform/providers/econdb/openbb_econdb/models/country_profile.py3
-rw-r--r--openbb_platform/providers/econdb/openbb_econdb/models/economic_indicators.py17
-rw-r--r--openbb_platform/providers/econdb/openbb_econdb/models/yield_curve.py5
-rw-r--r--openbb_platform/providers/econdb/openbb_econdb/utils/helpers.py11
-rw-r--r--openbb_platform/providers/econdb/openbb_econdb/utils/main_indicators.py9
-rw-r--r--openbb_platform/providers/federal_reserve/openbb_federal_reserve/utils/ny_fed_api.py29
-rw-r--r--openbb_platform/providers/fmp/openbb_fmp/models/forward_ebitda_estimates.py4
-rw-r--r--openbb_platform/providers/fmp/openbb_fmp/models/forward_eps_estimates.py4
-rw-r--r--openbb_platform/providers/fmp/openbb_fmp/models/price_target_consensus.py3
-rw-r--r--openbb_platform/providers/fmp/openbb_fmp/utils/helpers.py5
-rw-r--r--openbb_platform/providers/fred/openbb_fred/models/hqm.py5
-rw-r--r--openbb_platform/providers/fred/openbb_fred/models/regional.py3
-rw-r--r--openbb_platform/providers/fred/openbb_fred/models/spot.py3
-rw-r--r--openbb_platform/providers/fred/openbb_fred/utils/fred_base.py5
-rw-r--r--openbb_platform/providers/fred/openbb_fred/utils/fred_helpers.py9
-rw-r--r--openbb_platform/providers/government_us/openbb_government_us/models/treasury_prices.py7
-rw-r--r--openbb_platform/providers/intrinio/openbb_intrinio/models/cash_flow.py3
-rw-r--r--openbb_platform/providers/intrinio/openbb_intrinio/models/company_news.py7
-rw-r--r--openbb_platform/providers/intrinio/openbb_intrinio/models/equity_historical.py3
-rw-r--r--openbb_platform/providers/intrinio/openbb_intrinio/models/etf_holdings.py6
-rw-r--r--openbb_platform/providers/intrinio/openbb_intrinio/models/etf_search.py5
-rw-r--r--openbb_platform/providers/intrinio/openbb_intrinio/models/financial_ratios.py3
-rw-r--r--openbb_platform/providers/intrinio/openbb_intrinio/models/forward_ebitda_estimates.py3
-rw-r--r--openbb_platform/providers/intrinio/openbb_intrinio/models/forward_eps_estimates.py3
-rw-r--r--openbb_platform/providers/intrinio/openbb_intrinio/models/forward_pe_estimates.py3
-rw-r--r--openbb_platform/providers/intrinio/openbb_intrinio/models/forward_sales_estimates.py3
-rw-r--r--openbb_platform/providers/intrinio/openbb_intrinio/models/income_statement.py3
-rw-r--r--openbb_platform/providers/intrinio/openbb_intrinio/models/market_snapshots.py9
-rw-r--r--openbb_platform/providers/intrinio/openbb_intrinio/models/options_unusual.py4
-rw-r--r--openbb_platform/providers/intrinio/openbb_intrinio/models/price_target_consensus.py3
-rw-r--r--openbb_platform/providers/intrinio/openbb_intrinio/models/reported_financials.py3
-rw-r--r--openbb_platform/providers/intrinio/openbb_intrinio/utils/helpers.py5
-rw-r--r--openbb_platform/providers/nasdaq/openbb_nasdaq/models/equity_search.py3
-rw-r--r--openbb_platform/providers/nasdaq/openbb_nasdaq/utils/helpers.py5
-rw-r--r--openbb_platform/providers/oecd/openbb_oecd/models/consumer_price_index.py13
-rw-r--r--openbb_platform/providers/oecd/openbb_oecd/models/gdp_forecast.py7
-rw-r--r--openbb_platform/providers/oecd/openbb_oecd/models/house_price_index.py12
-rw-r--r--openbb_platform/providers/oecd/openbb_oecd/models/immediate_interest_rate.py12
-rw-r--r--openbb_platform/providers/oecd/openbb_oecd/models/share_price_index.py12
-rw-r--r--openbb_platform/providers/oecd/openbb_oecd/models/unemployment.py12
-rw-r--r--openbb_platform/providers/oecd/openbb_oecd/utils/helpers.py3
-rw-r--r--openbb_platform/providers/polygon/openbb_polygon/models/currency_pairs.py3
-rw-r--r--openbb_platform/providers/polygon/openbb_polygon/models/currency_snapshots.py6
-rw-r--r--openbb_platform/providers/polygon/openbb_polygon/utils/helpers.py3
-rw-r--r--openbb_platform/providers/sec/openbb_sec/models/company_filings.py5
-rw-r--r--openbb_platform/providers/sec/openbb_sec/models/etf_holdings.py4
-rw-r--r--openbb_platform/providers/sec/openbb_sec/models/schema_files.py3
-rw-r--r--openbb_platform/providers/sec/openbb_sec/models/symbol_map.py3
-rw-r--r--openbb_platform/providers/sec/openbb_sec/utils/frames.py15
-rw-r--r--openbb_platform/providers/sec/openbb_sec/utils/helpers.py7
-rw-r--r--openbb_platform/providers/sec/openbb_sec/utils/parse_13f.py19
-rw-r--r--openbb_platform/providers/seeking_alpha/openbb_seeking_alpha/models/calendar_earnings.py3
-rw-r--r--openbb_platform/providers/seeking_alpha/openbb_seeking_alpha/models/forward_eps_estimates.py15
-rw-r--r--openbb_platform/providers/seeking_alpha/openbb_seeking_alpha/models/forward_sales_estimates.py11
-rw-r--r--openbb_platform/providers/tiingo/openbb_tiingo/utils/helpers.py5
-rw-r--r--openbb_platform/providers/tmx/openbb_tmx/models/company_news.py3
-rw-r--r--openbb_platform/providers/tmx/openbb_tmx/models/equity_historical.py3
-rw-r--r--openbb_platform/providers/tmx/openbb_tmx/models/index_constituents.py5
-rw-r--r--openbb_platform/providers/tmx/openbb_tmx/models/price_target_consensus.py5
-rw-r--r--openbb_platform/providers/tmx/openbb_tmx/utils/helpers.py15
-rw-r--r--openbb_platform/providers/tradier/openbb_tradier/models/equity_historical.py3
-rw-r--r--openbb_platform/providers/tradier/openbb_tradier/models/equity_quote.py3
-rw-r--r--openbb_platform/providers/tradier/openbb_tradier/models/equity_search.py7
-rw-r--r--openbb_platform/providers/tradier/openbb_tradier/models/options_chains.py5
-rw-r--r--openbb_platform/providers/tradingeconomics/openbb_tradingeconomics/models/economic_calendar.py5
-rw-r--r--openbb_platform/providers/yfinance/openbb_yfinance/models/historical_dividends.py7
-rw-r--r--openbb_platform/providers/yfinance/openbb_yfinance/models/key_executives.py5
-rw-r--r--openbb_platform/providers/yfinance/openbb_yfinance/models/options_chains.py3
-rw-r--r--openbb_platform/providers/yfinance/openbb_yfinance/models/price_target_consensus.py3
75 files changed, 263 insertions, 201 deletions
diff --git a/.pre-commit-config.yaml b/.pre-commit-config.yaml
index 0b0a2174e4d..651aae6c924 100644
--- a/.pre-commit-config.yaml
+++ b/.pre-commit-config.yaml
@@ -36,7 +36,7 @@ repos:
entry: codespell
args:
[
- "--ignore-words-list=VAI,MIS,shs,gard,te,commun,parana,ro,zar,vie,hel,jewl,zlot,ba,buil,coo,ether,hist,hsi,mape,navagation,operatio,pres,ser,yeld,shold,ist,varian,datas,ake,creat,statics,ket,toke,certi,buidl,ot,fo",
+ "--ignore-words=.codespell.ignore",
"--quiet-level=2",
"--skip=./**/tests/**,./**/test_*.py,.git,*.css,*.csv,*.html,*.ini,*.ipynb,*.js,*.json,*.lock,*.scss,*.txt,*.yaml,build/pyinstaller/*,./website/config.toml",
"-x=.github/workflows/general-linting.yml"
diff --git a/openbb_platform/providers/biztoc/openbb_biztoc/utils/helpers.py b/openbb_platform/providers/biztoc/openbb_biztoc/utils/helpers.py
index 061271c8a5d..ffb5c2a5a8b 100644
--- a/openbb_platform/providers/biztoc/openbb_biztoc/utils/helpers.py
+++ b/openbb_platform/providers/biztoc/openbb_biztoc/utils/helpers.py
@@ -5,6 +5,7 @@ from typing import Dict, List, Literal
import requests
import requests_cache
+from openbb_core.app.model.abstract.error import OpenBBError
from openbb_core.app.utils import get_user_cache_directory
# pylint: disable=C0325
@@ -115,10 +116,10 @@ def get_news(
"tag": f"tag/{tag}",
}
if filter_ == "source" and source.lower() not in sources:
- raise ValueError(f"{source} not a valid source. Valid sources: {sources}")
+ raise OpenBBError(f"{source} not a valid source. Valid sources: {sources}")
if filter_ == "tag" and tag.lower().replace(" ", "") not in tags:
- raise ValueError(f"{tag} not a valid tag. Valid tags: {tags}")
+ raise OpenBBError(f"{tag} not a valid tag. Valid tags: {tags}")
url = (
f"https://biztoc.p.rapidapi.com/search?q={term}"
@@ -127,7 +128,7 @@ def get_news(
)
r = requests.get(url, headers=headers, timeout=5)
if r.status_code != 200:
- raise RuntimeError(f"HTTP error - > {r.text}")
+ raise OpenBBError(f"HTTP error - > {r.text}")
try:
results = r.json()
diff --git a/openbb_platform/providers/cboe/openbb_cboe/models/futures_curve.py b/openbb_platform/providers/cboe/openbb_cboe/models/futures_curve.py
index fee62fb339a..ce1a267f4df 100644
--- a/openbb_platform/providers/cboe/openbb_cboe/models/futures_curve.py
+++ b/openbb_platform/providers/cboe/openbb_cboe/models/futures_curve.py
@@ -4,6 +4,7 @@
from typing import Any, Dict, List, Optional
from openbb_cboe.utils.helpers import get_settlement_prices
+from openbb_core.app.model.abstract.error import OpenBBError
from openbb_core.provider.abstract.fetcher import Fetcher
from openbb_core.provider.standard_models.futures_curve import (
FuturesCurveData,
@@ -52,7 +53,7 @@ class CboeFuturesCurveFetcher(
raise EmptyDataError()
if symbol not in FUTURES["product"].unique().tolist():
- raise RuntimeError(
+ raise OpenBBError(
"The symbol, "
f"{symbol}"
", is not valid. Chose from: "
diff --git a/openbb_platform/providers/cboe/openbb_cboe/models/options_chains.py b/openbb_platform/providers/cboe/openbb_cboe/models/options_chains.py
index ca049ee80a7..16735cc4b76 100644
--- a/openbb_platform/providers/cboe/openbb_cboe/models/options_chains.py
+++ b/openbb_platform/providers/cboe/openbb_cboe/models/options_chains.py
@@ -10,6 +10,7 @@ from openbb_cboe.utils.helpers import (
get_company_directory,
get_index_directory,
)
+from openbb_core.app.model.abstract.error import OpenBBError
from openbb_core.provider.abstract.annotated_result import AnnotatedResult
from openbb_core.provider.abstract.fetcher import Fetcher
from openbb_core.provider.standard_models.options_chains import (
@@ -66,7 +67,7 @@ class CboeOptionsChainsFetcher(
INDEXES = INDEXES.set_index("index_symbol")
if symbol not in SYMBOLS.index:
- raise RuntimeError(f"{symbol} was not found in the Cboe options directory.")
+ raise OpenBBError(f"{symbol} was not found in the Cboe options directory.")
quotes_url = (
f"https://cdn.cboe.com/api/global/delayed_quotes/options/_{symbol}.json"
diff --git a/openbb_platform/providers/ecb/openbb_ecb/models/currency_reference_rates.py b/openbb_platform/providers/ecb/openbb_ecb/models/currency_reference_rates.py
index 3eea898ff3c..278e28677f3 100644
--- a/