added checks during cancel Order (#683)

- check to make sure the correct order is being cancelled on Binance and Coinbase
- pass through OrderRejectReason in Binance UserDataStream

Author: vslee

src/ExchangeSharp/API/Exchanges/BinanceGroup/BinanceGroupCommon.cs

@@ -724,11 +724,14 @@ protected override async Task OnCancelOrderAsync(string orderId, string? marketS
 			Dictionary<string, object> payload = await GetNoncePayloadAsync();
 			if (string.IsNullOrWhiteSpace(marketSymbol))
 			{
-				throw new InvalidOperationException("Binance cancel order request requires symbol");
+				throw new ArgumentNullException("Binance cancel order request requires symbol");
 			}
 			payload["symbol"] = marketSymbol!;
 			payload["orderId"] = orderId;
-            _ = await MakeJsonRequestAsync<JToken>("/order", BaseUrlApi, payload, "DELETE");
+            var token = await MakeJsonRequestAsync<JToken>("/order", BaseUrlApi, payload, "DELETE");
+			var cancelledOrder = ParseOrder(token);
+			if (cancelledOrder.OrderId != orderId)
+				throw new APIException($"Cancelled {cancelledOrder.OrderId} when trying to cancel {orderId}");
 		}
 
 		/// <summary>A withdrawal request. Fee is automatically subtracted from the amount.</summary>

src/ExchangeSharp/API/Exchanges/BinanceGroup/Models/UserDataStream.cs

@@ -87,7 +87,7 @@ public ExchangeOrderResult ExchangeOrderResult
 					ClientOrderId = ClientOrderId,
 					Result = status,
 					ResultCode = CurrentOrderStatus,
-					Message = null, // can use for something in the future if needed
+					Message = OrderRejectReason, // can use for multiple things in the future if needed
 					Amount = CumulativeFilledQuantity,
 					Price = OrderPrice,
 					AveragePrice = CumulativeQuoteAssetTransactedQuantity / CumulativeFilledQuantity, // Average price can be found by doing Z divided by z.

src/ExchangeSharp/API/Exchanges/Coinbase/ExchangeCoinbaseAPI.cs

@@ -786,7 +786,9 @@ private async Task MakeFillRequest(DateTime? afterDate, string productId, List<E
 
 		protected override async Task OnCancelOrderAsync(string orderId, string marketSymbol = null)
 		{
-			await MakeJsonRequestAsync<JArray>("orders/" + orderId, null, await GetNoncePayloadAsync(), "DELETE");
+			var jToken = await MakeJsonRequestAsync<JToken>("orders/" + orderId, null, await GetNoncePayloadAsync(), "DELETE");
+			if (jToken.ToStringInvariant() != orderId)
+				throw new APIException($"Cancelled {jToken.ToStringInvariant()} when trying to cancel {orderId}");
 		}
 	}