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

Add test for InSpec Attributes functionality #266

Merged
merged 1 commit into from Aug 10, 2017

Conversation

Projects
None yet
3 participants
@adamleff
Collaborator

adamleff commented Aug 10, 2017

The work introduced by #262 was lacking an integration test for Test Kitchen. I also renamed the suite to be clearer about what it does and to avoid any confusion that the work may be a work-in-progress.

Add test for InSpec Attributes functionality
The work introduced by #262 was lacking an integration test for Test Kitchen. I
also renamed the suite to be clearer about what it does and to avoid any
confusion that the work may be a work-in-progress.

Signed-off-by: Adam Leff <adam@leff.co>

@adamleff adamleff requested review from alexpop, chris-rock and arlimus Aug 10, 2017

@chris-rock

Great improvement @adamleff

@alexpop

short and sweet

@adamleff

This comment has been minimized.

Collaborator

adamleff commented Aug 10, 2017

Travis failures are due to berkshelf/solve and molinillo issues, for which a fix is in the works. Merging.

@adamleff adamleff merged commit c1a3cbf into master Aug 10, 2017

1 of 2 checks passed

continuous-integration/travis-ci/pr The Travis CI build failed
Details
DCO This commit has a DCO Signed-off-by
Details

@adamleff adamleff deleted the adamleff/attributes-cleanup branch Aug 10, 2017

@adamleff adamleff removed the in progress label Aug 10, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment