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.

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

adapt integration test to changes in command_wrapper #35

Merged
merged 1 commit into from
Dec 7, 2015

Conversation

srenatus
Copy link
Contributor

@srenatus srenatus commented Dec 4, 2015

command_wrapper tries to verify the connection (which includes sudo)
using a call to "echo". If this fails, it'll raise an error. This change
is now reflected in the test.

fixes #34.

@srenatus srenatus changed the title adept integration test to changes in command_wrapper adapt integration test to changes in command_wrapper Dec 4, 2015
command_wrapper tries to verify the connection (which includes sudo)
using a call to "echo". If this fails, it'll raise an error. This change
is now reflected in the test.

fixes #34.
@chris-rock chris-rock force-pushed the sr/fix-integration-test branch from 8168604 to 93b158c Compare December 7, 2015 10:56
@srenatus srenatus added the bug label Dec 7, 2015
@arlimus
Copy link
Contributor

arlimus commented Dec 7, 2015

Nice catch, thank you @srenatus !

arlimus added a commit that referenced this pull request Dec 7, 2015
adapt integration test to changes in command_wrapper
@arlimus arlimus merged commit f7213d3 into master Dec 7, 2015
@arlimus arlimus deleted the sr/fix-integration-test branch December 7, 2015 12:12
@chef-supermarket
Copy link

Hi. Your friendly Curry bot here. Just letting you know that all commit authors have become authorized to contribute. I have added the "Signed CLA" label to this issue so it can easily be found in the future.

@clintoncwolfe clintoncwolfe added Type: Bug Feature not working as expected and removed Type: Bug labels Mar 19, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Type: Bug Feature not working as expected
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Integration test failures
4 participants