Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Issue2632 #2633

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Issue2632 #2633

wants to merge 3 commits into from

Commits on Jan 2, 2022

  1. Updated ParseReadNumber default case logic to use ThrowReaderError ex… 

    …ception and allow deserialization to continue
    Erik Magana committed Jan 2, 2022
    Configuration menu
    Copy the full SHA
    4e8aed4 View commit details
    Browse the repository at this point in the history

  2. Added test for Issue JamesNK#2632. Test does the following: 

    - Deserializes an array with objects that match a DateTime property.
- Verifies deserialization creates an array with X number of entries (in this case it is 1)
- Utilizes the approach used in WebAPI as that is where the issue was first being observed

Additionally, updated existing tests for Issue JamesNK#1778, which originally created the regression
    Erik Magana committed Jan 2, 2022
    Configuration menu
    Copy the full SHA
    de99f29 View commit details
    Browse the repository at this point in the history

  3. Adding in the test for Issue JamesNK#2632

    Erik Magana committed Jan 2, 2022
    Configuration menu
    Copy the full SHA
    f15dede View commit details
    Browse the repository at this point in the history