standardize exception types (#735)

vslee · web-flow · commit cbdd172cdd7b · 2022-02-12T15:30:58.000-08:00
- use NotImplementedException, InvalidOperationException, NotSupportedException, ArgumentException, ArgumentNullException when appropriate
- opens up room in the future to create more specific exceptions derived from above exceptions
diff --git a/src/ExchangeSharp/API/Common/BaseAPI.cs b/src/ExchangeSharp/API/Common/BaseAPI.cs
@@ -418,7 +418,7 @@ public async Task<object> GenerateNonceAsync()
 							break;
 
 						default:
-							throw new InvalidOperationException("Invalid nonce style: " + NonceStyle);
+							throw new NotImplementedException("Invalid nonce style: " + NonceStyle);
 					}
 
 					// check for duplicate nonce
diff --git a/src/ExchangeSharp/API/Exchanges/Aquanow/ExchangeAquanowAPI.cs b/src/ExchangeSharp/API/Exchanges/Aquanow/ExchangeAquanowAPI.cs
@@ -105,7 +105,7 @@ protected override async Task ProcessRequestAsync(IHttpWebRequest request, Dicti
 		// DONE
 		protected override async Task<ExchangeOrderResult> OnPlaceOrderAsync(ExchangeOrderRequest order)
 		{
-			if (order.IsPostOnly != null) throw new NotImplementedException("Post Only orders are not supported by this exchange or not implemented in ExchangeSharp. Please submit a PR if you are interested in this feature.");
+			if (order.IsPostOnly != null) throw new NotSupportedException("Post Only orders are not supported by this exchange or not implemented in ExchangeSharp. Please submit a PR if you are interested in this feature.");
 			// In Aquanow market order, when buying crypto the amount of crypto that is bought is the receiveQuantity
 			// and when selling the amount of crypto that is sold is the deliverQuantity
 			string amountParameter = order.IsBuy ? "receiveQuantity" : "deliverQuantity";
@@ -201,7 +201,7 @@ protected override async Task<ExchangeOrderResult> OnGetOrderDetailsAsync(string
 			{
 				return null;
 			}
-			if (isClientOrderId) throw new NotImplementedException("Querying by client order ID is not implemented in ExchangeSharp. Please submit a PR if you are interested in this feature");
+			if (isClientOrderId) throw new NotSupportedException("Querying by client order ID is not implemented in ExchangeSharp. Please submit a PR if you are interested in this feature");
 			var payload = await GetNoncePayloadAsync();
 			JToken result = await MakeJsonRequestAsync<JToken>($"/trades/v1/order?orderId={orderId}", null, payload, "GET");
 			bool isBuy = result["tradeSide"].ToStringInvariant() == "buy" ? true : false;
diff --git a/src/ExchangeSharp/API/Exchanges/BL3P/ExchangeBL3PAPI.cs b/src/ExchangeSharp/API/Exchanges/BL3P/ExchangeBL3PAPI.cs
@@ -233,7 +233,7 @@ private string GetSignKey(IHttpWebRequest request, string formData)
 
 		protected override async Task<ExchangeOrderResult> OnPlaceOrderAsync(ExchangeOrderRequest order)
 		{
-			if (order.IsPostOnly != null) throw new NotImplementedException("Post Only orders are not supported by this exchange or not implemented in ExchangeSharp. Please submit a PR if you are interested in this feature.");
+			if (order.IsPostOnly != null) throw new NotSupportedException("Post Only orders are not supported by this exchange or not implemented in ExchangeSharp. Please submit a PR if you are interested in this feature.");
 			var roundedAmount = order.RoundAmount();
 			var amountInt = converterToEight.FromDecimal(roundedAmount);
 
@@ -314,7 +314,7 @@ protected override async Task<ExchangeOrderResult> OnGetOrderDetailsAsync(string
 		{
 			if (string.IsNullOrWhiteSpace(marketSymbol))
 				throw new ArgumentException("Value cannot be null or whitespace.", nameof(marketSymbol));
-			if (isClientOrderId) throw new NotImplementedException("Querying by client order ID is not implemented in ExchangeSharp. Please submit a PR if you are interested in this feature");
+			if (isClientOrderId) throw new NotSupportedException("Querying by client order ID is not implemented in ExchangeSharp. Please submit a PR if you are interested in this feature");
 
 			var data = new Dictionary<string, object>
 			{
diff --git a/src/ExchangeSharp/API/Exchanges/BinanceGroup/BinanceGroupCommon.cs b/src/ExchangeSharp/API/Exchanges/BinanceGroup/BinanceGroupCommon.cs
@@ -554,7 +554,7 @@ protected override async Task<ExchangeOrderResult> OnPlaceOrderAsync(ExchangeOrd
 			else if (order.IsPostOnly == true)
 			{
 				if (order.OrderType == OrderType.Limit)	payload["type"] = "LIMIT_MAKER"; // LIMIT_MAKER are LIMIT orders that will be rejected if they would immediately match and trade as a taker.
-				else throw new NotImplementedException("PostOnly with non limit orders are not currently supported on Binance. Please submit a PR if you are interested in this feature");
+				else throw new NotSupportedException("PostOnly with non limit orders are not currently supported on Binance. Please submit a PR if you are interested in this feature");
 			}
 			else
 				payload["type"] = order.OrderType.ToStringUpperInvariant();
@@ -587,7 +587,7 @@ protected override async Task<ExchangeOrderResult> OnGetOrderDetailsAsync(string
 			Dictionary<string, object> payload = await GetNoncePayloadAsync();
 			if (string.IsNullOrWhiteSpace(marketSymbol))
 			{
-				throw new InvalidOperationException("Binance single order details request requires symbol");
+				throw new ArgumentNullException("Binance single order details request requires symbol");
 			}
 			payload["symbol"] = marketSymbol!;
 
diff --git a/src/ExchangeSharp/API/Exchanges/BitBank/ExchangeBitBankAPI.cs b/src/ExchangeSharp/API/Exchanges/BitBank/ExchangeBitBankAPI.cs
@@ -128,7 +128,7 @@ protected override Task OnGetHistoricalTradesAsync(Func<IEnumerable<ExchangeTrad
 		protected override async Task<ExchangeOrderResult> OnPlaceOrderAsync(ExchangeOrderRequest order)
 		{
 			if (order.OrderType == OrderType.Stop)
-				throw new InvalidOperationException("Bitbank does not support stop order");
+				throw new NotSupportedException("Bitbank does not support stop order");
 			Dictionary<string, object> payload = await GetNoncePayloadAsync();
 			if (order.IsPostOnly != null)
 				payload["post_only"] = order.IsPostOnly;
@@ -154,11 +154,11 @@ protected override async Task OnCancelOrderAsync(string orderId, string marketSy
 
 		protected override async Task<ExchangeOrderResult> OnGetOrderDetailsAsync(string orderId, string marketSymbol = null, bool isClientOrderId = false)
 		{
-			if (isClientOrderId) throw new NotImplementedException("Querying by client order ID is not implemented in ExchangeSharp. Please submit a PR if you are interested in this feature");
+			if (isClientOrderId) throw new NotSupportedException("Querying by client order ID is not implemented in ExchangeSharp. Please submit a PR if you are interested in this feature");
 			var payload = await GetNoncePayloadAsync();
 			payload.Add("order_id", orderId);
-			if (marketSymbol == null)
-				throw new InvalidOperationException($"BitBank API requires marketSymbol for {nameof(GetOrderDetailsAsync)}");
+			if (string.IsNullOrWhiteSpace(marketSymbol))
+				throw new ArgumentNullException($"BitBank API requires marketSymbol for {nameof(GetOrderDetailsAsync)}");
 			payload.Add("pair", marketSymbol);
 			JToken token = await MakeJsonRequestAsync<JToken>("/user/spot/order", baseUrl: BaseUrlPrivate, payload: payload);
 			return ParseOrder(token);
diff --git a/src/ExchangeSharp/API/Exchanges/Bitfinex/ExchangeBitfinexAPI.cs b/src/ExchangeSharp/API/Exchanges/Bitfinex/ExchangeBitfinexAPI.cs
@@ -504,7 +504,7 @@ protected override async Task<ExchangeOrderResult> OnPlaceOrderAsync(ExchangeOrd
 
 		protected override async Task<ExchangeOrderResult> OnGetOrderDetailsAsync(string orderId, string marketSymbol = null, bool isClientOrderId = false)
 		{
-			if (isClientOrderId) throw new NotImplementedException("Querying by client order ID is not implemented in ExchangeSharp. Please submit a PR if you are interested in this feature");
+			if (isClientOrderId) throw new NotSupportedException("Querying by client order ID is not implemented in ExchangeSharp. Please submit a PR if you are interested in this feature");
 			if (string.IsNullOrWhiteSpace(orderId))
 			{
 				return null;
diff --git a/src/ExchangeSharp/API/Exchanges/Bitstamp/ExchangeBitstampAPI.cs b/src/ExchangeSharp/API/Exchanges/Bitstamp/ExchangeBitstampAPI.cs
@@ -195,7 +195,7 @@ private static Dictionary<string, decimal> ExtractDictionary(JObject responseObj
 
         protected override async Task<ExchangeOrderResult> OnPlaceOrderAsync(ExchangeOrderRequest order)
         {
-			if (order.IsPostOnly != null) throw new NotImplementedException("Post Only orders are not supported by this exchange or not implemented in ExchangeSharp. Please submit a PR if you are interested in this feature.");
+			if (order.IsPostOnly != null) throw new NotSupportedException("Post Only orders are not supported by this exchange or not implemented in ExchangeSharp. Please submit a PR if you are interested in this feature.");
 			string action = order.IsBuy ? "buy" : "sell";
             string market = order.OrderType == OrderType.Market ? "/market" : "";
             string url = $"/{action}{market}/{order.MarketSymbol}/";
diff --git a/src/ExchangeSharp/API/Exchanges/Bittrex/ExchangeBittrexAPI.cs b/src/ExchangeSharp/API/Exchanges/Bittrex/ExchangeBittrexAPI.cs
@@ -415,7 +415,7 @@ protected override async Task<ExchangeOrderResult> OnPlaceOrderAsync(ExchangeOrd
 
 		protected override async Task<ExchangeOrderResult> OnGetOrderDetailsAsync(string orderId, string marketSymbol = null, bool isClientOrderId = false)
 		{
-			if (isClientOrderId) throw new NotImplementedException("Querying by client order ID is not implemented in ExchangeSharp. Please submit a PR if you are interested in this feature");
+			if (isClientOrderId) throw new NotSupportedException("Querying by client order ID is not implemented in ExchangeSharp. Please submit a PR if you are interested in this feature");
 			if (string.IsNullOrWhiteSpace(orderId))
 			{
 				return null;
diff --git a/src/ExchangeSharp/API/Exchanges/Bybit/ExchangeBybitAPI.cs b/src/ExchangeSharp/API/Exchanges/Bybit/ExchangeBybitAPI.cs
@@ -819,7 +819,7 @@ protected override async Task<ExchangeOrderResult> OnPlaceOrderAsync(ExchangeOrd
 
 		public async Task<ExchangeOrderResult> OnAmendOrderAsync(ExchangeOrderRequest order)
 		{
-			if (order.IsPostOnly != null) throw new NotImplementedException("Post Only orders are not supported by this exchange or not implemented in ExchangeSharp. Please submit a PR if you are interested in this feature.");
+			if (order.IsPostOnly != null) throw new NotSupportedException("Post Only orders are not supported by this exchange or not implemented in ExchangeSharp. Please submit a PR if you are interested in this feature.");
 			var payload = new Dictionary<string, object>();
 			payload["symbol"] = order.MarketSymbol;
 			if(order.OrderId != null)
diff --git a/src/ExchangeSharp/API/Exchanges/Coinbase/ExchangeCoinbaseAPI.cs b/src/ExchangeSharp/API/Exchanges/Coinbase/ExchangeCoinbaseAPI.cs
@@ -741,7 +741,10 @@ protected override async Task<ExchangeOrderResult> OnGetOrderDetailsAsync(string
 		{ // Orders may be queried using either the exchange assigned id or the client assigned client_oid. When using client_oid it must be preceded by the client: namespace.
 			JToken obj = await MakeJsonRequestAsync<JToken>("/orders/" + (isClientOrderId ? "client:" : "") + orderId,
 				null, await GetNoncePayloadAsync(), "GET");
-			return ParseOrder(obj);
+			var order = ParseOrder(obj);
+			if (!order.MarketSymbol.Equals(marketSymbol, StringComparison.InvariantCultureIgnoreCase))
+				throw new DataMisalignedException($"Order {orderId} found, but symbols {order.MarketSymbol} and {marketSymbol} don't match");
+			else return order;
 		}
 
 		protected override async Task<IEnumerable<ExchangeOrderResult>> OnGetOpenOrderDetailsAsync(string marketSymbol = null)
diff --git a/src/ExchangeSharp/API/Exchanges/Coinmate/ExchangeCoinmateAPI.cs b/src/ExchangeSharp/API/Exchanges/Coinmate/ExchangeCoinmateAPI.cs
@@ -165,7 +165,7 @@ protected override async Task<ExchangeOrderResult> OnPlaceOrderAsync(ExchangeOrd
 
 			if (order.OrderType != OrderType.Limit && order.OrderType != OrderType.Stop)
 			{
-				throw new NotImplementedException("This type of order is currently not supported.");
+				throw new NotSupportedException("This type of order is currently not supported.");
 			}
 
 			payload["amount"] = order.Amount;
@@ -182,7 +182,7 @@ protected override async Task<ExchangeOrderResult> OnPlaceOrderAsync(ExchangeOrd
 			{
 				if (!long.TryParse(order.ClientOrderId, out var clientOrderId))
 				{
-					throw new InvalidOperationException("ClientId must be numerical for Coinmate");
+					throw new ArgumentException("ClientId must be numerical for Coinmate");
 				}
 
 				payload["clientOrderId"] = clientOrderId;
diff --git a/src/ExchangeSharp/API/Exchanges/Digifinex/ExchangeDigifinexAPI.cs b/src/ExchangeSharp/API/Exchanges/Digifinex/ExchangeDigifinexAPI.cs
@@ -366,7 +366,7 @@ protected override async Task<IEnumerable<ExchangeOrderResult>> OnGetCompletedOr
 
 		protected override async Task<ExchangeOrderResult> OnGetOrderDetailsAsync(string orderId, string marketSymbol = null, bool isClientOrderId = false)
 		{
-			if (isClientOrderId) throw new NotImplementedException("Querying by client order ID is not implemented in ExchangeSharp. Please submit a PR if you are interested in this feature");
+			if (isClientOrderId) throw new NotSupportedException("Querying by client order ID is not implemented in ExchangeSharp. Please submit a PR if you are interested in this feature");
 			Dictionary<string, object> payload = await GetNoncePayloadAsync();
 			JToken token = await MakeJsonRequestAsync<JToken>($"/spot/order?order_id={orderId}", payload: payload);
 			var x = token["data"];
diff --git a/src/ExchangeSharp/API/Exchanges/FTX/FTXGroupCommon.cs b/src/ExchangeSharp/API/Exchanges/FTX/FTXGroupCommon.cs
@@ -291,7 +291,7 @@ protected async override Task<ExchangeOrderBook> OnGetOrderBookAsync(string mark
 		/// <inheritdoc />
 		protected async override Task<ExchangeOrderResult> OnGetOrderDetailsAsync(string orderId, string marketSymbol = null, bool isClientOrderId = false)
 		{ // https://docs.ftx.com/#get-order-status and https://docs.ftx.com/#get-order-status-by-client-id
-			if (!string.IsNullOrEmpty(marketSymbol)) throw new NotImplementedException("Searching by marketSymbol is either not implemented by or supported by this exchange. Please submit a PR if you are interested in this feature");
+			if (!string.IsNullOrEmpty(marketSymbol)) throw new NotSupportedException("Searching by marketSymbol is either not implemented by or supported by this exchange. Please submit a PR if you are interested in this feature");
 
 			var url = "/orders/";
 			if (isClientOrderId)
diff --git a/src/ExchangeSharp/API/Exchanges/GateIo/ExchangeGateIoAPI.cs b/src/ExchangeSharp/API/Exchanges/GateIo/ExchangeGateIoAPI.cs
@@ -293,7 +293,7 @@ protected override async Task<Dictionary<string, decimal>> OnGetAmountsAvailable
 		protected override async Task<ExchangeOrderResult> OnPlaceOrderAsync(ExchangeOrderRequest order)
 		{
 			if (order.OrderType != OrderType.Limit)
-				throw new InvalidOperationException("Gate.io API supports only limit orders");
+				throw new NotSupportedException("Gate.io API supports only limit orders");
 
 			var payload = await GetNoncePayloadAsync();
 			AddOrderToPayload(order, payload);
@@ -380,11 +380,11 @@ private static ExchangeAPIOrderResult ParseExchangeAPIOrderResult(string status,
 
 		protected override async Task<ExchangeOrderResult> OnGetOrderDetailsAsync(string orderId, string symbol = null, bool isClientOrderId = false)
 		{
-			if (string.IsNullOrEmpty(symbol))
+			if (string.IsNullOrWhiteSpace(symbol))
 			{
-				throw new InvalidOperationException("MarketSymbol is required for querying order details with Gate.io API");
+				throw new ArgumentNullException("MarketSymbol is required for querying order details with Gate.io API");
 			}
-			if (isClientOrderId) throw new NotImplementedException("Querying by client order ID is not implemented in ExchangeSharp. Please submit a PR if you are interested in this feature");
+			if (isClientOrderId) throw new NotSupportedException("Querying by client order ID is not implemented in ExchangeSharp. Please submit a PR if you are interested in this feature");
 
 			var payload = await GetNoncePayloadAsync();
 			var responseToken = await MakeJsonRequestAsync<JToken>($"/spot/orders/{orderId}?currency_pair={symbol}", payload: payload);
@@ -394,9 +394,9 @@ protected override async Task<ExchangeOrderResult> OnGetOrderDetailsAsync(string
 
 		protected override async Task<IEnumerable<ExchangeOrderResult>> OnGetOpenOrderDetailsAsync(string symbol = null)
 		{
-			if (string.IsNullOrEmpty(symbol))
+			if (string.IsNullOrWhiteSpace(symbol))
 			{
-				throw new InvalidOperationException("MarketSymbol is required for querying open orders with Gate.io API");
+				throw new ArgumentNullException("MarketSymbol is required for querying open orders with Gate.io API");
 			}
 
 			var payload = await GetNoncePayloadAsync();
@@ -423,9 +423,9 @@ protected override async Task<IEnumerable<ExchangeOrderResult>> OnGetCompletedOr
 
 		protected override async Task OnCancelOrderAsync(string orderId, string symbol = null, bool isClientOrderId = false)
 		{
-			if (string.IsNullOrEmpty(symbol))
+			if (string.IsNullOrWhiteSpace(symbol))
 			{
-				throw new InvalidOperationException("MarketSymbol is required for cancelling order with Gate.io API");
+				throw new ArgumentNullException("MarketSymbol is required for cancelling order with Gate.io API");
 			}
 			if (isClientOrderId) throw new NotSupportedException("Cancelling by client order ID is not supported in ExchangeSharp. Please submit a PR if you are interested in this feature");
 
diff --git a/src/ExchangeSharp/API/Exchanges/Hitbtc/ExchangeHitbtcAPI.cs b/src/ExchangeSharp/API/Exchanges/Hitbtc/ExchangeHitbtcAPI.cs
@@ -259,7 +259,7 @@ protected override async Task<Dictionary<string, decimal>> OnGetAmountsAvailable
         /// <returns></returns>
         protected override async Task<ExchangeOrderResult> OnGetOrderDetailsAsync(string orderId, string marketSymbol = null, bool isClientOrderId = false)
 		{
-			if (isClientOrderId) throw new NotImplementedException("Querying by client order ID is not implemented in ExchangeSharp. Please submit a PR if you are interested in this feature");
+			if (isClientOrderId) throw new NotSupportedException("Querying by client order ID is not implemented in ExchangeSharp. Please submit a PR if you are interested in this feature");
 			JToken obj = await MakeJsonRequestAsync<JToken>("/history/order/" + orderId + "/trades", null, await GetNoncePayloadAsync());
             if (obj != null && obj.HasValues) return ParseCompletedOrder(obj);
             return null;
diff --git a/src/ExchangeSharp/API/Exchanges/Kraken/ExchangeKrakenAPI.cs b/src/ExchangeSharp/API/Exchanges/Kraken/ExchangeKrakenAPI.cs
@@ -715,11 +715,10 @@ protected override async Task<ExchangeOrderResult> OnGetOrderDetailsAsync(string
 			{
 				return null;
 			}
-			if (isClientOrderId) throw new NotImplementedException("Querying by client order ID is not implemented in ExchangeSharp. Please submit a PR if you are interested in this feature");
+			if (isClientOrderId) throw new NotSupportedException("Querying by client order ID is not implemented in ExchangeSharp. Please submit a PR if you are interested in this feature");
 			object nonce = await GenerateNonceAsync();
 			Dictionary<string, object> payload = new Dictionary<string, object>(StringComparer.OrdinalIgnoreCase)
-			{ { "txid", orderId }, { "nonce", nonce }
-			};
+			{ { "txid", orderId }, { "nonce", nonce } };
 			JToken result = await MakeJsonRequestAsync<JToken>("/0/private/QueryOrders", null, payload);
 			ExchangeOrderResult orderResult = new ExchangeOrderResult { OrderId = orderId };
 			if (result == null || result[orderId] == null)
diff --git a/src/ExchangeSharp/API/Exchanges/KuCoin/ExchangeKuCoinAPI.cs b/src/ExchangeSharp/API/Exchanges/KuCoin/ExchangeKuCoinAPI.cs
@@ -381,13 +381,10 @@ protected override async Task<IEnumerable<ExchangeOrderResult>> OnGetOpenOrderDe
         protected override async Task<ExchangeOrderResult> OnGetOrderDetailsAsync(string orderId, string marketSymbol = null, bool isClientOrderId = false)
 		{
 			var payload = await GetNoncePayloadAsync();
-			if (isClientOrderId)
-				payload["clientOid"] = Guid.NewGuid();
-			else
-				payload["clientOid"] = Guid.NewGuid();
-
-			JToken token = await MakeJsonRequestAsync<JToken>("/orders?&" + CryptoUtility.GetFormForPayload(payload, false), null, payload);
-			var isActive = token["id"].ToObject<bool>();
+			JToken token = await MakeJsonRequestAsync<JToken>(
+				   $"/orders{(isClientOrderId ? "/client-order" : null)}/{orderId}"
+				   + CryptoUtility.GetFormForPayload(payload, false), null, payload);
+			var isActive = token["isActive"].ToObject<bool>();
 			if (isActive)
 				return ParseOpenOrder(token);
 			else return ParseCompletedOrder(token);
diff --git a/src/ExchangeSharp/API/Exchanges/LBank/ExchangeLBankAPI.cs b/src/ExchangeSharp/API/Exchanges/LBank/ExchangeLBankAPI.cs
@@ -297,7 +297,7 @@ protected override async Task<Dictionary<string, decimal>> OnGetAmountsAsync()
 		//PlaceOrder   9
 		protected override async Task<ExchangeOrderResult> OnPlaceOrderAsync(ExchangeOrderRequest order)
 		{
-			if (order.IsPostOnly != null) throw new NotImplementedException("Post Only orders are not supported by this exchange or not implemented in ExchangeSharp. Please submit a PR if you are interested in this feature.");
+			if (order.IsPostOnly != null) throw new NotSupportedException("Post Only orders are not supported by this exchange or not implemented in ExchangeSharp. Please submit a PR if you are interested in this feature.");
 			Dictionary<string, object> payload = new Dictionary<string, object>
 		   {
 				{ "amount", order.Amount },
@@ -361,7 +361,7 @@ protected override async Task OnCancelOrderAsync(string orderId, string symbol =
 		//GetOrderDetails   13
 		protected override async Task<ExchangeOrderResult> OnGetOrderDetailsAsync(string orderId, string symbol = null, bool isClientOrderId = false)
 		{
-			if (isClientOrderId) throw new NotImplementedException("Querying by client order ID is not implemented in ExchangeSharp. Please submit a PR if you are interested in this feature");
+			if (isClientOrderId) throw new NotSupportedException("Querying by client order ID is not implemented in ExchangeSharp. Please submit a PR if you are interested in this feature");
 			Dictionary<string, object> payload = new Dictionary<string, object>
 			{
 				{ "api_key", PublicApiKey.ToUnsecureString() },
diff --git a/src/ExchangeSharp/API/Exchanges/Livecoin/ExchangeLivecoinAPI.cs b/src/ExchangeSharp/API/Exchanges/Livecoin/ExchangeLivecoinAPI.cs
diff --git a/src/ExchangeSharp/API/Exchanges/NDAX/ExchangeNDAXAPI.cs b/src/ExchangeSharp/API/Exchanges/NDAX/ExchangeNDAXAPI.cs
diff --git a/src/ExchangeSharp/API/Exchanges/OKGroup/OKGroupCommon.cs b/src/ExchangeSharp/API/Exchanges/OKGroup/OKGroupCommon.cs
diff --git a/src/ExchangeSharp/API/Exchanges/Poloniex/ExchangePoloniexAPI.cs b/src/ExchangeSharp/API/Exchanges/Poloniex/ExchangePoloniexAPI.cs
diff --git a/src/ExchangeSharp/API/Exchanges/Yobit/ExchangeYobitAPI.cs b/src/ExchangeSharp/API/Exchanges/Yobit/ExchangeYobitAPI.cs
diff --git a/src/ExchangeSharp/API/Exchanges/_Base/ExchangeAPI.cs b/src/ExchangeSharp/API/Exchanges/_Base/ExchangeAPI.cs

Original file line number	Diff line number	Diff line change
`@@ -418,7 +418,7 @@ public async Task<object> GenerateNonceAsync()`
`418`	`418`	`break;`
`419`	`419`
`420`	`420`	`default:`
`421`		`- throw new InvalidOperationException("Invalid nonce style: " + NonceStyle);`
	`421`	`+ throw new NotImplementedException("Invalid nonce style: " + NonceStyle);`
`422`	`422`	`}`
`423`	`423`
`424`	`424`	`// check for duplicate nonce`
Original file line number	Diff line number	Diff line change
`@@ -105,7 +105,7 @@ protected override async Task ProcessRequestAsync(IHttpWebRequest request, Dicti`
`105`	`105`	`// DONE`
`106`	`106`	`protected override async Task<ExchangeOrderResult> OnPlaceOrderAsync(ExchangeOrderRequest order)`
`107`	`107`	`{`
`108`		`- if (order.IsPostOnly != null) throw new NotImplementedException("Post Only orders are not supported by this exchange or not implemented in ExchangeSharp. Please submit a PR if you are interested in this feature.");`
	`108`	`+ if (order.IsPostOnly != null) throw new NotSupportedException("Post Only orders are not supported by this exchange or not implemented in ExchangeSharp. Please submit a PR if you are interested in this feature.");`
`109`	`109`	`// In Aquanow market order, when buying crypto the amount of crypto that is bought is the receiveQuantity`
`110`	`110`	`// and when selling the amount of crypto that is sold is the deliverQuantity`
`111`	`111`	`string amountParameter = order.IsBuy ? "receiveQuantity" : "deliverQuantity";`
`@@ -201,7 +201,7 @@ protected override async Task<ExchangeOrderResult> OnGetOrderDetailsAsync(string`
`201`	`201`	`{`
`202`	`202`	`return null;`
`203`	`203`	`}`
`204`		`- if (isClientOrderId) throw new NotImplementedException("Querying by client order ID is not implemented in ExchangeSharp. Please submit a PR if you are interested in this feature");`
	`204`	`+ if (isClientOrderId) throw new NotSupportedException("Querying by client order ID is not implemented in ExchangeSharp. Please submit a PR if you are interested in this feature");`
`205`	`205`	`var payload = await GetNoncePayloadAsync();`
`206`	`206`	`JToken result = await MakeJsonRequestAsync<JToken>($"/trades/v1/order?orderId={orderId}", null, payload, "GET");`
`207`	`207`	`bool isBuy = result["tradeSide"].ToStringInvariant() == "buy" ? true : false;`
Original file line number	Diff line number	Diff line change
`@@ -233,7 +233,7 @@ private string GetSignKey(IHttpWebRequest request, string formData)`
`233`	`233`
`234`	`234`	`protected override async Task<ExchangeOrderResult> OnPlaceOrderAsync(ExchangeOrderRequest order)`
`235`	`235`	`{`
`236`		`- if (order.IsPostOnly != null) throw new NotImplementedException("Post Only orders are not supported by this exchange or not implemented in ExchangeSharp. Please submit a PR if you are interested in this feature.");`
	`236`	`+ if (order.IsPostOnly != null) throw new NotSupportedException("Post Only orders are not supported by this exchange or not implemented in ExchangeSharp. Please submit a PR if you are interested in this feature.");`
`237`	`237`	`var roundedAmount = order.RoundAmount();`
`238`	`238`	`var amountInt = converterToEight.FromDecimal(roundedAmount);`
`239`	`239`
`@@ -314,7 +314,7 @@ protected override async Task<ExchangeOrderResult> OnGetOrderDetailsAsync(string`
`314`	`314`	`{`
`315`	`315`	`if (string.IsNullOrWhiteSpace(marketSymbol))`
`316`	`316`	`throw new ArgumentException("Value cannot be null or whitespace.", nameof(marketSymbol));`
`317`		`- if (isClientOrderId) throw new NotImplementedException("Querying by client order ID is not implemented in ExchangeSharp. Please submit a PR if you are interested in this feature");`
	`317`	`+ if (isClientOrderId) throw new NotSupportedException("Querying by client order ID is not implemented in ExchangeSharp. Please submit a PR if you are interested in this feature");`
`318`	`318`
`319`	`319`	`var data = new Dictionary<string, object>`
`320`	`320`	`{`
Original file line number	Diff line number	Diff line change
`@@ -554,7 +554,7 @@ protected override async Task<ExchangeOrderResult> OnPlaceOrderAsync(ExchangeOrd`
`554`	`554`	`else if (order.IsPostOnly == true)`
`555`	`555`	`{`
`556`	`556`	`if (order.OrderType == OrderType.Limit) payload["type"] = "LIMIT_MAKER"; // LIMIT_MAKER are LIMIT orders that will be rejected if they would immediately match and trade as a taker.`
`557`		`- else throw new NotImplementedException("PostOnly with non limit orders are not currently supported on Binance. Please submit a PR if you are interested in this feature");`
	`557`	`+ else throw new NotSupportedException("PostOnly with non limit orders are not currently supported on Binance. Please submit a PR if you are interested in this feature");`
`558`	`558`	`}`
`559`	`559`	`else`
`560`	`560`	`payload["type"] = order.OrderType.ToStringUpperInvariant();`
`@@ -587,7 +587,7 @@ protected override async Task<ExchangeOrderResult> OnGetOrderDetailsAsync(string`
`587`	`587`	`Dictionary<string, object> payload = await GetNoncePayloadAsync();`
`588`	`588`	`if (string.IsNullOrWhiteSpace(marketSymbol))`
`589`	`589`	`{`
`590`		`- throw new InvalidOperationException("Binance single order details request requires symbol");`
	`590`	`+ throw new ArgumentNullException("Binance single order details request requires symbol");`
`591`	`591`	`}`
`592`	`592`	`payload["symbol"] = marketSymbol!;`
`593`	`593`
Original file line number	Diff line number	Diff line change
`@@ -504,7 +504,7 @@ protected override async Task<ExchangeOrderResult> OnPlaceOrderAsync(ExchangeOrd`
`504`	`504`
`505`	`505`	`protected override async Task<ExchangeOrderResult> OnGetOrderDetailsAsync(string orderId, string marketSymbol = null, bool isClientOrderId = false)`
`506`	`506`	`{`
`507`		`- if (isClientOrderId) throw new NotImplementedException("Querying by client order ID is not implemented in ExchangeSharp. Please submit a PR if you are interested in this feature");`
	`507`	`+ if (isClientOrderId) throw new NotSupportedException("Querying by client order ID is not implemented in ExchangeSharp. Please submit a PR if you are interested in this feature");`
`508`	`508`	`if (string.IsNullOrWhiteSpace(orderId))`
`509`	`509`	`{`
`510`	`510`	`return null;`
Original file line number	Diff line number	Diff line change
`@@ -195,7 +195,7 @@ private static Dictionary<string, decimal> ExtractDictionary(JObject responseObj`
`195`	`195`
`196`	`196`	`protected override async Task<ExchangeOrderResult> OnPlaceOrderAsync(ExchangeOrderRequest order)`
`197`	`197`	`{`
`198`		`- if (order.IsPostOnly != null) throw new NotImplementedException("Post Only orders are not supported by this exchange or not implemented in ExchangeSharp. Please submit a PR if you are interested in this feature.");`
	`198`	`+ if (order.IsPostOnly != null) throw new NotSupportedException("Post Only orders are not supported by this exchange or not implemented in ExchangeSharp. Please submit a PR if you are interested in this feature.");`
`199`	`199`	`string action = order.IsBuy ? "buy" : "sell";`
`200`	`200`	`string market = order.OrderType == OrderType.Market ? "/market" : "";`
`201`	`201`	`string url = $"/{action}{market}/{order.MarketSymbol}/";`
Original file line number	Diff line number	Diff line change
`@@ -415,7 +415,7 @@ protected override async Task<ExchangeOrderResult> OnPlaceOrderAsync(ExchangeOrd`
`415`	`415`
`416`	`416`	`protected override async Task<ExchangeOrderResult> OnGetOrderDetailsAsync(string orderId, string marketSymbol = null, bool isClientOrderId = false)`
`417`	`417`	`{`
`418`		`- if (isClientOrderId) throw new NotImplementedException("Querying by client order ID is not implemented in ExchangeSharp. Please submit a PR if you are interested in this feature");`
	`418`	`+ if (isClientOrderId) throw new NotSupportedException("Querying by client order ID is not implemented in ExchangeSharp. Please submit a PR if you are interested in this feature");`
`419`	`419`	`if (string.IsNullOrWhiteSpace(orderId))`
`420`	`420`	`{`
`421`	`421`	`return null;`
Original file line number	Diff line number	Diff line change
`@@ -819,7 +819,7 @@ protected override async Task<ExchangeOrderResult> OnPlaceOrderAsync(ExchangeOrd`
`819`	`819`
`820`	`820`	`public async Task<ExchangeOrderResult> OnAmendOrderAsync(ExchangeOrderRequest order)`
`821`	`821`	`{`
`822`		`- if (order.IsPostOnly != null) throw new NotImplementedException("Post Only orders are not supported by this exchange or not implemented in ExchangeSharp. Please submit a PR if you are interested in this feature.");`
	`822`	`+ if (order.IsPostOnly != null) throw new NotSupportedException("Post Only orders are not supported by this exchange or not implemented in ExchangeSharp. Please submit a PR if you are interested in this feature.");`
`823`	`823`	`var payload = new Dictionary<string, object>();`
`824`	`824`	`payload["symbol"] = order.MarketSymbol;`
`825`	`825`	`if(order.OrderId != null)`