summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
* [CHEF-4918] Don't destructively merge subhashes in hash_only_merge!ssd/CHEF-4918Steven Danna2014-02-082-3/+12
| | | | | | hash_only_merge dups its inputs and then passes it to hash_only_merge!. Unfortunately, dup does not make a deep copy, leading hash_only_merge to mutate deeply nested structures.
* Merge pull request #1236 from ↵PrajaktaPurohit2014-02-0712-22/+205
|\ | | | | | | | | opscode/praj/OC-10990/runid_in_CC_and_knife_integration Adding request_id to the set of headers for every request that will be sent to erchef
| * - Adding X-Remote-Request-Id to the set of headers for every request from ↵PrajaktaPurohit2014-02-0712-22/+205
|/ | | | | | | | | | CCR and knife that will be sent to erchef - Each knife request has a different X-Remote-Request-Id, where as it remains the same for all requests originating from the same chef-client run. - Adding and fixing tests
* Merge branch 'nomethod-gobbling'danielsdeleo2014-02-072-5/+20
|\ | | | | | | Fixes https://tickets.opscode.com/browse/CHEF-5011
| * Localize rescues in Recipe method_missing DSLdanielsdeleo2014-02-072-5/+20
|/
* Merge branch 'inline-syntax-check'danielsdeleo2014-02-062-7/+107
|\ | | | | | | | | * fixes https://tickets.opscode.com/browse/CHEF-4986 * Improves syntax check speed for Ruby 1.9+, especially when using bundler.
| * Use RubyVM to syntax check in-process where possibledanielsdeleo2014-02-062-7/+107
|/
* Merge pull request #1251 from opscode/adamed/oc-11191-spec-workaroundAdam Edwards2014-02-051-0/+6
|\ | | | | OC-11191: Workaround for apparent memory leak in CHEF-5004
| * OC-11191: Workaround for apparent memory leak in CHEF-5004Adam Edwards2014-02-041-0/+6
|/
* Remove `rake gem` dep on deleted tasksdanielsdeleo2014-02-041-3/+0
| | | | | Dependency between task "gem" and "docs:all" caused "gem" to fail now that the obsolete "docs" tasks are removed.
* Bump version number for 11.10.0.rc.2 releasedanielsdeleo2014-02-041-1/+1
|
* Remove obsolete doc generator depsdanielsdeleo2014-02-043-74/+0
|
* add updated html files for knife and chef- man pagesjamescott2014-02-0467-4829/+1977
|
* remove old crufty man pagesjamescott2014-02-0424-4943/+0
|
* Merge pull request #1240 from opscode/praj/resource_name_and_id_to_strPrajaktaPurohit2014-02-042-2/+57
|\ | | | | The resource_name and resource_id while being sent to reporting are always strings.
| * Adding tests to make sure the resource_name and resource_id are of typePrajakta Purohit2014-02-041-0/+55
| | | | | | | | String irrespective of input value type
| * Making sure that the resource_name and resource_id while being sent to ↵Prajakta Purohit2014-01-311-2/+2
| | | | | | | | reporting are always strings.
* | add HTML file for manpage output testingjamescott2014-02-0487-0/+15534
| |
* | add knife compare to man pagejamescott2014-02-041-0/+107
| |
* | Better specify what happens for NoMethodErrors in the DSLdanielsdeleo2014-02-034-3/+55
| |
* | Expose resource creation via more static methodsdanielsdeleo2014-02-033-39/+147
| |
* | Merge branch 'CHEF-3506'danielsdeleo2014-02-036-7/+36
|\ \ | | | | | | | | | | | | | | | Related: https://tickets.opscode.com/browse/CHEF-3506 https://tickets.opscode.com/browse/CHEF-3964
| * | [CHEF-3506] suppress final node save when using override run listdanielsdeleo2014-02-036-7/+36
|/ /
* | Update file providers to close before unlinking.danielsdeleo2014-02-032-0/+2
| | | | | | | | | | | | | | | | This is required on windows, but the Tempfile implementation hides this and silently ignores the error, meaning that we're leaving tempfiles around on windows after we thought we'd cleaned them up. Eventually GC would get to them, but this won't happen if there is a severe crash or something calls `Kernel.exit!`, so we prefer to clean up manually.
* | Close before unlink because windowsdanielsdeleo2014-02-031-8/+11
| |
* | update with note re: versionjamescott2014-02-032-0/+6
| |
* | Delegate sync_cookbooks to policy_builder, subclasses rely on itdanielsdeleo2014-02-034-5/+13
| |
* | Merge pull request #1246 from opscode/CHEF-4747-fixSerdar Sutay2014-02-031-1/+1
|\ \ | | | | | | Make sure --concurrency 1 works while uploading multiple cookbooks.
| * | Make sure --concurrency 1 works while uploading multiple cookbooks.sersut2014-02-031-1/+1
|/ /
* | Use custom tempfile to avoid ruby 1.8-related test failuresdanielsdeleo2014-02-031-1/+26
|/ | | | | | Tempfile in ruby 1.8.7 calls `File.exist?()`, which we set a stub on. For reasons that are not clear, this does not consistently fail. In any case, using a custom temporary file implementation resolves the issue.
* update man pages w/11-10 versionjamescott2014-01-3130-30/+30
|
* update man pagesjamescott2014-01-3123-202/+81
|
* Merge pull request #1238 from opscode/lcg/rspec-2.14-deprecation-warningslamont-granquist2014-01-30233-2254/+2261
|\ | | | | Lcg/rspec 2.14 deprecation warnings
| * omit test on ruby-1.8Lamont Granquist2014-01-301-1/+1
| | | | | | | | | | | | rspec and ruby-1.8 don't play well with this test: https://github.com/rspec/rspec-mocks/issues/549
| * ripping out rspec_reset entirelyLamont Granquist2014-01-295-16/+0
| |
| * fix busted spec exposed by rpsec-2.14-izationLamont Granquist2014-01-291-1/+1
| | | | | | | | | | | | | | the behavior it is testing has never worked. it can't really work since you're blowing up in String#[]= long before you could hit the attribute code. changed the test to test something slightly more useful.
| * remove some more deprecation warningsLamont Granquist2014-01-296-9/+9
| |
| * fix broken specs revealed by rspec-2.14-izationLamont Granquist2014-01-291-11/+9
| |
| * need action for define_resource_requirementsLamont Granquist2014-01-293-0/+3
| |
| * fix test, define_resource_requirements needs an actionLamont Granquist2014-01-291-0/+1
| |
| * fix broken file provider specsLamont Granquist2014-01-291-0/+10
| | | | | | | | | | fixing the should_not raise_error(Whatever) specs exposed that these were broken.
| * fix should_not raise_error(SpecificException)Lamont Granquist2014-01-2945-216/+216
| |
| * this test should have been failingLamont Granquist2014-01-291-8/+8
| | | | | | | | | | | | "root*goo" is now legal for username/groupname. not sure why this test was passing on rspec 2.13.x. changed the name to something which is guaranteed to be invalid.
| * the perils of regexp-based code editingLamont Granquist2014-01-291-1/+1
| |
| * s/mock\(/double(/gLamont Granquist2014-01-29114-363/+363
| |
| * s/stub!/stub/gLamont Granquist2014-01-29193-1625/+1625
| | | | | | | | fix deprecation warnings
| * fix obvious derpLamont Granquist2014-01-291-1/+1
| | | | | | | | | | | | the fact that rspec 2.14 accepted this and changed the should_not_receive into a should_receive is a bit of a bug in rspec.
| * fix rspec_reset problemLamont Granquist2014-01-295-4/+15
| | | | | | | | | | | | | | follows the pattern in the rspec-mocks spec tests of defining a helper module: https://github.com/rspec/rspec-mocks/blob/cee433c89125a3984df33c87eb61985613adce9b/spec/spec_helper.rb
| * fix "please stub a default message first"Lamont Granquist2014-01-291-17/+17
| | | | | | | | | | | | | | | | | | | | | | | | fixes this kind of thing: 1) Chef::Application class method: fatal! should log an error message to the logger Failure/Error: Chef::Log.stub!(:fatal).with("blah").and_return(true) #<IO:0x0000010106ab88> received :puts with unexpected arguments expected: ("FATAL: blah") got: (no args) Please stub a default value first if message might be received with other args as well. # /Users/lamont/oc/chef/spec/unit/application_spec.rb:234:in `block (3 levels) in <top (required)>'
| * bump to rspec 2.14.xLamont Granquist2014-01-291-1/+1
| |