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

WIP: Make cargo bin helpers work with empty env #54

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

WIP: Make cargo bin helpers work with empty env #54

wants to merge 4 commits into from

Commits on Oct 13, 2017

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

    @epage
    epage committed Oct 13, 2017
    Configuration menu
    Copy the full SHA
    aac5ec0 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 13, 2017
    Configuration menu
    Copy the full SHA
    aa89b08 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 #45
    @epage
    epage committed Oct 13, 2017
    Configuration menu
    Copy the full SHA
    fae7c90 View commit details
    Browse the repository at this point in the history

  4. Make cargo bin helpers work with empty env 

    Fixes #51
    @killercup
    killercup committed Oct 13, 2017
    Configuration menu
    Copy the full SHA
    2f0e943 View commit details
    Browse the repository at this point in the history