summaryrefslogtreecommitdiff
path: root/spec
Commit message (Collapse)AuthorAgeFilesLines
* Compensate for libyaml changes in yaml parsing test.fix-test-yaml-diffsPete Higgins2020-07-281-2/+6
| | | | Signed-off-by: Pete Higgins <pete@peterhiggins.org>
* Fix protocol negotiation for unversioned objectsLamont Granquist2020-07-281-6/+25
| | | | | | | | | We generally always do protocol negotiation on the get of the node object and the Chef::Node is an unversioned API, so we need to test that we backoff the API version to the highest version the server offers and then retry again. Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
* Merge pull request #10216 from chef/fix-406-no-method-errorTim Smith2020-07-271-0/+6
|\ | | | | Fix NoMethodError when server returns a 406.
| * Fix NoMethodError when server returns a 406.Pete Higgins2020-07-271-0/+6
| | | | | | | | Signed-off-by: Pete Higgins <pete@peterhiggins.org>
* | Define local test variables to avoid test load order bug.fix-spec-load-order-bugPete Higgins2020-07-272-16/+36
|/ | | | Signed-off-by: Pete Higgins <pete@peterhiggins.org>
* Merge pull request #10205 from chef/remove-some-test-cruftTim Smith2020-07-244-30/+2
|\ | | | | Remove some test cruft around diffing
| * Use built in constant.remove-some-test-cruftPete Higgins2020-07-242-3/+1
| | | | | | | | Signed-off-by: Pete Higgins <pete@peterhiggins.org>
| * Remove unnecessary check for `diff`.Pete Higgins2020-07-243-13/+1
| | | | | | | | Signed-off-by: Pete Higgins <pete@peterhiggins.org>
| * Remove some unnecessary test cleanup.Pete Higgins2020-07-241-14/+0
| | | | | | | | Signed-off-by: Pete Higgins <pete@peterhiggins.org>
* | add a more useful test.Lamont Granquist2020-07-241-0/+8
|/ | | | | | | | | | | this requires that 'ls' be installed on our test nodes. which it should. the fact that i feel i have to write this means docker is the root of all evil in the universe. Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
* requires_git just doesn't exist anywhere, its not definedlcg/git-spec-cleanupLamont Granquist2020-07-241-1/+1
| | | | Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
* Merge pull request #10199 from chef/blocklistTim Smith2020-07-231-11/+98
|\ | | | | Rename Attribute Whitelist/Blacklist to Allowed/Blocked
| * Add a basic test for the legacy attributesTim Smith2020-07-231-0/+87
| | | | | | | | Signed-off-by: Tim Smith <tsmith@chef.io>
| * Rename Attribute Whitelist/Blacklist to Allowlist/BlocklistTim Smith2020-07-231-12/+12
| | | | | | | | | | | | This is an overdue change. These are not terms we should be using as a company or community. To quote our Code of Conduct (https://community.chef.io/code-of-conduct/). "Be careful in the words that you choose. Be kind to others. Practice empathy." This change aims to do exactly that, while maintaining a level of backwards compatibility so we can transition existing users off the legacy configs. Signed-off-by: Tim Smith <tsmith@chef.io>
* | Merge pull request #10193 from chef/lcg/default-pathsTim Smith2020-07-232-21/+21
|\ \ | |/ |/| Implement default_paths API
| * Convert to default_paths APIlcg/default-pathsLamont Granquist2020-07-222-21/+21
| | | | | | | | Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
* | Set log level to :fatal in test to prevent excess logging.fix-excess-logging-in-functional-testPete Higgins2020-07-225-21/+7
| | | | | | | | Signed-off-by: Pete Higgins <pete@peterhiggins.org>
* | Merge pull request #10118 from chef/macos_userdefaultsTim Smith2020-07-221-2/+103
|\ \ | |/ |/| Rework macos_userdefaults resource
| * eql -> eq in the specsTim Smith2020-07-211-17/+17
| | | | | | | | Signed-off-by: Tim Smith <tsmith@chef.io>
| * Update spec/unit/resource/macos_user_defaults_spec.rb Tim Smith2020-07-211-1/+1
| | | | | | | | | | Signed-off-by: Tim Smith <tsmith@chef.io> Co-authored-by: pete higgins <pete@peterhiggins.org>
| * Make sure we can compare keys in dictsTim Smith2020-07-211-0/+5
| | | | | | | | Signed-off-by: Tim Smith <tsmith@chef.io>
| * Add spec for dict valuesTim Smith2020-07-201-0/+5
| | | | | | | | Signed-off-by: Tim Smith <tsmith@chef.io>
| * Add specs and squash bugs that showed upTim Smith2020-07-201-7/+32
| | | | | | | | | | | | | | | | - Add back method to convert boolean-ish values to TRUE/FALSE - Be explicit about string values as well as all the other types so we always pass a type - Fix the type flag in the write command Signed-off-by: Tim Smith <tsmith@chef.io>
| * Avoid failures if the host was named "current" and add some more testsTim Smith2020-07-201-1/+33
| | | | | | | | Signed-off-by: Tim Smith <tsmith@chef.io>
| * Add specs for defaults_export_cmdTim Smith2020-07-201-0/+22
| | | | | | | | Signed-off-by: Tim Smith <tsmith@chef.io>
| * Test the default property valuesTim Smith2020-07-201-0/+12
| | | | | | | | Signed-off-by: Tim Smith <tsmith@chef.io>
* | Merge pull request #10137 from chef/lcg/extract-mixlib-shelloutLamont Granquist2020-07-221-25/+24
|\ \
| * | fix shell_out specs to use an instance everywhere.Lamont Granquist2020-07-211-25/+24
| | | | | | | | | | | | Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
* | | Remove global state from cookbook uploader.fix-cookbook-uploader-global-statePete Higgins2020-07-211-3/+1
|/ / | | | | | | Signed-off-by: Pete Higgins <pete@peterhiggins.org>
* | fix functional specVSingh/bootstrap-retry-limitVivek Singh2020-07-211-1/+1
| | | | | | | | Signed-off-by: Vivek Singh <vivek.singh@msystechnologies.com>
* | Remove unused codeVivek Singh2020-07-211-1/+0
| | | | | | | | Signed-off-by: Vivek Singh <vivek.singh@msystechnologies.com>
* | Fix test casesVivek Singh2020-07-211-5/+2
|/ | | | Signed-off-by: Vivek Singh <vivek.singh@msystechnologies.com>
* Merge pull request #10162 from ↵Tim Smith2020-07-201-2/+30
|\ | | | | | | | | MsysTechnologiesllc/dh/dont-check-list-profile-for-already-selected-bad-profile don't check list-profiles for already selected bad profile
| * don't check list-profiles for already selected bad profiledheerajd-msys2020-07-151-2/+30
| | | | | | | | Signed-off-by: dheerajd-msys <dheeraj.dubey@msystechnologies.com>
* | Rename the method for more pizazz.reset-logger-in-testPete Higgins2020-07-171-1/+1
| | | | | | | | Signed-off-by: Pete Higgins <pete@peterhiggins.org>
* | Remove an empty test file.Pete Higgins2020-07-161-24/+0
| | | | | | | | Signed-off-by: Pete Higgins <pete@peterhiggins.org>
* | Reset logger in test to avoid global state persisting.Pete Higgins2020-07-166-35/+29
| | | | | | | | Signed-off-by: Pete Higgins <pete@peterhiggins.org>
* | Merge pull request #10153 from chef/raise-on-raise_errorTim Smith2020-07-155-11/+9
|\ \ | | | | | | Don't allow tests to set an expectation without specific error type
| * | Remove a deprecated use of `.ordered` on an `allow` expectation.raise-on-raise_errorPete Higgins2020-07-151-1/+0
| | | | | | | | | | | | Signed-off-by: Pete Higgins <pete@peterhiggins.org>
| * | Don't allow expectations of raising an error without an error type.Pete Higgins2020-07-144-10/+9
| | | | | | | | | | | | Signed-off-by: Pete Higgins <pete@peterhiggins.org>
| * | Give an rspec file the proper suffix.Pete Higgins2020-07-141-0/+0
| | | | | | | | | | | | Signed-off-by: Pete Higgins <pete@peterhiggins.org>
* | | Merge pull request #9994 from chef-davin/windows_firewall_profileTim Smith2020-07-151-0/+77
|\ \ \ | | | | | | | | Create the windows_firewall_profile resource for use with enabling/disabling and configuring the Windows firewall Domain, Private, and Public profiles
| * \ \ Merge branch 'master' of github.com:chef/chef into windows_firewall_profileDavin Taddeo2020-07-1513-27/+49
| |\ \ \
| * \ \ \ Merge branch 'master' of github.com:chef/chef into windows_firewall_profileDavin Taddeo2020-07-136-23/+50
| |\ \ \ \
| * | | | | fix the name of the windows_firewall_profile spec.Davin Taddeo2020-07-101-16/+16
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Update the tests to accept ruby true/false as the correct (non-erroring) value Signed-off-by: Davin Taddeo <davin@chef.io>
| * | | | | Merge branch 'master' of github.com:chef/chef into windows_firewall_profileDavin Taddeo2020-07-1025-101/+85
| |\ \ \ \ \
| * \ \ \ \ \ Merge branch 'master' of github.com:chef/chef into windows_firewall_profileDavin Taddeo2020-07-0279-290/+365
| |\ \ \ \ \ \
| * \ \ \ \ \ \ Merge branch 'master' of github.com:chef/chef into windows_firewall_profileDavin Taddeo2020-06-245-23/+27
| |\ \ \ \ \ \ \
| * \ \ \ \ \ \ \ Merge branch 'master' of github.com:chef/chef into windows_firewall_profileDavin Taddeo2020-06-207-54/+142
| |\ \ \ \ \ \ \ \
| * | | | | | | | | unit test fix and chefstyle fixesDavin Taddeo2020-06-151-10/+10
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Davin Taddeo <davin@chef.io>