summaryrefslogtreecommitdiff
path: root/spec/unit/knife
Commit message (Collapse)AuthorAgeFilesLines
* Updating knife ssl fetch to correctly store certificates when it does not ↵2919/ssl_fetch_no_cntyler-ball2016-10-311-0/+38
| | | | | | contain a common name. Stores the certificate under the URI host instead of the common name in that case Signed-off-by: tyler-ball <tyleraball@gmail.com>
* Add the `--field-separator` flag to knife show commandsTom Duffield2016-10-281-0/+10
| | | | | | | | | | | | | 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>
* More Opscode -> Chef updatesTim Smith2016-10-201-1/+1
| | | | Signed-off-by: Tim Smith <tsmith@chef.io>
* add spec around node['class'] vs. node.class behaviorlcg/node-presenter-tweaksLamont Granquist2016-09-201-0/+7
| | | | | | we want attributes before methods Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
* add another spec for chef_environmentLamont Granquist2016-09-201-1/+8
| | | | Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
* fix method_access and array handling in node presenterLamont Granquist2016-09-201-1/+8
| | | | Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
* fix Style/BlockDelimiters, Style/MultilineBlockLayout and 0.42.0 engine upgradeLamont Granquist2016-08-1720-92/+99
| | | | Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
* autofixing Style/SpaceBeforeComment copLamont Granquist2016-08-161-2/+2
|
* Spec break on Windows due to temp dir and short path namesAdam Edwards2016-08-161-1/+1
|
* 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>
* Use Mixlib::Archive to extract tarballsThom May2016-07-071-0/+2
| | | | | | | 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 #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>
* | Deprecate `knife cookbook create` in favor of `chef generate cookbook`.tc/deprecate-chef-cookbook-generateTyler Cloke2016-06-101-0/+1
| |
* | Load cookbook versions correctly for knifetm/fix_cb_showThom May2016-05-161-90/+89
|/ | | | | | Fixes #4934 Signed-off-by: Thom May <thom@may.lt>
* fixes knife ssl check on windowsssl_check_winMatt Wrock2016-05-021-0/+16
|
* Too much log output and unnecessary warnings! Suppress that shit.Kartik Null Cating-Subramanian2016-04-251-2/+2
|
* Fix specsalam/knife-tweakSalim Alam2016-04-061-5/+6
|
* Revert "Spec break on Windows due to temp dir and short path names"Salim Alam2016-03-311-1/+1
| | | | This reverts commit e25b75e9fcd26ccc9c9094a43b614306de1047db.
* Spec break on Windows due to temp dir and short path namesadamedx/windows-ruby22-unit-tmpdirAdam Edwards2016-03-311-1/+1
|
* Merge pull request #4757 from tas50/better_mdLamont Granquist2016-03-311-48/+48
|\ | | | | Improve the docs generated by knife cookbook create
| * Chef the specsTim Smith2016-03-261-48/+48
| |
* | Add option to set ssh timeoutsalam/timeoutSalim Alam2016-03-291-0/+17
|/
* Merge pull request #4670 from tas50/knife_wording-3Tim Smith2016-03-232-3/+3
|\ | | | | Improve wording in Knife
| * Fix failing specsTim Smith2016-03-062-3/+3
| |
* | chefstyle Style/AndOr fixesLamont Granquist2016-03-142-4/+4
|/ | | | | this is part of our informal style guide, lets make it formal since clearly its not getting followed very well.
* Merge pull request #4617 from chef/lcg/chefstyle-perfLamont Granquist2016-03-041-2/+2
|\ | | | | Autofixing new Perf cops in 0.37.2
| * Autofixing new Perf cops in 0.37.2Lamont Granquist2016-02-231-2/+2
| | | | | | | | | | | | | | | | | | | | 6 Performance/Casecmp 18 Performance/Detect 1 Performance/RangeInclude 27 Performance/RedundantBlockCall 6 Performance/RedundantMatch 5 Performance/RedundantMerge 18 Performance/StringReplacement
* | Adding specs for rehash updatetyler-ball2016-03-042-0/+22
| |
* | Copy client.d files during bootstrapJay Mundrawala2016-02-241-0/+49
|/ | | | | When knife finds a client.d/ directory, it will upload all files nested under that directory.
* flip multiline function arguments aroundlcg/rubocop-0.37.2Lamont Granquist2016-02-191-3/+3
| | | | no enforced trailing comma on arguments...
* deal with 0.37.2 renamed copsLamont Granquist2016-02-1828-40/+40
| | | | | | | | | | | | | | | 252 Style/TrailingCommaInLiteral 84 Style/TrailingCommaInArguments 15 Style/SpaceAroundKeyword -- 351 Total We already dealt with SpaceAroundKeyword under its old name SpaceBeforeModifierKeyword, it looks like it got stricter about spaces after keywords. TrailingComma also got split, and it looks like the TrailingCommaInArguments behavior is new?
* Merge pull request #4574 from chef/lcg/cookbook-shadow-deprecationLamont Granquist2016-02-161-1/+3
|\ | | | | Extend cookbook shadowing deprecation warnings more broadly
| * don't warn about shadow cookbooks twiceLamont Granquist2016-02-161-1/+3
| | | | | | | | knife cookbook upload already warns once
* | Merge pull request #4524 from PatrickWalker/feature/4154Steven Murawski2016-02-161-1/+1
|\ \ | | | | | | Update to Log Level when showing unencrypted databag
| * | Update to Log Level when showing unencrypted databagPatrick Walker2016-02-031-1/+1
| | |
* | | Be way more explicit about how we're handling datatm/missed_migrationThom May2016-02-1515-21/+28
| | | | | | | | | | | | | | | | | | | | | This just codifies the behaviour we're actually using - that we're passing a json string and expecting a hash back. Also adds a deprecation warning to the use of Chef::JSONCompat.from_json
* | | convert MultilineOperationIndentation style to "indented"Lamont Granquist2016-02-121-3/+3
| | | | | | | | | | | | this looks nicer.
* | | pull rubocop 0.37.2 into chefstyleLamont Granquist2016-02-123-10/+10
| |/ |/| | | | | | | this is from the same ruleset that we had, but the new code catches more conditions.
* | Merge pull request #4541 from chef/lcg/chefstyle-batchLamont Granquist2016-02-1118-37/+37
|\ \ | | | | | | fix a batch of cops
| * | autofixing auto-inserted delimiterslcg/chefstyle-batchLamont Granquist2016-02-0918-35/+35
| | |
| * | auto fixing some rubocopsLamont Granquist2016-02-0918-37/+37
| | | | | | | | | | | | | | | | | | | | | | | | | | | Style/NegatedWhile Style/ParenthesesAroundCondition Style/WhileUntilDo Style/WordArray Performance/ReverseEach Style/ColonMethodCall
* | | Ensure we do streaming requests correctly everywheretm/site_download_regressionThom May2016-02-091-5/+5
| | |
* | | Use a streaming request to download cookbookThom May2016-02-091-6/+4
|/ / | | | | | | fixes #4549
* | autofixing whitespace copsLamont Granquist2016-02-0575-611/+544
|/ | | | | | | | | | | | | | | | | | | | | | 4174 Style/SpaceInsideHashLiteralBraces 1860 Style/SpaceAroundOperators 1336 Style/SpaceInsideBlockBraces 1292 Style/AlignHash 997 Style/SpaceAfterComma 860 Style/SpaceAroundEqualsInParameterDefault 310 Style/EmptyLines 294 Style/IndentationConsistency 267 Style/TrailingWhitespace 238 Style/ExtraSpacing 212 Style/SpaceBeforeBlockBraces 166 Style/MultilineOperationIndentation 144 Style/TrailingBlankLines 120 Style/EmptyLineBetweenDefs 101 Style/IndentationWidth 82 Style/SpaceAroundBlockParameters 40 Style/EmptyLinesAroundMethodBody 29 Style/EmptyLinesAroundAccessModifier 1 Style/RescueEnsureAlignment
* Merge pull request #4481 from chef/jdm/fips-58Jay Mundrawala2016-02-032-1/+42
|\ | | | | Allow use of command line fips switch for knife
| * Allow use of command line fips switch for knifeJay Mundrawala2016-01-282-1/+42
| | | | | | | | | | | | | | | | | | | | | | | | This somewhat worked before. However, it was just for knife bootstrap. It also didn't support --no-fips in the case where it was in your knife.rb and you decided that you didn't want to use fips for that one call. The assumption here is fips mode you run knife with is the fips mode the node will get. This has the nice property that validatorless bootstraps will talk to the chef server in fips mode if the node is requested to be in fips mode.
* | Update all auth email address from opscode.com and getchef.com to chef.io.Noah Kantrowitz2016-02-0258-64/+64
| | | | | | Generated via git ls-files | xargs perl -pi -e "s/(Author.*?<[^@]+@)(?:opscode\\.com|getchef\\.com)(>)/\\1chef.io\\2/gi"
* | Replace all Opscode copyrights with Chef Software.Noah Kantrowitz2016-02-0263-64/+64
| | | | | | Created via git ls-files | xargs perl -pi -e "s/(Copyright.*?), Opscode(,)? Inc(\.)?/\\1, Chef Software Inc./gi"