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

Add ability to deserialize enums from SeqAccessDeserializer #2445

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

Add ability to deserialize enums from SeqAccessDeserializer #2445

wants to merge 8 commits into from

Commits on Mar 3, 2024

  1. Convert fn test_map_access_to_enum() to mod access_to_enum 

    (review this commit with "ignore whitespace changes" option on)
    @Mingun
    Mingun committed Mar 3, 2024
    Copy the full SHA
    a00ef2f View commit details
    Browse the repository at this point in the history

  2. Use functional names for test types

    @Mingun
    Mingun committed Mar 3, 2024
    Copy the full SHA
    49712d5 View commit details
    Browse the repository at this point in the history

  3. Add tests for all possible enum forms

    @Mingun
    Mingun committed Mar 3, 2024
    Copy the full SHA
    088c4b6 View commit details
    Browse the repository at this point in the history

  4. Add negative tests for MapAccessDeserializer

    @Mingun
    Mingun committed Mar 3, 2024
    Copy the full SHA
    ff069f8 View commit details
    Browse the repository at this point in the history

  5. Change error message for an empty map from which enum is deserialized 

    The new message is more logical, because actually map *is* expected,
but it should contain at least one entry
    @Mingun
    Mingun committed Mar 3, 2024
    Copy the full SHA
    3bb6af3 View commit details
    Browse the repository at this point in the history

  6. Add tests for deserialization of enum from sequence 

    failures (6):
    access_to_enum::seq::empty_seq
    access_to_enum::seq::newtype
    access_to_enum::seq::struct_
    access_to_enum::seq::tuple
    access_to_enum::seq::unit
    access_to_enum::seq::wrong_tag
    @Mingun
    Mingun committed Mar 3, 2024
    Copy the full SHA
    7697751 View commit details
    Browse the repository at this point in the history

  7. Implement deserialization of enum from sequence using SeqAccessDeseri… 

    …alizer

Fixes all tests
    @Mingun
    Mingun committed Mar 3, 2024
    Copy the full SHA
    e713fa9 View commit details
    Browse the repository at this point in the history

  8. Better document SeqAccessDeserializer and MapAccessDeserializer

    @Mingun
    Mingun committed Mar 3, 2024
    Copy the full SHA
    d338571 View commit details
    Browse the repository at this point in the history