summaryrefslogtreecommitdiff
path: root/spec/unit
Commit message (Collapse)AuthorAgeFilesLines
* 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).
* | | Remote resource should respect sensitive flagpraj/FLOW-334/sensitivePrajakta Purohit2016-06-131-0/+15
| | |
* | | Deprecate `knife cookbook create` in favor of `chef generate cookbook`.tc/deprecate-chef-cookbook-generateTyler Cloke2016-06-101-0/+1
|/ /
* | bug fix: correctly write out data collector metadata fileadamleff/node-uuid-fixAdam Leff2016-06-101-7/+3
| | | | | | | | | | | | | | | | | | When we changed the messages from class instances to a module, the reading of metadata into an instance variable was removed and introduced a bug when updating metadata. This bug causes the metadata to be properly read from disk, updated, but then the metadata from disk is simply re-written to disk without the intended updates.
* | Bug fix: updated_resource_count should only include updated resourcesAdam Leff2016-06-072-30/+14
| | | | | | | | | | | | | | | | After adding up-to-date and skipped resources to the DataCollector resource list, the logic that computed the number of resources updated needed to change accordingly. This change fixes that bug and also eliminates an unnecessary counter that tracks the total number of resources.
* | did not need to pend dsc_script ps_credential unit testsmurawski/pend_dsc_script_credential_testSteven Murawski2016-06-061-1/+0
| |
* | Merge pull request #4998 from chef/smurawski/pend_dsc_script_credential_testSteven Murawski2016-06-061-0/+1
|\ \ | | | | | | Skip `ps_credential` test for `dsc_script` until proper test cert
| * | skip the ps_credential test of dsc_script until we can get the right test certSteven Murawski2016-06-061-0/+1
| | |
* | | Don't mask directory deletion errors (#4991)Phil Dibowitz2016-06-061-1/+11
| | | | | | | | | closes #4988
* | | Creation of the new DataCollector reporterAdam Leff2016-06-023-0/+922
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The DataCollector reporter is a new method for exporting data about your Chef run. The details of this new feature can be found in [RFC 077](https://github.com/chef/chef-rfc/blob/master/rfc077-mode-agnostic-data-collection.md). Using the existing `EventDispatch` mechanics, the DataCollector reporter collects data about a Chef run (when it starts, when it ends, what resources were modified, etc.) and then POSTs them to a Data Collector server URL that can be specified in your Chef configuration. While similar functionality exists using the `ResourceReporter` and Chef Reporting, a new implementation was done to decouple the reporting of this data from requiring the use of a Chef Server (in the case of Chef Reporting), opening the door to users being able to implement their own webhook-style receiver to receive these messages and analyze them accordingly.
* | | Merge pull request #4957 from chef/tm/solo_fixesThom May2016-05-311-0/+25
|\ \ \ | | | | | | | | Ensure recipe-url works right in solo
| * | | Ensure recipe-url works right in solotm/solo_fixesThom May2016-05-201-0/+25
| |/ / | | | | | | | | | Signed-off-by: Thom May <thom@chef.io>
* | | Merge pull request #4908 from nathwill/systemd-unit-try-actionsThom May2016-05-311-0/+63
|\ \ \ | | | | | | | | add systemd_unit try-restart, reload-or-restart, reload-or-try-restart actions
| * | | add more unit actionsNathan Williams2016-05-061-0/+63
| | | |
* | | | Merge pull request #4966 from crigor/ebuild-version-characterThom May2016-05-311-0/+6
|\ \ \ \ | | | | | | | | | | Fix portage provider to support version with character
| * | | | Fix portage provider to support version with characterChristopher Rigor2016-05-241-0/+6
| | | | |
* | | | | fixes #4968 and only retrieves the latest version of packages from chocolateychoco_arMatt Wrock2016-05-271-15/+1
| | | | |
* | | | | break yum provider out into fileslcg/yum-readabilityLamont Granquist2016-05-251-0/+27
|/ / / /
* | | | * define exit codesSteven Murawski2016-05-233-4/+237
| |/ / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * exit code functional specs * audit exit codes * reboot now/reboot failed/reboot pending exit codes * Deal with forked and unforked process and get the right exit code * Reboot Now should really be reboot scheduled * pass exception rather than exit code * updated with sigint and sigterm * support legacy fatal!("", 2) behavior * fixup all fatal! and exit! calls
* | | Merge pull request #4902 from MichaelPereira/handle-user-id-git-resourceThom May2016-05-181-0/+36
|\ \ \ | | | | | | | | Handle numeric id for the user value in the git resource
| * | | remove extra space from expected unit test outputMichael Pereira2016-05-101-1/+1
| | | |
| * | | try @thommay suggestion for resolving unit test issueMichael Pereira2016-05-101-1/+1
| | | |
| * | | Handle numeric id for the user value in the git resourceMichael Pereira2016-05-051-0/+36
| | | |
View Lorry Controller Status