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 tests for process_ledger() #516

Merged
merged 4 commits into from
Jul 2, 2018
Merged

Add tests for process_ledger() #516

merged 4 commits into from
Jul 2, 2018

Commits on Jul 2, 2018

  1. Add tests for process_ledger()

    @garious
    garious committed Jul 2, 2018
    Configuration menu
    Copy the full SHA
    16d7b29 View commit details
    Browse the repository at this point in the history

  2. Consolidate ledger serialization code 

    The new read_entries() works, but is overly-contrained. Not
using that function yet, but adding it here in the hopes some
Rust guru will tell us how to get that lifetime constraint out
of there.

Fixes solana-labs#517
    @garious
    garious committed Jul 2, 2018
    Configuration menu
    Copy the full SHA
    771c1f4 View commit details
    Browse the repository at this point in the history

  3. Use Cursor instead of tempfile. 

    Faster and one less dependency.
    @garious
    garious committed Jul 2, 2018
    Configuration menu
    Copy the full SHA
    6463996 View commit details
    Browse the repository at this point in the history

  4. Use IntoInterator to simplify write_entries() usage

    @garious
    garious committed Jul 2, 2018
    Configuration menu
    Copy the full SHA
    d0168e4 View commit details
    Browse the repository at this point in the history