Closed gloriousCode closed 1 month ago
Attention: Patch coverage is 73.60406%
with 52 lines
in your changes missing coverage. Please review.
Project coverage is 38.20%. Comparing base (
c37a115
) to head (bd5d4c9
). Report is 5 commits behind head on master.:exclamation: Current head bd5d4c9 differs from pull request most recent head 171915c
Please upload reports for the commit 171915c to get more accurate results.
PR Description
These are the kinds of bugs that can only pop up after proper use, it was still an excellent PR Sam 🕊️
This PR does the following:
UpdateTradablePairs
utilising concurrency since its so slow. Reducesgo test -race -count=0
from28s
to20s
GetCurrencyTradeURL
IsPerpetualFuture
optionPairToString
and add test coverageguessAssetTypeFromInstrument
togetAssetFromPair
as its not guessinggetAssetPairByInstrument
to help parse WS pair data after getting some errorsany
subscriptions (eg trades) usinggetAssetPairByInstrument
any
can have multiple currencies in the slice/ the currency isn't in the channel nameTestGetHistoricTrades
to not check results, because sometimes there are no trades, but its an expensive test to go back further in time to ensure trades. If there is an issue, deribit will error.I made sure that
getAssetPairByInstrument
was performant before using itType of change
How has this been tested
TestOptionPairToString
GetCurrencyTradeURL
Checklist