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.

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

Update event stream tests #3254

Merged
merged 1 commit into from
Sep 12, 2024
Merged

Update event stream tests #3254

merged 1 commit into from
Sep 12, 2024

Commits on Sep 12, 2024

  1. Update event stream tests

    This fixes a pair of event stream tests. Implementations of
    `application/vnd.amazon.eventstream` are expected to check the CRC
    before attempting to semantically parse the prelude. The tests meant
    to enforce that were present, but the wrong error was being asserted.
    
    This change fixes the ordering of checks, fixes the tests, and adds
    two new test cases to make assertions about the error types that were
    previously being applied to the old tests.
    JordonPhillips committed Sep 12, 2024
    Configuration menu
    Copy the full SHA
    823c76b View commit details
    Browse the repository at this point in the history