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

Remove reflection usage from Console tests #73224

Merged
merged 3 commits into from
Aug 2, 2022
Merged

Remove reflection usage from Console tests #73224

merged 3 commits into from
Aug 2, 2022

Commits on Aug 2, 2022

  1. perform one sys-call instead of two to get file length

    @adamsitnik
    adamsitnik committed Aug 2, 2022
    Configuration menu
    Copy the full SHA
    d56906f View commit details
    Browse the repository at this point in the history

  2. use RandomAccess.Read and properly handle partial reads

    @adamsitnik
    adamsitnik committed Aug 2, 2022
    Configuration menu
    Copy the full SHA
    029508a View commit details
    Browse the repository at this point in the history

  3. include TermInfo in the test project, stop using reflection to test it

    @adamsitnik
    adamsitnik committed Aug 2, 2022
    Configuration menu
    Copy the full SHA
    1f8a596 View commit details
    Browse the repository at this point in the history