Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | [CHEF-4918] Don't destructively merge subhashes in hash_only_merge!ssd/CHEF-4918 | Steven Danna | 2014-02-08 | 2 | -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 ↵ | PrajaktaPurohit | 2014-02-07 | 12 | -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 ↵ | PrajaktaPurohit | 2014-02-07 | 12 | -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' | danielsdeleo | 2014-02-07 | 2 | -5/+20 |
|\ | | | | | | | Fixes https://tickets.opscode.com/browse/CHEF-5011 | ||||
| * | Localize rescues in Recipe method_missing DSL | danielsdeleo | 2014-02-07 | 2 | -5/+20 |
|/ | |||||
* | Merge branch 'inline-syntax-check' | danielsdeleo | 2014-02-06 | 2 | -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 possible | danielsdeleo | 2014-02-06 | 2 | -7/+107 |
|/ | |||||
* | Merge pull request #1251 from opscode/adamed/oc-11191-spec-workaround | Adam Edwards | 2014-02-05 | 1 | -0/+6 |
|\ | | | | | OC-11191: Workaround for apparent memory leak in CHEF-5004 | ||||
| * | OC-11191: Workaround for apparent memory leak in CHEF-5004 | Adam Edwards | 2014-02-04 | 1 | -0/+6 |
|/ | |||||
* | Remove `rake gem` dep on deleted tasks | danielsdeleo | 2014-02-04 | 1 | -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 release | danielsdeleo | 2014-02-04 | 1 | -1/+1 |
| | |||||
* | Remove obsolete doc generator deps | danielsdeleo | 2014-02-04 | 3 | -74/+0 |
| | |||||
* | add updated html files for knife and chef- man pages | jamescott | 2014-02-04 | 67 | -4829/+1977 |
| | |||||
* | remove old crufty man pages | jamescott | 2014-02-04 | 24 | -4943/+0 |
| | |||||
* | Merge pull request #1240 from opscode/praj/resource_name_and_id_to_str | PrajaktaPurohit | 2014-02-04 | 2 | -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 type | Prajakta Purohit | 2014-02-04 | 1 | -0/+55 |
| | | | | | | | | String irrespective of input value type | ||||
| * | Making sure that the resource_name and resource_id while being sent to ↵ | Prajakta Purohit | 2014-01-31 | 1 | -2/+2 |
| | | | | | | | | reporting are always strings. | ||||
* | | add HTML file for manpage output testing | jamescott | 2014-02-04 | 87 | -0/+15534 |
| | | |||||
* | | add knife compare to man page | jamescott | 2014-02-04 | 1 | -0/+107 |
| | | |||||
* | | Better specify what happens for NoMethodErrors in the DSL | danielsdeleo | 2014-02-03 | 4 | -3/+55 |
| | | |||||
* | | Expose resource creation via more static methods | danielsdeleo | 2014-02-03 | 3 | -39/+147 |
| | | |||||
* | | Merge branch 'CHEF-3506' | danielsdeleo | 2014-02-03 | 6 | -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 list | danielsdeleo | 2014-02-03 | 6 | -7/+36 |
|/ / | |||||
* | | Update file providers to close before unlinking. | danielsdeleo | 2014-02-03 | 2 | -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 windows | danielsdeleo | 2014-02-03 | 1 | -8/+11 |
| | | |||||
* | | update with note re: version | jamescott | 2014-02-03 | 2 | -0/+6 |
| | | |||||
* | | Delegate sync_cookbooks to policy_builder, subclasses rely on it | danielsdeleo | 2014-02-03 | 4 | -5/+13 |
| | | |||||
* | | Merge pull request #1246 from opscode/CHEF-4747-fix | Serdar Sutay | 2014-02-03 | 1 | -1/+1 |
|\ \ | | | | | | | Make sure --concurrency 1 works while uploading multiple cookbooks. | ||||
| * | | Make sure --concurrency 1 works while uploading multiple cookbooks. | sersut | 2014-02-03 | 1 | -1/+1 |
|/ / | |||||
* | | Use custom tempfile to avoid ruby 1.8-related test failures | danielsdeleo | 2014-02-03 | 1 | -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 version | jamescott | 2014-01-31 | 30 | -30/+30 |
| | |||||
* | update man pages | jamescott | 2014-01-31 | 23 | -202/+81 |
| | |||||
* | Merge pull request #1238 from opscode/lcg/rspec-2.14-deprecation-warnings | lamont-granquist | 2014-01-30 | 233 | -2254/+2261 |
|\ | | | | | Lcg/rspec 2.14 deprecation warnings | ||||
| * | omit test on ruby-1.8 | Lamont Granquist | 2014-01-30 | 1 | -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 entirely | Lamont Granquist | 2014-01-29 | 5 | -16/+0 |
| | | |||||
| * | fix busted spec exposed by rpsec-2.14-ization | Lamont Granquist | 2014-01-29 | 1 | -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 warnings | Lamont Granquist | 2014-01-29 | 6 | -9/+9 |
| | | |||||
| * | fix broken specs revealed by rspec-2.14-ization | Lamont Granquist | 2014-01-29 | 1 | -11/+9 |
| | | |||||
| * | need action for define_resource_requirements | Lamont Granquist | 2014-01-29 | 3 | -0/+3 |
| | | |||||
| * | fix test, define_resource_requirements needs an action | Lamont Granquist | 2014-01-29 | 1 | -0/+1 |
| | | |||||
| * | fix broken file provider specs | Lamont Granquist | 2014-01-29 | 1 | -0/+10 |
| | | | | | | | | | | fixing the should_not raise_error(Whatever) specs exposed that these were broken. | ||||
| * | fix should_not raise_error(SpecificException) | Lamont Granquist | 2014-01-29 | 45 | -216/+216 |
| | | |||||
| * | this test should have been failing | Lamont Granquist | 2014-01-29 | 1 | -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 editing | Lamont Granquist | 2014-01-29 | 1 | -1/+1 |
| | | |||||
| * | s/mock\(/double(/g | Lamont Granquist | 2014-01-29 | 114 | -363/+363 |
| | | |||||
| * | s/stub!/stub/g | Lamont Granquist | 2014-01-29 | 193 | -1625/+1625 |
| | | | | | | | | fix deprecation warnings | ||||
| * | fix obvious derp | Lamont Granquist | 2014-01-29 | 1 | -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 problem | Lamont Granquist | 2014-01-29 | 5 | -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 Granquist | 2014-01-29 | 1 | -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.x | Lamont Granquist | 2014-01-29 | 1 | -1/+1 |
| | |