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

Improved Verify #25

Merged
merged 7 commits into from
Apr 25, 2014
Merged

Improved Verify #25

merged 7 commits into from
Apr 25, 2014

Conversation

danielsdeleo
Copy link
Contributor

Add several improvements to make running chef verify a better and more useful experience for end users:

  • Add smoke tests and just run those by default: Run a basic command that exercises a meaningful part of the app's code but is fast and non-destructive.
  • Add a verbosity option so you don't need to be blasted by noise from successful tests.
  • Add a unit test option to turn unit tests back on.
  • Add an integration test option to turn integration tests back on.

@danielsdeleo
Copy link
Contributor Author

Oh, and note that chef/mixlib-cli#17 needs to be merged and shipped for this.

@danielsdeleo
Copy link
Contributor Author

/cc @opscode/client-eng

@lamont-granquist
Copy link
Contributor

works 4 me

@mcquin
Copy link
Contributor

mcquin commented Apr 25, 2014

👍

@danielsdeleo danielsdeleo merged commit d075911 into master Apr 25, 2014
@danielsdeleo danielsdeleo deleted the quick-verify branch April 25, 2014 22:13
@danielsdeleo danielsdeleo added this to the 0.1.0 milestone Apr 29, 2014
@danielsdeleo danielsdeleo mentioned this pull request Apr 29, 2014
Closed
@chef-boneyard chef-boneyard locked and limited conversation to collaborators Feb 14, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Milestone
0.1.0
Development

Successfully merging this pull request may close these issues.
3 participants
@danielsdeleo @lamont-granquist @mcquin