alihan/yfinance
1
0
Fork 0
mirror of https://github.com/ranaroussi/yfinance.git synced 2024-01-29 09:38:56 +03:00
Code Issues Packages Projects Releases Wiki Activity

Merge pull request #1793 from ranaroussi/fix/fetch-tkr-tz

Browse Source 
Fix _get_ticker_tz() args, were being swapped. Improve its unit test
This commit is contained in:
ValueRaider
2023-12-17 18:59:06 +00:00
committed by GitHub
parent 281cc64a4a 9021fe52b4
commit f753e6090d
2 changed files with 3 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
tests/ticker.py
View File

@@ -86,7 +86,7 @@ class TestTicker(unittest.TestCase):
# Test:
dat = yf.Ticker(tkr, session=self.session)
tz = dat._get_ticker_tz(proxy=None, timeout=None)
tz = dat._get_ticker_tz(proxy=None, timeout=5)
self.assertIsNotNone(tz)

4
yfinance/base.py
View File

@@ -1650,7 +1650,7 @@ class TickerBase:
return df2
def _get_ticker_tz(self,timeout, proxy=None):
def _get_ticker_tz(self, proxy, timeout):
proxy = proxy or self.proxy
if self._tz is not None:
return self._tz
@@ -1675,7 +1675,7 @@ class TickerBase:
return tz
@utils.log_indent_decorator
def _fetch_ticker_tz(self, timeout, proxy=None):
def _fetch_ticker_tz(self, proxy, timeout):
# Query Yahoo for fast price data just to get returned timezone
proxy = proxy or self.proxy
logger = utils.get_yf_logger()