NDAX: fix Trades websocket (#534)

- there is a bug in OnGetTickersAsync() which will need to be fixed at some point
- changed OnGetMarketSymbolsMetadataAsync() to fill in _marketSymbolToInstrumentIdMapping
- changed EnsureInstrumentIdsAvailable() to use OnGetMarketSymbolsMetadataAsync() instead
- fixes OnGetTradesWebSocketAsync()

Author: vslee

src/ExchangeSharp/API/Exchanges/NDAX/ExchangeNDAXAPI.cs

@@ -60,8 +60,9 @@ protected internal override async Task<IEnumerable<ExchangeMarket>> OnGetMarketS
             var result = await MakeJsonRequestAsync<IEnumerable<Instrument>>("GetInstruments", null,
                 new Dictionary<string, object>()
                     { {"OMSId", 1}}, "POST");
-
-            return result.Select(instrument => instrument.ToExchangeMarket());
+			_marketSymbolToInstrumentIdMapping = result.ToDictionary(keySelector: instrument => instrument.Symbol.Replace("_", ""),
+				elementSelector: instrument => instrument.InstrumentId);
+			return result.Select(instrument => instrument.ToExchangeMarket());
         }
 
         protected override async Task<IEnumerable<ExchangeOrderResult>> OnGetCompletedOrderDetailsAsync(string symbol = null,
@@ -325,7 +326,7 @@ private async Task EnsureInstrumentIdsAvailable()
         {
             if (_marketSymbolToInstrumentIdMapping == null)
             {
-                await GetTickersAsync();
+				await OnGetMarketSymbolsMetadataAsync();
             }
         }
 
@@ -421,11 +422,14 @@ await socket.SendMessageAsync(new MessageFrame
 
 		protected override async Task<IWebSocket> OnGetTradesWebSocketAsync(Func<KeyValuePair<string, ExchangeTrade>, Task> callback, params string[] marketSymbols)
 		{
-			await EnsureInstrumentIdsAvailable();
-			var instrumentIds = marketSymbols == null || marketSymbols.Length == 0 ?
-				(await GetMarketSymbolsMetadataAsync()).Select(s => (long?)long.Parse(s.AltMarketSymbol)).ToArray() :
-				await GetInstrumentIdFromMarketSymbol(marketSymbols);
-
+			long?[] instrumentIds;
+			if (marketSymbols == null || marketSymbols.Length == 0)
+				instrumentIds = (await GetMarketSymbolsMetadataAsync()).Select(s => (long?)long.Parse(s.AltMarketSymbol)).ToArray();
+			else
+			{
+				await EnsureInstrumentIdsAvailable();
+				instrumentIds = await GetInstrumentIdFromMarketSymbol(marketSymbols);
+			}
 			return await ConnectWebSocketAsync("", async (socket, bytes) =>
 				{
 					var messageFrame =