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 streaming output arg #177

Merged
merged 2 commits into from
Jul 14, 2013
Merged

Fix streaming output arg #177

merged 2 commits into from
Jul 14, 2013

Commits on Jul 12, 2013

  1. Fix streaming output arg customization 

    I noticed that there actually are unit tests for this, but
we weren't checking the RC of the run command, only that the request
params looked how we wanted them to.  As a result, I've updated the
base test class to check for expected RCs from the commands that are
run.  This caused the tests to fail and should ensure that if we
break the streaming output arg customization, we'll get a test failure.
    @jamesls
    jamesls committed Jul 12, 2013
    Configuration menu
    Copy the full SHA
    8bff54e View commit details
    Browse the repository at this point in the history

  2. Update integration test to check for Amazon EC2 

    This is just due to the way the docs were being rendered
so I picked a more reliable doc string change.

All the integration tests are passing.
    @jamesls
    jamesls committed Jul 12, 2013
    Configuration menu
    Copy the full SHA
    0ce965e View commit details
    Browse the repository at this point in the history