Skip to content
This repository has been archived by the owner on Dec 29, 2021. It is now read-only.

fix: Remove unrelated output in main_binary #50

Merged
merged 3 commits into from
Oct 17, 2017
Merged

fix: Remove unrelated output in main_binary #50

merged 3 commits into from
Oct 17, 2017

Commits on Oct 17, 2017

  1. fix(error): Report context for spawmn failures

    @epage
    epage committed Oct 17, 2017
    Configuration menu
    Copy the full SHA
    41f4e10 View commit details
    Browse the repository at this point in the history

  2. tests: Validate main_binary / cargo_binary 

    `clap` couldn't be used for command line arguments.  I didn't want to
add it as a dependency for clients, which meant it had to be optional.
The problem is that `main_binary` and `cargo_binary` run with optional
features enabled.

Note: you cannot use `main_binary` or `cargo_binary`
- With `Environment::empty` because the executable can't be found
- In skeptic tests (like the README) because the working dir is changed
    @epage
    epage committed Oct 17, 2017
    Configuration menu
    Copy the full SHA
    6c3550c View commit details
    Browse the repository at this point in the history

  3. fix: Remove unrelated output in main_binary 

    We use cargo run behind the scenes to run the main binary but that
should only be an implementation detail.

Fixes assert-rs#45
    @epage
    epage committed Oct 17, 2017
    Configuration menu
    Copy the full SHA
    45a4aa6 View commit details
    Browse the repository at this point in the history