summaryrefslogtreecommitdiff
path: root/spec/unit
Commit message (Collapse)AuthorAgeFilesLines
* Stop using obsolete URI.escape; use Addressable::URI instead.John Keiser2016-08-221-2/+4
| | | | See http://stackoverflow.com/questions/2824126/whats-the-difference-between-uri-escape-and-cgi-escape#answer-13059657 for details on why.
* user provider manage_home behavior and refactorLamont Granquist2016-08-197-32/+101
| | | | | | | | | split out the user provider into 1:1 resources. fix the behavior of the linux user provider to implement manage_home behavior correctly. Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
* fix Style/BlockDelimiters, Style/MultilineBlockLayout and 0.42.0 engine upgradeLamont Granquist2016-08-17113-936/+992
| | | | Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
* fix Style/VariableName copLamont Granquist2016-08-162-4/+4
| | | | Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
* fix some Style/TrailingWhitespaceLamont Granquist2016-08-161-30/+30
| | | | | | I'm not sure how this happened Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
* fixing Lint/Void copLamont Granquist2016-08-163-11/+11
|
* fix Lint/UnderscorePrefixedVariableName copLamont Granquist2016-08-162-10/+10
|
* autofixing Style/WhileUntilModifier copLamont Granquist2016-08-161-3/+1
|
* autofixing Style/SpaceBeforeSemicolon copLamont Granquist2016-08-161-2/+2
|
* autofixing Style/SpaceBeforeComment copLamont Granquist2016-08-161-2/+2
|
* auotfixing Style/SpaceAfterSemicolon copLamont Granquist2016-08-161-5/+5
|
* autofixing Style/RedundantException copLamont Granquist2016-08-162-3/+3
|
* autofixing Style/IndentAssignment copLamont Granquist2016-08-162-277/+277
|
* Spec break on Windows due to temp dir and short path namesAdam Edwards2016-08-162-2/+2
|
* rubygems memory perf issueslcg/rubygems_perf2Lamont Granquist2016-08-151-38/+46
| | | | | | set rubygems_cache_enabled to true in Chef::Config to get the old behavior back Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
* okay i really, really hate ruby 2.1 nowlcg/keepalives-again-againLamont Granquist2016-08-111-2/+2
| | | | | | this is reaching ruby 1.8.7-levels of pissing me off Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
* test Chef::HTTP and tweak cacheLamont Granquist2016-08-111-0/+14
| | | | | | this is mildly awful, but works around the String-vs-URI crap Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
* specs and lazying net-http client buildingLamont Granquist2016-08-112-0/+27
| | | | Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
* add tests for noopinglcg/apt-update-noopLamont Granquist2016-08-052-4/+28
|
* Merge branch 'master' into configoptionNoah Kantrowitz2016-08-0226-216/+809
|\
| * Merge pull request #5127 from chef/tm/empty_cron_attributesThom May2016-08-021-16/+16
| |\ | | | | | | Support setting an empty string for cron attrs
| | * Fix handling of empty string values for cron attrstm/empty_cron_attributesThom May2016-07-291-16/+16
| | | | | | | | | | | | | | | This is a rebase and clean up of #311 Signed-off-by: Thom May <thom@chef.io>
| * | Invalidate the cache on file creationtm/cache_invalidateThom May2016-08-021-0/+1
| | | | | | | | | | | | Signed-off-by: Thom May <thom@chef.io>
| * | Also clear notifications when deleting a resource.Noah Kantrowitz2016-07-281-0/+30
| |/
| * We don't care so much about categories on the supermarkettm/fix_supermarket_share_errorThom May2016-07-141-4/+4
| | | | | | | | Signed-off-by: Thom May <thom@may.lt>
| * added "zypper info" tests for 1.12.* and 1.13.* formatEike Waldt2016-07-141-2/+18
| |
| * Merge pull request #5103 from chef/lcg/fix-5012Lamont Granquist2016-07-121-0/+6
| |\ | | | | | | add back method_missing support to set_unless
| | * fix mixup in the test titlesLamont Granquist2016-07-111-2/+2
| | |
| | * add back method_missing support to set_unlessLamont Granquist2016-07-111-0/+6
| | | | | | | | | | | | closes #5012
| * | Merge pull request #5093 from chef/aix2Matt Wrock2016-07-081-0/+13
| |\ \ | | | | | | | | Warn if not installing an individual bff fileset
| | * | warn if not installing an individual bff filesetaix2Matt Wrock2016-07-071-0/+13
| | |/
| * | fix tests to go red on masterLamont Granquist2016-07-081-10/+12
| | | | | | | | | | | | its necessary to set the breadcrumb to something wrong first.
| * | fix for #5094 12.12.13 node.default_unless issueLamont Granquist2016-07-082-20/+75
| |/ | | | | | | | | | | note that the node_spec test in this commit is not red on master so it ain't testing the right thing yet, but this fixes the issue in chef-shell...
| * Use Mixlib::Archive to extract tarballsThom May2016-07-072-6/+8
| | | | | | | | | | | | | | this allows us to be truely cross platform, and also to ignore unsafe paths and permissions. Signed-off-by: Thom May <thom@chef.io>
| * Merge pull request #5076 from chef/adamleff/ipo-277-uri-validationAdam Leff2016-07-051-1/+41
| |\ | | | | | | Data Collector server URL validation, and disable on host down
| | * Data Collector server URL validation, and disable on host downAdam Leff2016-07-031-1/+41
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | If a user configured data_collector.server_url in their Chef config as an empty string, we blissfully passed it along to Chef::HTTP which would eventually raise a TypeError when trying to dup the URI's host. This change validates the server_url when setting up the Data Collector and gives helpful error messages to the user. We also count Errno::EHOSTDOWN as an error worthy of disabling the Data Collector reporter for a given run if the user so chooses.
| * | tweak 3694 warningsLamont Granquist2016-07-011-0/+21
| |/ | | | | | | | | | | | | | | - clean up "ZenMaster" resource - clean up 3694 detection to use properties - unlazy the resource_name in the trivial resource check - fixes an issue with resources-as-definitions pattern emitting 3694 errors for trivial resources
| * Merge pull request #5065 from chef/adamleff/include-node-in-converge-messagesAdam Leff2016-06-301-43/+19
| |\ | | | | | | Adding node object to Data collector run_converge message
| | * ensuring the node in the run_converge message is a node object and that it ↵adamleff/include-node-in-converge-messagesAdam Leff2016-06-281-1/+17
| | | | | | | | | | | | gets serialized correctly
| | * Adding node object to run_converge message, include ignore_failure propertyAdam Leff2016-06-281-42/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Many use cases that involve consuming the run_converge messages and displaying them to end users include needing additional data about the node that generated the run_converge message. This change consolidates the run_converge and node_update message into a single message making it easier for users to filter run_converge messages based on attributes and criteria of the node itself. Additionally, the ignore_failure property on a resource is now included for each resource in the resource list so end users can decided whether the failure was a hard failure or a soft failure.
| * | Attributes v1.1 changeslcg/attributes-v1.1Lamont Granquist2016-06-278-46/+481
| |/ | | | | | | | | | | | | | | | | | | | | - fixes *_unless behavior and set_unless_value_present hack from Chef 12 - simplifies rm_* code - introduces functional read/write/unlink/exist? API - deprecates method_missing access to attributes for Chef 13 - deprecates set/set_unless aliases for Chef 14 - removes MultiMash mess that I wrote for Chef 13 https://github.com/chef/chef/pull/5029 for more details
| * Merge pull request #5058 from ↵Adam Leff2016-06-242-27/+43
| |\ | | | | | | | | | | | | chef/adamleff/IPO-196/add-unprocessed-resources-to-data-collector Expand data_collector resource list to include all resources
| | * Expand data_collector resource list to include all resourcesAdam Leff2016-06-242-27/+43
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Historically when a Chef run fails, the event handlers only report on resources that have been processed during the run. This change allows the Data Collector to report those resources in the resource collection that have not yet been processed so users can better ascertain how much of their run was completed, and specifically what resources were not processed as a result of the Chef failure. Instead of building up a list of resources as they are processed, this change instead creates a resource report for each resource+action in the resource collection and then modifies each of those reports once the resource is processed.
| * | Ensure that chef-solo uses the expected repo dirtm/correct_repo_pathThom May2016-06-241-0/+7
| |/ | | | | | | | | | | | | | | | | Solo has always defaulted to `/var/chef` but local-mode uses the user's home directory, so we need to ensure that we set the correct path before entering local-mode. Fixes: #5051 Signed-off-by: Thom May <thom@may.lt>
| * fixes some uninstall scenarios by calling uninstall string directlyuninstallMatt Wrock2016-06-221-9/+14
| |
| * Merge pull request #4896 from chef/tm/include_knife_supermarketThom May2016-06-221-0/+1
| |\ | | | | | | Deprecate knife-supermarket gem
| | * Deprecate knife-supermarket gemtm/include_knife_supermarketThom May2016-05-051-0/+1
| | | | | | | | | | | | | | | | | | | | | Include all knife supermarket sub commands here, and allow all knife cookbook site commands to have a configurable supermarket URL. Signed-off-by: Thom May <thom@may.lt>
| * | Add most common knife commands to integration teststm/knife_integration_testsThom May2016-06-222-49/+0
| | | | | | | | | | | | Signed-off-by: Thom May <thom@chef.io>
| * | Merge pull request #5022 from coderanger/3694deprecationNoah Kantrowitz2016-06-161-6/+10
| |\ \ | | | | | | | | Convert the 3694 warning to a deprecation so it will be subject to the usual deprecation formatting (collected at the bottom, can be made an error, etc).
| | * | Convert the 3694 warning to a deprecation so it will be subject to the usual ↵Noah Kantrowitz2016-06-101-6/+10
| | | | | | | | | | | | | | | | deprecation formatting (collected at the bottom, can be made an error, etc).