summaryrefslogtreecommitdiff
path: root/spec
Commit message (Collapse)AuthorAgeFilesLines
...
| * | | | | | | | | cleaned comments in specc52275322016-09-282-32/+0
| | | | | | | | | |
| * | | | | | | | | added (un)lock_package for apt and zypperc52275322016-09-283-34/+105
| | | | | | | | | |
| * | | | | | | | | implement lockingc52275322016-09-282-0/+55
| | | | | | | | | |
* | | | | | | | | | Update output assertion for environment set specstd/rfc018/add-field-separator-flagTom Duffield2016-10-281-1/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This test was originally written against a KnifeSupport that wasn't actually pulling in the instance variables (i.e. :attribute) when parsing the data. When running this command manually with Knife v12.15.19, I get the output as now described in this test. Signed-off-by: Tom Duffield <tom@chef.io>
* | | | | | | | | | Add the `--field-separator` flag to knife show commandsTom Duffield2016-10-283-5/+38
| |_|_|_|_|_|_|_|/ |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | For the commands that allow you to filter which attributes it shows you using the `--attribute` flag, add the `--field-separator` flag to allow the user to customize which character(s) they want to use to deliniate the attribute hierarchy. ```shell knife node show NODE -S: -a "packages:Chef Client v12.12.15" ``` Signed-off-by: Tom Duffield <tom@chef.io>
* | | | | | | | | Merge pull request #5394 from ↵Bryan McLellan2016-10-271-4/+13
|\ \ \ \ \ \ \ \ \ | |_|_|_|_|_|_|_|/ |/| | | | | | | | | | | | | | | | | | | | | | | | | | MsysTechnologiesllc/ali/package_resource_uninstall_fix Newly creating uninstall string using the product code value retrieved from the uninstall_string of the Windows registry to avoid using invalid uninstall_string.
| * | | | | | | | Newly creating uninstall string using the product code value retrieved from ↵aliasgar162016-10-131-4/+13
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | the uninstall_string of the Windows registry to avoid using invalid uninstall_string. Signed-off-by: aliasgar16 <aliasgar.batterywala@msystechnologies.com>
* | | | | | | | | rename __path to __path__ and __node to __node__lcg/node-path-trackingLamont Granquist2016-10-252-22/+22
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | consistency with Unchain here: https://github.com/chef/chef/blob/master/lib/chef/decorator/unchain.rb#L23-L24 and with SimpleDelegator: https://ruby-doc.org/stdlib-2.1.0/libdoc/delegate/rdoc/SimpleDelegator.html#method-i-__getobj__ Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
* | | | | | | | | add tests from #4207Lamont Granquist2016-10-251-0/+19
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | validates that we fixed the top_level_breadcrumb-state related bug which has been reported Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
* | | | | | | | | remove breadcrumb stateLamont Granquist2016-10-251-87/+24
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
* | | | | | | | | standardize initializer args for VividMash+ImmutableMashLamont Granquist2016-10-251-2/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | if anyone is using these directly this might be a breaking change, although it makes it considerably easier to use VividMash and IMO ImmutableMash is an implementation detail of the deep merge cache and Chef::Node object. we definitely have never documented these APIs, so I think the onus is on the consumer to update their code. VividMash.new() should now work. VividMash.new({ foo: :bar }) should also now work. IDK what object people would have been passing in as the root object before. Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
* | | | | | | | | add specsLamont Granquist2016-10-251-0/+84
| |_|/ / / / / / |/| | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
* | | | | | | | Merge pull request #5483 from chef/lcg/package-inline-resourcesLamont Granquist2016-10-241-3/+3
|\ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | Core: set use_inline_resources in package superclass
| * | | | | | | | remove excessive mockinglcg/package-inline-resourcesLamont Granquist2016-10-211-3/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | we have too much collaboration and state in our objects to write pure unit tests. Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
* | | | | | | | | Mock users in functional group testsCOOL-597/use-all-for-assertionTom Duffield2016-10-211-0/+1
|/ / / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Because `Provider::Group::Suse` expects users to exist, we need to mock out the `Etc` call that checks for our non-existent spec users. Signed-off-by: Tom Duffield <tom@chef.io>
* | | | | | | | Merge pull request #5476 from tas50/copyrightsLamont Granquist2016-10-215-7/+7
|\ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | Fix copyrights and update a few more Opscode references to Chef
| * | | | | | | | More Opscode -> Chef updatesTim Smith2016-10-204-6/+6
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Tim Smith <tsmith@chef.io>
| * | | | | | | | Fix a few bad copyrightsTim Smith2016-10-201-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | I think this happened when we merged the Opscode and Chef copyrights Signed-off-by: Tim Smith <tsmith@chef.io>
* | | | | | | | | Merge pull request #5475 from tas50/spec_typoLamont Granquist2016-10-211-1/+1
|\ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | Fix a typo in the apt_repository spec
| * | | | | | | | | Fix spec typoTim Smith2016-10-191-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Tim Smith <tsmith@chef.io>
* | | | | | | | | | Merge pull request #5472 from chef/COOL-597/address-suse-test-failuresTom Duffield2016-10-212-2/+94
|\ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | Address test failures in SUSE 11.4 and 12.1
| * | | | | | | | | | Raise error if suse group manager adds non-existent userCOOL-597/address-suse-test-failuresTom Duffield2016-10-201-0/+90
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The expected behavior of chef, as defined by our tests, is that if a user specified in a group resource does not exist than chef should raise an error. On suse, we use groupmod which will not raise an error if you attempt to add a non-existent user to a group. This change adds in a simple check that will raise an error if the specified does not exist. Signed-off-by: Tom Duffield <tom@chef.io>
| * | | | | | | | | | Update useradd test to account for new SUSE 12.1 behaviorTom Duffield2016-10-201-2/+4
| | |/ / / / / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The `usermod` binary in SUSE 12.1 no longer throws an error if you attempt to unlock a user account that has no password. Signed-off-by: Tom Duffield <tom@chef.io>
* | | | | | | | | | Merge pull request #5479 from chef/lcg/rubocop-0.44.1Lamont Granquist2016-10-202-3/+3
|\ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | fixes for rubocop 0.44.1 engine
| * | | | | | | | | | fixes for rubocop 0.44.1 enginelcg/rubocop-0.44.1Lamont Granquist2016-10-202-3/+3
| |/ / / / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | a few cops we have enabled got a little more accurate/stricter Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
* | | | | | | | | | Attributes: add tests for regression in 12.0 fixed recentlyLamont Granquist2016-10-201-0/+28
|/ / / / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Tests for the issue that led up to https://github.com/chef/chef/issues/5447 The Hash tests actually pass against 12.13.37 so it appearly that only broke the Immutable array objects in 12.0. This both explains why we didn't have any existing tests that caught it, and why it went uncaught for so long. It also adds some ammunition to the argument that fixing this in 12.14.x was not a major breaking change to users that had been relying on the buggy 12.0 behavior. Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
* | | | | | | | | Merge pull request #5463 from chef/tm/last_json_deprecationsThom May2016-10-191-3/+0
|\ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | [Knife] Fix deprecation warning when creating data bag items
| * | | | | | | | | Return a raw hash when creating objectstm/last_json_deprecationsThom May2016-10-181-3/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fixes chef/chef-vault#215 Signed-off-by: Thom May <thom@may.lt>
* | | | | | | | | | Implement `--config-option` CLI opt for knifedan/config-option-in-knifedanielsdeleo2016-10-181-0/+31
|/ / / / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The `--config-option` CLI option allows setting any `Chef::Config` setting on the command line, which allows applications to be used without a configuration file even if a desired config option does not have a corresponding CLI option. This CLI option was present in the help output for knife but was not actually implemented. Moving the behavior into chef-config allows us to use it for both `chef-client` and `knife`. Signed-off-by: Daniel DeLeo <dan@chef.io>
* | | | | | | | | Merge pull request #5430 from MsysTechnologiesllc/nim/double_quotes_issueMatt Wrock2016-10-141-1/+25
|\ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | Fixed double quotes issue in powershell_out
| * | | | | | | | | Verifying powershell stdout in specsNimishaS2016-10-101-0/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: NimishaS <nimisha.sharad@msystechnologies.com>
| * | | | | | | | | running powershell specs for only windowsNimishaS2016-10-071-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: NimishaS <nimisha.sharad@msystechnologies.com>
| * | | | | | | | | Fixed double quotes issue in powershell_outNimishaS2016-10-071-0/+22
| | |_|_|_|_|_|/ / | |/| | | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: NimishaS <nimisha.sharad@msystechnologies.com>
* | | | | | | | | Merge pull request #5452 from ↵Thom May2016-10-141-0/+6
|\ \ \ \ \ \ \ \ \ | |_|_|/ / / / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | phene/better-user-provider-comparison-of-home-paths User Provider: Fix home folder comparison
| * | | | | | | | Refactored Chef::Provider::User to short circuit and provider better ↵Geoffrey Hichborn2016-10-131-0/+6
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | comparison of home paths to fix Issue #5444 Signed-off-by: Geoffrey Hichborn <geoff@socialcast.com>
* | | | | | | | | use delayed_action in accumulator speclcg/delayed_actionLamont Granquist2016-10-111-6/+2
|/ / / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
* | | | | | | | add some integration testing around accumulatorsLamont Granquist2016-10-101-0/+236
| |_|_|_|_|_|/ |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | probably also our best documentation on how to write accumulators right now Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
* | | | | | | Adding support for rfc 62 exit code 213Jeremy J. Miller2016-10-061-0/+10
|/ / / / / / | | | | | | | | | | | | | | | | | | Signed-off-by: Jeremy J. Miller <jm@chef.io>
* | | | | | Merge pull request #5417 from chef/lcg/issue5089Thom May2016-10-061-0/+15
|\ \ \ \ \ \ | | | | | | | | | | | | | | Fix for "Chefspec template rendering fails when cookbook_name != directory name"
| * | | | | | add a spec for the fix to #5089Lamont Granquist2016-10-051-0/+15
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
* | | | | | | Revert Handling double quotes in powershell_outMatt Wrock2016-10-052-14/+37
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Matt Wrock <matt@mattwrock.com>
* | | | | | | Merge pull request #5408 from coderanger/solaris-mNoah Kantrowitz2016-10-061-1/+9
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | Fix solaris handling for useradd -m/-M behavior
| * | | | | | | Fix solaris handling for -m/-M behavior. Fixes #5403.Noah Kantrowitz2016-10-051-1/+9
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Noah Kantrowitz <noah@coderanger.net>
* | | | | | | | Merge pull request #5381 from ↵Bryan McLellan2016-10-052-37/+14
|\ \ \ \ \ \ \ \ | |_|/ / / / / / |/| | | | | | | | | | | | | | | | | | | | | | | MsysTechnologiesllc/nim/double_quotes_parsing_error Handling double quotes in powershell_out
| * | | | | | | Raising Mixlib::ShellOut::ShellCommandFailed exceptionNimishaS2016-10-032-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: NimishaS <nimisha.sharad@msystechnologies.com>
| * | | | | | | Removed unwanted templatesNimishaS2016-09-303-3/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: NimishaS <nimisha.sharad@msystechnologies.com>
| * | | | | | | Handling double quotes. Added specsNimishaS2016-09-305-37/+17
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: NimishaS <nimisha.sharad@msystechnologies.com>
* | | | | | | | Merge pull request #5319 from tas50/apt_update_specsThom May2016-10-041-17/+25
|\ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | Test apt_update on non-linux behavior
| * | | | | | | | Test apt_update on non-linux behaviorTim Smith2016-09-141-17/+25
| | |_|_|_|_|_|/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Make sure on non-linux without running which we still end up with a no-op Signed-off-by: Tim Smith <tsmith@chef.io>
* | | | | | | | Merge pull request #5245 from chef/jk/knife-cookbook-delete-testThom May2016-10-041-59/+48
|\ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | Don't let knife cookbook delete tests print to stdout
View Lorry Controller Status