flywithu/CryptoExchange.Net
1
0
Fork 0
mirror of https://github.com/JKorf/CryptoExchange.Net synced 2025-06-07 16:06:15 +00:00
Code Issues Releases Wiki Activity

Removed special characters in ClientOrderIdSeperator to adhere to field content rules

Browse Source
This commit is contained in:
Jkorf 2024-12-02 14:38:22 +01:00
parent 271743b669
commit fd4e8da938
2 changed files with 4 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
CryptoExchange.Net.UnitTests/ExchangeHelpersTests.cs
View File

@ -73,11 +73,11 @@ namespace CryptoExchange.Net.UnitTests
}
[Test]
[TestCase("123", "BKR", 32, true, "BKR|JK|123")]
[TestCase("123", "BKR", 32, true, "BKRJK123")]
[TestCase("123", "BKR", 32, false, "123")]
[TestCase("123123123123123123123123123123", "BKR", 32, true, "123123123123123123123123123123")] // 30
[TestCase("123123123123123123123123123", "BKR", 32, true, "123123123123123123123123123")] // 27
[TestCase("1231231231231231231231231", "BKR", 32, true, "BKR|JK|1231231231231231231231231")] // 25
[TestCase("12312312312312312312312312312", "BKR", 32, true, "12312312312312312312312312312")] // 27
[TestCase("123123123123123123123123123", "BKR", 32, true, "BKRJK123123123123123123123123123")] // 25
[TestCase(null, "BKR", 32, true, null)]
public void ApplyBrokerIdTests(string clientOrderId, string brokerId, int maxLength, bool allowValueAdjustement, string expected)
{

2
CryptoExchange.Net/LibraryHelpers.cs
View File

@ -12,7 +12,7 @@ namespace CryptoExchange.Net
/// <summary>
/// Client order id seperator
/// </summary>
public const string ClientOrderIdSeperator = "|JK|";
public const string ClientOrderIdSeperator = "JK";
/// <summary>
/// Apply broker id to a client order id