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

(#21602) Updated rake_tasks.rb to include 'integration' folder when running spec tests. #35

Merged
merged 1 commit into from
Aug 6, 2013

Conversation

fatmcgav
Copy link
Contributor

@fatmcgav fatmcgav commented Jul 4, 2013

Added 'integration' to the directories considered when running :spec_standalone from rake_tasks.rb.

@puppetcla
Copy link

Waiting for CLA signature by @fatmcgav

@fatmcgav - We require a Contributor License Agreement (CLA) for people who contribute to Puppet, but we have an easy click-through license with instructions, which is available at https://cla.puppetlabs.com/

Note: if your contribution is trivial and you think it may be exempt from the CLA, please post a short reply to this comment with details. http://docs.puppetlabs.com/community/trivial_patch_exemption.html

@puppetcla
Copy link

CLA signed by all contributors.

@hunner
Copy link
Contributor

hunner commented Jul 11, 2013

I'm curious what kind of integration tests you're using?

@fatmcgav
Copy link
Contributor Author

@hunner Am starting to build some integration tests into my NetApp device module...
Example is https://github.com/fatmcgav/fatmcgav-netapp/blob/master/spec/integration/puppet/provider/netapp_quota/netapp_quota_spec.rb

The above was thanks to @stschulte, however have got a couple more locally that need to push at some point...

Cheers
Gav

adrienthebo added a commit that referenced this pull request Aug 6, 2013
(#21602) Updated rake_tasks.rb to include 'integration' folder when running spec tests.
@adrienthebo adrienthebo merged commit 6af9e36 into puppetlabs:master Aug 6, 2013
@adrienthebo
Copy link
Contributor

summary: merged into master; thanks for the contribution and sorry about the delay in getting this merged!

@fatmcgav fatmcgav deleted the ticket/master/21602 branch August 7, 2013 14:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants