summaryrefslogtreecommitdiff
path: root/spec/unit
Commit message (Collapse)AuthorAgeFilesLines
* Remove should_exit out of parameters and make ui.confirm less ugly.sersut2014-03-192-63/+102
|
* [CHEF-5057] Add default choice option to confirm promptEric Sigler2014-03-191-7/+32
| | | | | | | | | | | This commit allows a default choice to be provided for a ui.confirm prompt. "-y" flag still takes precedence, and the "default" for the default is the previous behavior ("Y/N", no default choice). Testing done: bundle exec rake spec - passes bundle exec rspec spec/unit/knife/core - passes
* [CHEF-4918] Don't destructively merge subhashes in hash_only_merge!Steven Danna2014-03-191-0/+7
| | | | | | 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.
* [CHEF-3812] Update test to check for ui.fatalJulian C. Dunn2014-03-191-3/+4
|
* [CHEF-3812] Support a --before option in "knife node run_list add"Julian C. Dunn2014-03-191-0/+22
|
* Merge pull request #1280 from josephholsten/CHEF-5064Serdar Sutay2014-03-191-0/+5
|\ | | | | CHEF-5064: ensure Chef::REST does not modify options in-place
| * ensure Chef::REST does not modify options in-placeJoseph Anthony Pasquale Holsten2014-02-251-0/+5
| |
* | Merge pull request #1321 from opscode/btm/CHEF-3714Bryan McLellan2014-03-191-0/+8
|\ \ | | | | | | CHEF-3714: add a file_edited? method
| * | CHEF-3714: Update file_edited? for FileEdit refactorBryan McLellan2014-03-191-2/+3
| | |
| * | CHEF-3714 - Method returns if a file was editedNanuk Krinner2014-03-191-0/+7
| | | | | | | | | | | | | | | | | | Implement a method in Chef::Util::FileEdit that returns if a file was edited This commit contains the method originally written by rhafer@suse.com and a unit test.
* | | Merge pull request #1268 from ccope/solaris11-ips-fixSerdar Sutay2014-03-191-174/+191
|\ \ \ | |/ / |/| | [CHEF-5037] default to IPS packages on Solaris 5.11+
| * | add tests for solaris package providersCam Cope2014-02-181-174/+191
| | |
* | | CHEF-3714: Update tests for changes from CHEF-5018Bryan McLellan2014-03-192-40/+18
| | | | | | | | | | | | Signed-off-by: Bryan McLellan <btm@opscode.com>
* | | CHEF-4173: Test that insert_line_if_no_match works more than onceChris Bandy2014-03-191-0/+8
| | |
* | | CHEF-3714: New method to advertise unwritten editsChris Bandy2014-03-191-0/+9
| | |
* | | CHEF-3714: Use Editor to manipulate textChris Bandy2014-03-192-0/+33
| | |
* | | CHEF-3714: New object dedicated to manipulating textChris Bandy2014-03-191-0/+152
| | |
* | | CHEF-4851 - Broke testing of cookbooks and versions into own includes.Josh Murphy2014-03-191-1/+3
| | | | | | | | | | | | | | | - This is to fix Travis builds for 1.8.7 which has brought back the dependencies in different orders - I can replicate Travis behavior on my Mac where sometimes the tests pass and sometimes the pass fail before this fix
* | | CHEF-4851 - Tested stderr string output compared to what function is being ↵Josh Murphy2014-03-191-6/+8
| | | | | | | | | | | | called
* | | CHEF-4851 - Added testing for missing dependencies in cookbook uploadJosh Murphy2014-03-191-0/+42
| | |
* | | CHEF-1752: Support epochs in debian package versionsBryan McLellan2014-03-191-0/+4
| | | | | | | | | | | | | | | | | | | | | | | | Debian allows a colon in package versions to handle cases where the upstream version of a release may be newer than a prior release, but not numerically sort as higher. H/T to Eric Herot <eric@where.com> for the original patch
* | | Exctracted resource exception message to separate method to be able to ↵Pavel Brylov2014-03-191-0/+5
| | | | | | | | | | | | overwrite it in subclasses. Added custom_exception_message to subversion resource to filter out password if any
* | | Merge pull request #1235 from higanworks/CHEF-4990Serdar Sutay2014-03-191-10/+35
|\ \ \ | | | | | | | | CHEF-4990 Fix provider for the state of 'maintenance' Solaris services.
| * | | CHEF-4990sawanoboly2014-02-201-10/+35
| | | |
* | | | Merge pull request #1213 from MSOpenTech/CHEF-4962Serdar Sutay2014-03-191-6/+17
|\ \ \ \ | | | | | | | | | | CHEF-4962, knife ssh will use a cloud attribute for port if available.
| * | | | CHEF-4962, knife ssh will use a cloud atttribute for port if available.Jeff Mendoza2014-02-201-6/+17
| |/ / / | | | | | | | | | | | | | | | | | | | | The Azure cloud can have multiple nodes behind a single ip/fqdn with different ssh ports for each. knife ssh should be able to run on a search of nodes and use the correct ssh port for each.
* | | | Merge pull request #1245 from slantview/CHEF-4075Serdar Sutay2014-03-191-1/+45
|\ \ \ \ | | | | | | | | | | Update to allow boolean and numeric attributes
| * | | | update tests and add new tests for limiting attribute typesSteve Rude2014-02-041-1/+45
| | | | |
* | | | | Map 'ibm_powerkvm' to Red Hat-like package managerLance Bragstad2014-03-191-1/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Include 'ibm_powerkvm' in provider mappings. Fixes CHEF-5135
* | | | | CHEF-3698: Do not set log_level by defaultBryan McLellan2014-03-191-1/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The default log level for Chef 10 was "info" but on Chef 11 it is "auto" which does not work on Chef 10. This prevents you from bootstrapping a Chef 10 node from a Chef 11 workstation. We can leave out log_level and Chef will use the built in defaults.
* | | | | Merge pull request #1308 from opscode/CHEF-4396Serdar Sutay2014-03-181-1/+48
|\ \ \ \ \ | | | | | | | | | | | | Don't honor splay setting when sent USR1 signal.
| * | | | | Workaround ruby 1.8 IO#getc returning integer typesdanielsdeleo2014-03-181-11/+29
| | | | | | | | | | | | | | | | | | | | | | | | Also make test more likely to halt when failing.
| * | | | | Make the spec for USR1 signal faster.sersut2014-03-181-8/+16
| | | | | |
| * | | | | Updates based on PR comments.sersut2014-03-181-8/+1
| | | | | |
| * | | | | Don't honor splay setting when sent USR1 signal.sersut2014-03-181-1/+29
| | | | | |
* | | | | | Fix the spec failures on ruby 1.8.7.sersut2014-03-181-2/+2
| | | | | |
* | | | | | Spec for new option to ui.confirm.sersut2014-03-171-0/+9
| | | | | |
* | | | | | Convert --force option of knife client delete to --delete-validators to be ↵sersut2014-03-171-4/+4
| | | | | | | | | | | | | | | | | | | | | | | | consistent with knife client bulk delete.
* | | | | | Specs for --delete-validators options.sersut2014-03-171-38/+121
|/ / / / /
* | | | | Change missing dependency from hard error to warning for now.danielsdeleo2014-03-141-1/+3
| | | | |
* | | | | Raise an error when including a recipe from an unreachable cookbookdanielsdeleo2014-03-144-10/+21
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fixes CHEF-4367. When attempting to load a recipe belonging to a cookbook that is not in the run_list or any dependencies of cookbooks in the run_list, chef will now produce an error like this: Chef::Exceptions::MissingCookbookDependency ------------------------------------------- Recipe `ancient::aliens` is not in the run_list, and cookbook 'ancient' is not a dependency of any cookbook in the run_list. To load thisrecipe, first add a dependency on cookbook 'ancient' in the cookbook you're including it from in that cookbook's metadata. This error will occur when chef-solo users use `include_recipe` without specifying the dependency in metadata; prior to this patch, chef would typically fail reading an undefined attribute, which commonly would result in a NoMethodError for nil.
* | | | | Fix RSpec deprecation warnings for Mac OSX service testsdanielsdeleo2014-03-141-3/+3
| | | | |
* | | | | Merge branch 'CHEF-4799' of github.com:criteo/chef into contributions-cc-51sersut2014-03-122-1/+12
|\ \ \ \ \
| * | | | | [CHEF-4799] Handle non-dupable elements when duping attribute arraysPierre Ynard2014-01-102-1/+12
| | | | | |
* | | | | | Merge branch 'jtimberman/CHEF-4927' into contributions-cc-51sersut2014-03-121-0/+5
|\ \ \ \ \ \
| * | | | | | CHEF-4927 - coerce group GID to stringjtimberman2014-01-071-0/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | To remain consistent with the way the user provider works, coerce both new_resource and current_resource GIDs to strings for comparison. This allows users to write their resources like: group "zone" do gid 233 end or group "zone" do gid "233" end Without this change, every time Chef runs when a string GID is specified, it will modify the group, which is not desirable.
* | | | | | | Fix the Ohai resource for Ohai 7.sersut2014-03-071-3/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Conflicts: spec/unit/provider/ohai_spec.rb
* | | | | | | Add enable/disable to MacOSX service providerPhil Dibowitz2014-03-061-11/+29
| | | | | | | | | | | | | | | | | | | | | | | | | | | | v11 version of #1267
* | | | | | | Merge pull request #1289 from opscode/lcg/CHEF-5030lamont-granquist2014-03-061-24/+251
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | CHEF-5030: improve debian ifconfig provider code
| * | | | | | | CHEF-5030: improve debian ifconfig provider codeLamont Granquist2014-03-051-24/+251
| | |_|_|_|/ / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | - dynamically create directory resource to create /etc/network/interfaces.d - better idempotency and accuracy in what we report to the user - dependency injection in tests that uses real tempfiles instead of mocking File class, etc