summaryrefslogtreecommitdiff
path: root/spec/unit
Commit message (Collapse)AuthorAgeFilesLines
* Handle nil run list option in knife bootstrapnil-run-list-bootstrapdanielsdeleo2015-11-051-0/+15
|
* Fixed knife_spec unit testJay Mundrawala2015-11-051-1/+4
| | | | | This is failing on both my and btm's machine. No idea how it passes in other places.
* Merge pull request #3836 from nathwill/svc-helpers-altThom May2015-11-051-30/+6
|\ | | | | simplify service helpers
| * fix specsNathan Williams2015-10-231-1/+1
| |
| * i have no idea what is happeningNathan Williams2015-10-231-30/+6
| |
* | windows service uses log fileMatt Wrock2015-11-051-46/+82
| | | | | | | | location from config if none is given on commandline
* | Add tests for Chef::HTTP retry behaviordanielsdeleo2015-11-041-1/+103
| |
* | Improve detection of ChefFS-based commands in `knife rehash`ssd/rehash-cheffsSteven Danna2015-10-291-0/+8
| | | | | | | | | | | | | | | | | | | | | | | | ChefFS-based commands have a superclass (Chef::ChefFS::Knife) which defines its own inherited method that calls super. This breaks our detection of where the subcommand is defined since the file with the definition is no longer at the top of the call stack. This commit special-cases subclasses with a superclass of Chef::ChefFS::Knife to account for this. Fixes #4089
* | Make Property.derive create derived properties of the same typejk/custom_propertiesJohn Keiser2015-10-281-0/+48
| |
* | Updated --no-color unit tests based on feedback from btm and KartikTyler Fitch2015-10-281-11/+11
| |
* | Update knife bootstrap command to honor --no-color flag in chef-client run ↵Tyler Fitch2015-10-281-4/+11
| | | | | | | | that is part of the bootstrap process.
* | add YARD docsLamont Granquist2015-10-261-1/+1
| |
* | stub #validate! methodLamont Granquist2015-10-261-0/+2
| |
* | fix some busted specsLamont Granquist2015-10-261-1/+1
| |
* | improve error message, add more testsLamont Granquist2015-10-261-1/+51
| |
* | extend metadata to include chef_version and ohai_versionLamont Granquist2015-10-261-1/+71
| |
* | Use SHA256 instead for registry_key when data is not displayableJay Mundrawala2015-10-241-2/+2
| |
* | fix tests for my changeAubrey Holland2015-10-241-1/+1
| |
* | We shouldn't be subclassing Struct.new - if the file is loaded twice we get ↵tyler-ball2015-10-241-45/+44
| | | | | | | | a superclass mismatch error
* | write spec for --json-attribute-file and move configration under render_templatesawanoboly2015-10-241-6/+33
| |
* | Consider only knife configuration for chef environment in bootstrap contextEvan Gilman2015-10-241-1/+1
| | | | | | | | | | | | Setting these values in Chef::Config is being deprecated in favor of using the knife config directly. https://github.com/chef/chef/issues/3946
* | Remove duplicate 'Accept' header in specJulian C. Dunn2015-10-241-2/+2
| |
* | add spec for stdout/stderr suppresion in error of sensitive execute resourceRanjib Dey2015-10-241-0/+9
| |
* | Adjust tests per BTMJeff Blaine2015-10-241-8/+8
| |
* | Changes --hide-healthy to --hide-by-mins MINSJeff Blaine2015-10-241-9/+9
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fixes #3679 Reasoning: The definition of "healthy" is overloaded in the old form (--hide-healthy) to be "a host that has run chef". The code makes no check to determine if the chef run was successful, so it has the capability to provide false positives for "health". Just because a node object was saved with ohai_time set doesn't mean the Chef run was successful. There are exception handlers like lastrun that intentionally save the node object on exception. The previous 1 hour hardcoded time was totally arbitrary. Perhaps "healthy" to others means "nodes that have run chef in the last 4 hours" (or 30 minutes, or 3 days...).
* | Define == for node objectsDavid Bresson2015-10-241-0/+8
| |
* | provider/user/dscl: Set "comment" default valueMikhail Zholobov2015-10-241-0/+7
|/ | | | | "comment" attribute should be defined to create user account properly on OS X. Otherwise, the RealName will be empty.
* provider/user/dscl: Set default gid to 20Mikhail Zholobov2015-10-231-0/+7
| | | | | gid should always be defined to create user account properly on OS X. "staff" (gid 20) is a default group for regular user accounts.
* overhaul solaris SMF service providerlcg/3187Dave Eddy2015-10-231-39/+110
|
* Merge pull request #4079 from chef/service_loggerMatt Wrock2015-10-231-2/+10
|\ | | | | add logger to windows service shellout
| * add logger to windows service shelloutservice_loggerMatt Wrock2015-10-201-2/+10
| |
* | Added run list expansion json conversion. Creates a json tree from the run ↵kyleen/addExpandedRunListKyleen MacGugan2015-10-212-3/+25
| | | | | | | | | | | | | | | | | | | | list expansion trace. Added event to report expanded run list Updated json conversion to use chef compat json. Remove RunListExpansion from JsonCompat
* | Update messaging to match LWRP -> Custom Resource renameTim Smith2015-10-211-1/+1
| | | | | | | | If you showed up in a post-LWRP world these would be confusing
* | Merge pull request #3119 from cmluciano/cml/bug1586Phil Dibowitz2015-10-202-1/+10
|\ \ | | | | | | Fix condition of removing a group before user error.
| * | Add recommended test from btmChristopher M. Luciano2015-10-121-0/+9
| | |
| * | Fix condition of removing a group before user error.cmluciano2015-04-301-1/+1
| | | | | | | | | | | | | | | | | | | | | This should fix [issue 1586](https://github.com/chef/chef/issues/1586). The issue is that if a group is removed before the user, the GID remained and causes an error. This should remove the check for remove actions.
* | | Make modules with properties work and inherit as expectedjk/property_mixinJohn Keiser2015-10-201-0/+97
| |/ |/|
* | Fix search result paginationssd/search-pagination-fixSteven Danna2015-10-091-1/+19
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The start and rows parameter that are passed as part of the search request are passed directly to Solr on the backend. Results from Solr may contain deleted nodes no longer in the erchef database. Erchef will filter such nodes from the results. Thus, a user may receive fewer rows than they asked for. Incrementing 'start' only by the number of rows received will then result in the next Solr response overlapping the first, which can lead to duplicate results. In the case of a Solr results page that was completely filtered, it would lead to an infinite loop. This commit changes the code to always increment by the requested page size (args[:rows]) when it is non-nil. Incrementing by the length of the response set is still wrong in the case when the args[:rows] is nil, but the server doesn't give us anything else to increment by. Fixes #4027
* | Accept coercion as a way to accept nil valuesJohn Keiser2015-10-081-0/+8
| |
* | fix unit testsMatt Wrock2015-10-081-2/+2
| |
* | Windows cli tools should have color true by defaultadamedx/windows-color-defaultsAdam Edwards2015-10-062-4/+4
| |
* | Quote paths.mcquin/chef-3883Claire McQuin2015-10-061-2/+12
| |
* | Fix dispatch when there are different receiversJohn Keiser2015-10-051-2/+45
| | | | | | | | | | with different numbers of arguments. Fixes https://github.com/chef/chef-dk/issues/546)
* | Merge pull request #4032 from chef/mcquin/nano/disable-32-bitClaire McQuin2015-10-051-13/+17
|\ \ | | | | | | Raise error when running 32-bit scripts on Windows Nano.
| * | Raise error when running 32-bit scripts on Windows Nano.mcquin/nano/disable-32-bitClaire McQuin2015-10-051-13/+17
| | |
* | | Merge pull request #4003 from chef/tm/duplicate_recipesThom May2015-10-052-1/+7
|\ \ \ | |/ / |/| | Ensure that our list of recipes is backwards compat
| * | Ensure that our list of recipes is backwards compattm/duplicate_recipesThom May2015-09-292-1/+7
| | | | | | | | | | | | | | | | | | | | | | | | | | | Prior to chef 12.2 we included unexpanded 'cookbook' names for default recipes. In 12.2, we moved to expanded ('cookbook::default') names, which broke some searches. However, some of our users have now moved to searching for expanded, so we need to cater for both. Fixes #3767
* | | Merge pull request #4016 from chef/mcquin/nano-ps-scriptClaire McQuin2015-10-021-26/+42
|\ \ \ | | | | | | | | Use -Command flag on Nano
| * | | Use -Command flag on NanoClaire McQuin2015-10-011-26/+42
| | | |
* | | | Merge pull request #4005 from chef/salam/template-fixSalim Alam2015-10-011-2/+52
|\ \ \ \ | | | | | | | | | | Use Chef::FileContentManagement::Tempfile to create temp file
View Lorry Controller Status