flywithu/CryptoExchange.Net
1
0
Fork 0
mirror of https://github.com/JKorf/CryptoExchange.Net.git synced 2026-08-11 08:22:53 +00:00
Code Issues Releases Wiki Activity

Fixed bug in bool converter

Browse Source
This commit is contained in:
Jkorf
2026-06-29 11:20:06 +02:00
parent f4736bcf49
commit dd1cefdc90
2 changed files with 12 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files
CryptoExchange.Net.UnitTests/ConverterTests/BoolConverterTests.cs
+11
View File
@@ -25,6 +25,17 @@ namespace CryptoExchange.Net.UnitTests.ConverterTests
Assert.That(output!.Value == expected); Assert.That(output!.Value == expected);
} }
[TestCase(1, true)]
[TestCase(2, true)]
[TestCase(0, false)]
[TestCase(-1, false)]
public void TestBoolConverterInts(int value, bool? expected)
{
var val = value == null ? "null" : $"{value}";
var output = JsonSerializer.Deserialize<STJBoolObject>($"{{ \"Value\": {val} }}", SerializerOptions.WithConverters(new TestSerializerContext()));
Assert.That(output!.Value == expected);
}
[TestCase("1", true)] [TestCase("1", true)]
[TestCase("true", true)] [TestCase("true", true)]
[TestCase("yes", true)] [TestCase("yes", true)]
CryptoExchange.Net/Converters/SystemTextJson/BoolConverter.cs
+1 -1
View File
@@ -59,7 +59,7 @@ namespace CryptoExchange.Net.Converters.SystemTextJson
if (reader.TokenType == JsonTokenType.Number) if (reader.TokenType == JsonTokenType.Number)
{ {
var number = reader.GetInt16(); var number = reader.GetInt16();
if (number > 1) if (number >= 1)
return true; return true;
return false; return false;