Skip to content

Commit a34b337

Browse files
authored
Fix long line issues (#502)
1 parent 52c33ce commit a34b337

File tree

2 files changed

+9
-5
lines changed

2 files changed

+9
-5
lines changed

alpaca_trade_api/rest.py

+3-2
Original file line numberDiff line numberDiff line change
@@ -113,13 +113,14 @@ def validate(amount: int, unit: TimeFrameUnit):
113113
raise "Amount must be a positive integer value."
114114

115115
if unit == TimeFrameUnit.Minute and amount > 59:
116-
raise "Second or Minute units can only be used with amounts between 1-59."
116+
raise "Second or Minute units can only be used " + \
117+
"with amounts between 1-59."
117118

118119
if unit == TimeFrameUnit.Hour and amount > 23:
119120
raise "Hour units can only be used with amounts 1-23"
120121

121122

122-
# TimeFrame.Minute, TimeFrame.Hour and TimeFrame.Day kept for backwards compatibility
123+
# These are kept for backwards compatibility
123124
TimeFrame.Minute = TimeFrame(1, TimeFrameUnit.Minute)
124125
TimeFrame.Hour = TimeFrame(1, TimeFrameUnit.Hour)
125126
TimeFrame.Day = TimeFrame(1, TimeFrameUnit.Day)

tests/test_rest.py

+6-3
Original file line numberDiff line numberDiff line change
@@ -791,15 +791,17 @@ def test_data(reqmock):
791791

792792
def test_timeframe(reqmock):
793793
# Custom timeframe: Minutes
794-
reqmock.get('https://data.alpaca.markets/v2/stocks/AAPL/bars?timeframe=45Min&adjustment=raw&'
794+
reqmock.get('https://data.alpaca.markets/v2/stocks/AAPL/bars?'
795+
'timeframe=45Min&adjustment=raw&'
795796
'start=2021-06-08&end=2021-06-08', text='{}')
796797
api = tradeapi.REST('key-id', 'secret-key', api_version='v1')
797798
timeframe = tradeapi.TimeFrame(45, tradeapi.TimeFrameUnit.Minute)
798799
api.get_bars('AAPL', timeframe, '2021-06-08', '2021-06-08')
799800
assert reqmock.called
800801

801802
# Custom timeframe: Hours
802-
reqmock.get('https://data.alpaca.markets/v2/stocks/AAPL/bars?timeframe=23Hour&adjustment=raw&'
803+
reqmock.get('https://data.alpaca.markets/v2/stocks/AAPL/bars?'
804+
'timeframe=23Hour&adjustment=raw&'
803805
'start=2021-06-08&end=2021-06-08', text='{}')
804806
timeframe = tradeapi.TimeFrame(23, tradeapi.TimeFrameUnit.Hour)
805807
api.get_bars('AAPL', timeframe, '2021-06-08', '2021-06-08')
@@ -817,7 +819,8 @@ def test_timeframe(reqmock):
817819
timeframe = tradeapi.TimeFrame(23, tradeapi.TimeFrameUnit.Hour)
818820
timeframe.amount = 5
819821
timeframe.unit = tradeapi.TimeFrameUnit.Minute
820-
reqmock.get('https://data.alpaca.markets/v2/stocks/AAPL/bars?timeframe=5Min&adjustment=raw&'
822+
reqmock.get('https://data.alpaca.markets/v2/stocks/AAPL/bars?'
823+
'timeframe=5Min&adjustment=raw&'
821824
'start=2021-06-08&end=2021-06-08', text='{}')
822825
api.get_bars('AAPL', timeframe, '2021-06-08', '2021-06-08')
823826
assert reqmock.called

0 commit comments

Comments
 (0)