Skip to content

Commit

Permalink
Remove ultima from deps (#5736)
Browse files Browse the repository at this point in the history
* remove ultima from deps

* skip ultima test

* black

* change skip location

* try except statement to avoid error on the ci
  • Loading branch information
hjoaquim authored and piiq committed Nov 17, 2023
1 parent 279b7aa commit 9f38a02
Show file tree
Hide file tree
Showing 3 changed files with 6 additions and 3 deletions.
1 change: 0 additions & 1 deletion openbb_platform/dev_install.py
Original file line number Diff line number Diff line change
Expand Up @@ -46,7 +46,6 @@
openbb-nasdaq = { path = "./providers/nasdaq", optional = true, develop = true }
openbb-seeking-alpha = { path = "./providers/seeking_alpha", optional = true, develop = true }
openbb-stockgrid = { path = "./providers/stockgrid" ,optional = true, develop = true }
openbb-ultima = { path = "./providers/ultima", optional = true, develop = true }
openbb-wsj = { path = "./providers/wsj", develop = true }
openbb-yfinance = { path = "./providers/yfinance", optional = true, develop = true }
Expand Down
Original file line number Diff line number Diff line change
@@ -1,6 +1,10 @@
import pytest
from openbb_core.app.service.user_service import UserService
from openbb_ultima.models.company_news import UltimaCompanyNewsFetcher

try:
from openbb_ultima.models.company_news import UltimaCompanyNewsFetcher
except ImportError:
pytest.skip("openbb-ultima is not installed on the CI.", allow_module_level=True)

test_credentials = UserService().default_user_settings.credentials.model_dump(
mode="json"
Expand All @@ -21,6 +25,7 @@ def vcr_config():


@pytest.mark.record_http
@pytest.mark.skip(reason="openbb-ultima is not installed on the CI.")
def test_ultima_company_news_fetcher(credentials=test_credentials):
params = {"symbols": "AAPL, MSFT"}

Expand Down
1 change: 0 additions & 1 deletion openbb_platform/pyproject.toml
Original file line number Diff line number Diff line change
Expand Up @@ -78,7 +78,6 @@ all = [
"openbb-seeking-alpha",
"openbb-stockgrid",
"openbb-technical",
"openbb-ultima",
"openbb-wsj",
"openbb-yfinance",
]
Expand Down

0 comments on commit 9f38a02

Please sign in to comment.