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

Fix Excessive Encoding in Test Logs #92286

Merged
merged 4 commits into from
Sep 27, 2023
Merged

Fix Excessive Encoding in Test Logs #92286

merged 4 commits into from
Sep 27, 2023

Commits on Sep 19, 2023

  1. Removed special encoding that was rendering the test logs near 

    impossible to read properly.
    @ivdiazsa
    ivdiazsa committed Sep 19, 2023
    Configuration menu
    Copy the full SHA
    8d79fd6 View commit details
    Browse the repository at this point in the history

Commits on Sep 20, 2023

  1. Adjusted the offending test to print the invalid character's hex code 

    instead, and fixed it alongside its sibling test because they didn't
handle all correct/incorrect cases properly.
    @ivdiazsa
    ivdiazsa committed Sep 20, 2023
    Configuration menu
    Copy the full SHA
    6a7932c View commit details
    Browse the repository at this point in the history

Commits on Sep 25, 2023

  1. Added special handling for illegal XML characters in the test results' 

    XML logs.
    @ivdiazsa
    ivdiazsa committed Sep 25, 2023
    Configuration menu
    Copy the full SHA
    29d0f74 View commit details
    Browse the repository at this point in the history

Commits on Sep 26, 2023

  1. Simplified the sanitizing algorithm to one pass, as per Dan's feedback.

    @ivdiazsa
    ivdiazsa committed Sep 26, 2023
    Configuration menu
    Copy the full SHA
    8abb04c View commit details
    Browse the repository at this point in the history