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

They are integration tests, not smoke tests #1415

Merged
merged 1 commit into from
Nov 30, 2017

Conversation

nathenharvey
Copy link
Contributor

@nathenharvey nathenharvey commented Oct 3, 2017

Update the tests for the new directory name

The tests now live in an integration directory instead of a smoke directory.

Check List

@nathenharvey nathenharvey requested a review from a team October 3, 2017 18:18
@nathenharvey nathenharvey changed the title Nathen/they are integration tests They are integration tests, not smoke tests Oct 3, 2017
@lamont-granquist
Copy link
Contributor

back to the future?

RELEASE_NOTES.md Outdated
@@ -31,3 +31,8 @@ cookbooks to supermarkets.
The cookbook generator now adds a LICENSE file when creating a new
cookbook.

## Rename `smoke` tests to `integration` tests.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@nathenharvey Can you rebase this and shove this into the top of the release notes please

* Integration tests are for multiple classes/units, like a cookbook
* Smoke tests quickly determine if the system is "working"
* The tests now live in an `integration` directory instead of a `smoke`
directory
* Add the renamed integration test to the RELEASE_NOTES

Signed-off-by: Nathen Harvey <nharvey@chef.io>
@nathenharvey nathenharvey force-pushed the nathen/they_are_integration_tests branch from 0cad62e to 3d09730 Compare November 29, 2017 20:06
@tas50 tas50 merged commit 127ecf2 into master Nov 30, 2017
@nathenharvey nathenharvey deleted the nathen/they_are_integration_tests branch November 30, 2017 14:38
@chef-boneyard chef-boneyard locked and limited conversation to collaborators Apr 2, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Development

Successfully merging this pull request may close these issues.

5 participants