summaryrefslogtreecommitdiff
path: root/spec
Commit message (Collapse)AuthorAgeFilesLines
* Add specs for the openssl mixinTim Smith2018-08-161-11/+616
| | | | Signed-off-by: Tim Smith <tsmith@chef.io>
* Add unit tests for the EC public/private key resourcesTim Smith2018-08-152-0/+107
| | | | | | 95% the same as the rsa public/private key tests Signed-off-by: Tim Smith <tsmith@chef.io>
* Add a few specs to cron_manage and cron_dTim Smith2018-08-132-0/+62
| | | | Signed-off-by: Tim Smith <tsmith@chef.io>
* Fix the default action in the cron_access specTim Smith2018-08-131-1/+1
| | | | Signed-off-by: Tim Smith <tsmith@chef.io>
* Add cron_d and cron_access resources from the cron cookbookTim Smith2018-08-132-0/+64
| | | | | | Cron_d was the #1 requested resource that we already had written. Everyone uses it and we should ship it. It's not a replacement for the existing cron cookbook, but just a different way of managing cron. If people want to manage the whole file or individual snippets we'll give them the option, just as we've always done by having the core resource and this resource. Signed-off-by: Tim Smith <tsmith@chef.io>
* Merge pull request #7537 from ↵Tim Smith2018-08-131-3/+3
|\ | | | | | | | | MsysTechnologiesllc/ash/MSYS-880_Fix_issue_of_setting_comment_for_windows_user Fix issue of setting comment for windows user
| * Fix issue of setting comment for windows userNAshwini2018-08-101-3/+3
| | | | | | | | Signed-off-by: NAshwini <ashwini.nehate@msystechnologies.com>
* | Merge pull request #7529 from chef/lcg/rspec-testingLamont Granquist2018-08-134-6/+22
|\ \ | | | | | | Run rspec tests within a kitchen container on CentOS 7
| * | fix the mount unit testsLamont Granquist2018-08-101-0/+1
| | | | | | | | | | | | Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
| * | add centos-7 rspec testingLamont Granquist2018-08-103-6/+21
| |/ | | | | | | | | | | running rspec inside a test-kitchen provisioned centos-7 docker container Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
* | Update specs for the full path to profiles binaryTim Smith2018-08-101-3/+3
|/ | | | Signed-off-by: Tim Smith <tsmith@chef.io>
* add rdoc and testsLamont Granquist2018-08-061-0/+26
| | | | Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
* Merge pull request #7495 from ↵Tim Smith2018-07-312-1/+9
|\ | | | | | | | | MsysTechnologiesllc/piyush/added_downgrade_exception MSYS-858 : added warning if allow_downgrade set to be false and tried to install older version
| * MSYS-858 : added warning if allow_downgrade set to be false and tried to ↵piyushawasthi2018-07-262-1/+9
| | | | | | | | | | | | install older version Signed-off-by: piyushawasthi <piyush.awasthi@msystechnologies.com>
* | Merge pull request #7514 from ↵Tim Smith2018-07-311-0/+4
|\ \ | | | | | | | | | | | | MsysTechnologiesllc/vasundhara/MSYS-872_Fix_for_registry_key_resource_does_not_recongnize_keys_when_passed_as_attribute Fix for registry_key resource doesn't recognise keys when passed as attribute.
| * | MSYS-872 Fix for registry key resource doesn't recognise keys when passed as ↵Vasu11052018-07-311-0/+4
| | | | | | | | | | | | | | | | | | attribute Signed-off-by: Vasu1105 <vasundhara.jagdale@msystechnologies.com>
* | | [SHACK-304] Deprecation checking turns up false positivetyler-ball2018-07-311-0/+10
|/ / | | | | | | | | | | | | | | | | | | | | | | When the location string during a deprecation check looks like "/opt/chefdk/embedded/lib/ruby/gems/2.5.0/gems/chef-14.3.37/spec/unit/chef_class_spec.rb:141:in `block (4 levels) in <top (required)>'" then trying to catch silence deprecation warnings using only an index number (EG, "Chef::Config[:silence_deprecation_warnings] = [0]") can end up matching part of that location string incorrectly. In this case it matched the 0 in the ruby version. Signed-off-by: tyler-ball <tball@chef.io>
* | Merge pull request #7464 from ↵Tim Smith2018-07-252-0/+116
|\ \ | | | | | | | | | | | | MsysTechnologiesllc/vasundhara/MSYS-853_Add_support_for_setting_task_priority Add support for setting task priority
| * | Add support for setting task priorityvasu11052018-07-252-0/+116
| |/ | | | | | | Signed-off-by: vasu1105 <vasundhara.jagdale@msystechnologies.com>
* | Assume credentials supplied are still valid if they cannot be validated due ↵Stuart Preston2018-07-252-19/+139
|/ | | | | to a Windows account restriction (#7416) Signed-off-by: Stuart Preston <stuart@chef.io>
* Merge pull request #7498 from chef/sp/7440Tim Smith2018-07-231-4/+1
|\ | | | | Functional tests: Ensure we logon with the local security_user account
| * Chefstyle fixTim Smith2018-07-231-1/+1
| | | | | | | | Signed-off-by: Tim Smith <tsmith@chef.io>
| * Ensure we logon with the local security_user accountsp/7440Stuart Preston2018-07-231-4/+1
| | | | | | | | Signed-off-by: Stuart Preston <stuart@chef.io>
* | Merge pull request #7497 from ↵Tim Smith2018-07-231-8/+24
|\ \ | |/ |/| | | | | MsysTechnologiesllc/nimesh/MSYS-855_fixing_windows_task_quotes Handling Quotes in Windows Task Commands and Arguments
| * Handling quotes in Windows Task commandsnimesh-msys2018-07-231-8/+24
| | | | | | | | | | | | | | | | | | | | - Added a custom method to split the command and its arguments - Checked the beaviour when they are inclusive of single/double quotes - Added their specs to test the functionality. Covered a [tpec](https://github.com/chef/chef/compare/sp/7413) Fixes: MSYS-855 Signed-off-by: nimesh-msys <nimesh.patni@msystechnologies.com>
* | Merge pull request #7480 from chef/route_platform_familyTim Smith2018-07-181-8/+8
|\ \ | | | | | | Expand platform support for the route resource
| * | Expand platform support for the route resourceroute_platform_familyTim Smith2018-07-161-8/+8
| | | | | | | | | | | | | | | | | | Support all RHEL derivatives as well as Amazon. This gets us Amazon, Oracle, Scientific, Xen, and a few others. Signed-off-by: Tim Smith <tsmith@chef.io>
* | | Merge pull request #7488 from chef/functional_cleanupTim Smith2018-07-186-30/+27
|\ \ \ | |_|/ |/| | Misc cleanup to our functional tests
| * | Remove the RHEL 5 exclusionsTim Smith2018-07-162-24/+23
| | | | | | | | | | | | | | | | | | We don't test on RHEL 5 anymore. Signed-off-by: Tim Smith <tsmith@chef.io>
| * | Add the DNF checks for Amazon LinuxTim Smith2018-07-163-3/+3
| | | | | | | | | | | | | | | | | | At some point it's getting DNF too. Might as well add it to the mix so we don't have things break later. Signed-off-by: Tim Smith <tsmith@chef.io>
| * | Remove check to exclude tests on macos 10.6Tim Smith2018-07-162-2/+0
| | | | | | | | | | | | | | | | | | There's no need to check for this at this point. Signed-off-by: Tim Smith <tsmith@chef.io>
| * | Use entirely fake names in the group functional testsTim Smith2018-07-161-1/+1
| | | | | | | | | | | | | | | | | | We were burned by this before on Travis Signed-off-by: Tim Smith <tsmith@chef.io>
* | | Merge pull request #7484 from chef/attribute_to_propertiesTim Smith2018-07-1732-107/+107
|\ \ \ | | | | | | | | Update wording in resource specs for attributes -> properties
| * | | Update wording in resource specs for attributes -> propertiesTim Smith2018-07-1032-107/+107
| | | | | | | | | | | | | | | | Signed-off-by: Tim Smith <tsmith@chef.io>
* | | | Merge pull request #7474 from chef/group_propertiesTim Smith2018-07-172-28/+24
|\ \ \ \ | |_|_|/ |/| | | group: convert to properties with descriptions and improve comma separated parsing
| * | | Improve comma separated list splitting in group resourcegroup_propertiesTim Smith2018-07-171-16/+16
| | |/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This is the same thing I did in sudo. It just makes the string parsing a bit more user spacing proof. accepted before: - 1,2,3 accepted now: - 1,2,3 - 1, 2, 3 - 1 ,2 ,3 - any combo of the above 3 Signed-off-by: Tim Smith <tsmith@chef.io>
| * | Fix functional specs to set properties to empty arraysTim Smith2018-07-161-2/+2
| | | | | | | | | | | | | | | | | | These can't be directly set to nil anymore. Signed-off-by: Tim Smith <tsmith@chef.io>
| * | Simplify platform detection in the group functional specsTim Smith2018-07-161-5/+5
| | | | | | | | | | | | | | | | | | We're trying to catch everything linux here. Just catch that. Signed-off-by: Tim Smith <tsmith@chef.io>
| * | Remove the gating logic for testing on macOS 10.6Tim Smith2018-07-161-5/+1
| | | | | | | | | | | | | | | | | | This is ancient and we don't have to worry about it. Signed-off-by: Tim Smith <tsmith@chef.io>
* | | Remove the gateway propertyTim Smith2018-07-161-1/+1
| | | | | | | | | | | | Signed-off-by: Tim Smith <tsmith@chef.io>
* | | Merge branch 'vlan' of https://github.com/tomdoherty/chef into tomdoherty-vlanTim Smith2018-07-161-0/+2
|\ \ \
| * | | Allow specifying VLAN on RHEL/CentosTom Doherty2017-09-131-0/+2
| | | | | | | | | | | | | | | | Signed-off-by: Tom Doherty <tom.doherty@fixnetix.com>
| * | | Refactor and add an additional unit testTom Doherty2017-09-081-1/+11
| | | | | | | | | | | | | | | | Signed-off-by: Tom Doherty <tom.doherty@fixnetix.com>
| * | | Simplify and add unit test for default routeTom Doherty2017-09-081-0/+10
| | | | | | | | | | | | | | | | Signed-off-by: Tom Doherty <tom.doherty@fixnetix.com>
* | | | Merge pull request #6168 from oclaussen/generate_valid_gemfileTim Smith2018-07-161-1/+8
|\ \ \ \ | |_|/ / |/| | | Make gem_installer generate a valid Gemfile
| * | | Generate a valid Gemfile, even multiple gem declaration with keyword ↵Ole Claussen2017-05-291-1/+8
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | arguments exist. Previously, keyword argemunts to the gem declarations would just be appended, resulting in invalid Gemfiles. Now, keyword arguments are merged together. In case of conflict, the last declaration will win. Signed-off-by: Ole Claussen <claussen.ole@gmail.com>
* | | | Merge pull request #7466 from chef/knife_supermarketTim Smith2018-07-134-15/+19
|\ \ \ \ | | | | | | | | | | Move all knife cookbook site plugin logic into knife supermarket
| * | | | Move all plugin logic into knife supermarketknife_supermarketTim Smith2018-07-134-15/+19
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | We want to deprecate knife cookbook site. This moves all the logic into the knife supermarket plugins and has the legacy knife cookbook site plugins inherit from that. Signed-off-by: Tim Smith <tsmith@chef.io>
* | | | | Merge pull request #7465 from chef/lcg/fix-dup-query-paramsTim Smith2018-07-131-1/+7
|\ \ \ \ \ | | | | | | | | | | | | Fix duplicated query parameters to resolve Chef::HTTP::Simple regression
| * | | | | Fix duplicated query parametersLamont Granquist2018-07-131-1/+7
| |/ / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | rubocop rule led to us mutating the argument to the constructor here and continuously appending query params. Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>