summaryrefslogtreecommitdiff
path: root/spec/unit
Commit message (Collapse)AuthorAgeFilesLines
* deal with 0.37.2 renamed copsLamont Granquist2016-02-1863-275/+275
| | | | | | | | | | | | | | | 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 #4111 from mikedodge04/launchdPhil Dibowitz2016-02-172-0/+220
|\ | | | | launchd for osx
| * bundle exec chefstyle -aMike Dodge2016-02-152-51/+50
| |
| * fixed provider specMike Dodge2016-02-151-8/+11
| |
| * added specsMike Dodge2016-02-152-0/+218
| |
* | Merge pull request #4574 from chef/lcg/cookbook-shadow-deprecationLamont Granquist2016-02-162-2/+13
|\ \ | | | | | | 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
| * | fix specsLamont Granquist2016-02-121-1/+10
| | |
* | | Merge pull request #4586 from chef/tm/trailing_whitespaceThom May2016-02-161-1/+1
|\ \ \ | | | | | | | | fix trailing whitespace
| * | | fix trailing whitespacetm/trailing_whitespaceThom May2016-02-161-1/+1
| | | |
* | | | Merge pull request #4582 from chef/tm/prepend-chefThom May2016-02-161-0/+85
|\ \ \ \ | |/ / / |/| | | Prepend chef to tempfile name
| * | | add tempfile teststm/prepend-chefThom May2016-02-161-0/+85
| | |/ | |/|
* | | Merge pull request #4427 from juliandunn/remove-duplicate-shellcmddefLamont Granquist2016-02-162-17/+20
|\ \ \ | | | | | | | | Refactor test to remove duplicate definition of ShellCmdResult
| * | | Refactor test to remove duplicate definition of ShellCmdResultJulian C. Dunn2016-01-182-17/+20
| | | |
* | | | 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
| | | | |
* | | | | Merge pull request #4579 from chef/cmluciano/fixhttperrorThom May2016-02-161-2/+14
|\ \ \ \ \ | | | | | | | | | | | | Make sure we have a valid object before calling close!
| * | | | | Add test to ensure Tempfile does the right thingcmluciano/fixhttperrorThom May2016-02-161-2/+14
| | |_|/ / | |/| | |
* | | | | test that splitting a string on commas DTRTtm/split_group_membersThom May2016-02-161-0/+5
|/ / / /
* | | | 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
* | | | Merge pull request #4573 from chef/lcg/bump-rubocopLamont Granquist2016-02-129-181/+181
|\ \ \ \ | | | | | | | | | | pull rubocop 0.37.2 into chefstyle
| * | | | convert MultilineOperationIndentation style to "indented"Lamont Granquist2016-02-124-23/+23
| | | | | | | | | | | | | | | | | | | | this looks nicer.
| * | | | pull rubocop 0.37.2 into chefstyleLamont Granquist2016-02-126-163/+163
| | |_|/ | |/| | | | | | | | | | | | | | this is from the same ruleset that we had, but the new code catches more conditions.
* | | | Fix policyfile testsad/fix_policyfile_zeroandy-dufour2016-02-121-2/+6
|/ / /
* | | Merge pull request #4541 from chef/lcg/chefstyle-batchLamont Granquist2016-02-1177-300/+300
|\ \ \ | | | | | | | | fix a batch of cops
| * | | autofixing auto-inserted delimiterslcg/chefstyle-batchLamont Granquist2016-02-0970-276/+276
| | | |
| * | | auto fixing some rubocopsLamont Granquist2016-02-0977-300/+300
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Style/NegatedWhile Style/ParenthesesAroundCondition Style/WhileUntilDo Style/WordArray Performance/ReverseEach Style/ColonMethodCall
* | | | Merge pull request #4545 from chef/lcg/remove-rm-rfLamont Granquist2016-02-102-44/+29
|\ \ \ \ | | | | | | | | | | removing rm -rf in chef-solo recipe_url
| * | | | remove rm -rf in chef solo recipe_urlLamont Granquist2016-02-102-44/+29
| |/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | - deprecates '-r' used for the recipe_url in chef-solo - adds --delete-entire-chef-repo option for users who want the old behavior back. - cleans up some old code closes #3802 closes #1515 closes #1751
* | | | 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 some missed copsLamont Granquist2016-02-052-3/+3
| | | | | | | | | | | | second round of autofixing for some reason...
* | | autofixing whitespace copsLamont Granquist2016-02-05329-2551/+2263
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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
* | | Refactor rubygems_spec test and make it fastjdm/rubygems-testKartik Null Cating-Subramanian2016-02-041-254/+312
| | |
* | | WIP make rubygems unit specs unit likeJay Mundrawala2016-02-041-10/+49
| | |
* | | Chocolatey provider: Raise a MissingLibrary exception if we can't find ↵cd/choco-error-msgChris Doherty2016-02-031-1/+42
| | | | | | | | | | | | Chocolatey.
* | | Merge pull request #4523 from chef/chefstyle/fixesLamont Granquist2016-02-034-4/+4
|\ \ \ | |/ / |/| | auto-fixing some easy cops
| * | auto-fixing some easy copschefstyle/fixesLamont Granquist2016-02-034-4/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 3 Performance/CaseWhenSplat: Place when conditions with a splat at the end of the when branches. 1 Style/ElseAlignment: Align else with if. 1 Style/EvenOdd: Replace with Fixnum#even?. 1 Style/MethodDefParentheses: Use def with parentheses when there are parameters. 3 Style/PercentQLiterals: Do not use %Q unless interpolation is needed. Use %q. 3 Style/SpaceAfterColon: Space missing after colon. 2 Style/SpaceAfterMethodName: Do not put a space between a method name and the opening parenthesis. 2 Style/Tab: Tab detected. 1 Style/TrailingUnderscoreVariable: Do not use trailing _s in parallel assignment. 1 Style/UnneededCapitalW: Do not use %W unless interpolation is needed. If not, use %w.
* | | Merge branch 'smurawski/dsc_resource_converge_logging'Steven Murawski2016-02-032-9/+22
|\ \ \ | |/ / |/| |
| * | Allow PowerShellTypeCoercions to translate Chef::Node::ImmutableArraysmurawski/dsc_resource_converge_loggingSteven Murawski2016-02-031-0/+5
| | |
| * | Allow PowerShellTypeCoercions to translate Chef::Node::ImmutableMashSteven Murawski2016-02-031-9/+15
| | |
| * | include verbose output from the set method and shared function for accessing ↵Steven Murawski2016-02-031-0/+2
| | | | | | | | | | | | verbose log info.
* | | Allow multiple property_types in `is`jk/property-is-multiple-typesJohn Keiser2016-02-031-15/+76
| | |
* | | Merge pull request #4481 from chef/jdm/fips-58Jay Mundrawala2016-02-034-2/+95
|\ \ \ | |/ / |/| | Allow use of command line fips switch for knife
| * | Refactor chef-config fips configjdm/fips-58Jay Mundrawala2016-01-282-5/+5
| | | | | | | | | | | | | | | | | | | | | | | | This refactor allows for better mocking in the tests. Before, when init_openssl was called, classes would get monkey patched. While this may not cause problems now, some day in the future, somebody would have been like "WTF is going on".
| * | Allow use of command line fips switch for knifeJay Mundrawala2016-01-283-1/+94
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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.
* | | Merge pull request #4518 from coderanger/copyright-2016Noah Kantrowitz2016-02-02445-776/+776
|\ \ \ | | | | | | | | Copyright and author line update
| * | | Some more author domain stragglers.Noah Kantrowitz2016-02-029-9/+9
| | | |
| * | | Update for chef.io domain in copyright lines too.Noah Kantrowitz2016-02-023-3/+3
| | | |
| * | | Update all auth email address from opscode.com and getchef.com to chef.io.Noah Kantrowitz2016-02-02257-308/+308
| | | | | | | | | | | | Generated via git ls-files | xargs perl -pi -e "s/(Author.*?<[^@]+@)(?:opscode\\.com|getchef\\.com)(>)/\\1chef.io\\2/gi"
View Lorry Controller Status