asulwer / RulesEngine / 9762497107

93%

push

github

Add JSON Element Handling for System.Text.Json Compatibility (#37)

* The primary reason for this change is the difference in how System.Text.Json handles ExpandoObject compared to Newtonsoft.Json. When using System.Text.Json, properties within an ExpandoObject are deserialized as JsonElement rather than their respective .NET types (e.g., string, int). This behavior can lead to issues where the compiler cannot work with the dynamically deserialized properties directly.

* The primary reason for this change is the difference in how System.Text.Json handles ExpandoObject compared to Newtonsoft.Json. When using System.Text.Json, properties within an ExpandoObject are deserialized as JsonElement rather than their respective .NET types (e.g., string, int). This behavior can lead to issues where the compiler cannot work with the dynamically deserialized properties directly.

* fix unit testing

* The 'l' suffix is easily confused with the digit '1' -- use 'L' for clarity

---------

Co-authored-by: Renan Pereira <rpereira@glasslewis.com>

321 of 378 branches covered (84.92%)

Branch coverage included in aggregate %.

28 of 29 new or added lines in 2 files covered. (96.55%)

673 of 694 relevant lines covered (96.97%)

113.31 hits per line