This commit fixes couple of obvious fixes

imtiazc commented 8 years ago

Description

Resolves an unit test failure and updates Berksfile to make the source of Chef Supermarket configurable.

Issues Resolved

None

Check List

[] All tests pass. See https://github.com/chef-cookbooks/community_cookbook_documentation/blob/master/TESTING.MD
[x] New functionality includes testing.
[ ] New functionality has been documented in the README if applicable
[ ] All commits have been signed for the Developer Certificate of Origin. See https://github.com/chef-cookbooks/community_cookbook_documentation/blob/master/CONTRIBUTING.MD
A unit test error which is causing spec tests to fail
updates Berksfile to make Chef Supermarket address configurable

Obvious fix

Change-Id: I6d7bace128fccaee5c6b9a8ede1cf65815161d84 Signed-off-by: Imtiaz Chowdhury imtiaz.chowdhury@workday.com

imtiazc commented 7 years ago

We are using a private Chef supermarket in our datacenter and https://supermarket.chef.io/ is not accessible from there. With the change I proposed, one can use a different Chef repository by simply setting SUPERMARKET as an environment variable. If one does not set this value then the cookbook continues to use https://supermarket.chef.io/.

imtiazc commented 7 years ago

By the way, what is the process to fix the unit test?

tas50 commented 7 years ago

I've worked the same chefspec fix into master now. Closing this out at this point

sous-chefs / rsyslog