summaryrefslogtreecommitdiff
path: root/spec/unit
Commit message (Collapse)AuthorAgeFilesLines
* yet more lint copslcg/lint-cops-againLamont Granquist2016-01-131-3/+3
| | | | | | | fixes: Lint/AmbiguousOperator Lint/DuplicateMethods
* Merge pull request #4344 from chef/if/warn-on-reload-resourceThom May2016-01-131-2/+2
|\ | | | | Warn (v. info) when reloading resources
| * debug log messages upon reloaded resourcesif/warn-on-reload-resourceIsa Farnik2016-01-121-2/+2
| | | | | | | | tests for debug messages when reloading resources
* | Merge pull request #4388 from chef/lcg/more-rubocop-lintThom May2016-01-131-8/+5
|\ \ | |/ |/| more rubocop fixes
| * more rubocop fixeslcg/more-rubocop-lintLamont Granquist2016-01-121-8/+5
| | | | | | | | | | | | | | | | | | | | | | fixes: Lint/UnreachableCode Lint/NonLocalExitFromIterator Lint/LiteralInCondition Lint/EmptyEnsure Lint/DefEndAlignment Lint/CircularArgumentReference
* | Merge pull request #4384 from chef/lcg/really-fix-copsLamont Granquist2016-01-123-19/+21
|\ \ | | | | | | really fix cops
| * | really fix copslcg/really-fix-copsLamont Granquist2016-01-123-19/+21
| |/
* | Merge pull request #4378 from chef/choco_editsMatt Wrock2016-01-121-3/+31
|\ \ | | | | | | assert candidates exist for alternate sources and when pinning versions
| * | assert candidates exist for alternate sources and when pinning versionsMatt Wrock2016-01-121-3/+31
| | |
* | | Merge pull request #4375 from chef/no_version_detectMatt Wrock2016-01-121-69/+5
|\ \ \ | |_|/ |/| | fixes #4354 and fixes #4372 stop trying to auto detect versions for exe installers
| * | fexes #4354 and fiexs #4372 stop trying to auto detect versions for exe ↵no_version_detectMatt Wrock2016-01-111-69/+5
| |/ | | | | | | installers
* | Merge pull request #4381 from chef/lcg/useless-to-sLamont Granquist2016-01-126-13/+13
|\ \ | | | | | | chefstyle: fix Lint/StringConversionInInterpolation
| * | chefstyle: fix Lint/StringConversionInInterpolationLamont Granquist2016-01-126-13/+13
| |/ | | | | | | useless use of `"#{foo.to_s}"`
* | fix some lint copsLamont Granquist2016-01-1213-15/+15
|/ | | | | | | | | fixes the following offenses: 46 Lint/EndAlignment 21 Lint/BlockAlignment 3 Lint/SpaceBeforeFirstArg 1 Lint/DefEndAlignment
* Merge pull request #4106 from chef/lcg/chocolatey-packageLamont Granquist2016-01-113-1/+490
|\ | | | | add chocolatey_package to core chef
| * chocolatey multipackage providerlcg/chocolatey-packageLamont Granquist2016-01-083-1/+490
| |
* | Convert all uses of Chef::REST to Chef::ServerAPItm/no_more_restThom May2016-01-1138-268/+258
|/ | | | | | | In the process, stop auto-expanding JSON in the HTTP client, and let individual classes control that themselves. Fixes #2737, Fixes #3518
* Merge pull request #4315 from bbbco/masterThom May2016-01-041-0/+20
|\ | | | | Add extra tests around whether to skip with multiple guards
| * Add extra tests around whether to skip with multiple guardsbbbco2015-12-151-0/+20
| |
* | Merge pull request #4321 from chef/service_userMatt Wrock2015-12-291-27/+18
|\ \ | | | | | | fix run_as_user of windows_service
| * | fixes #3521 correcting format of user name passed to policy apis and does ↵Matt Wrock2015-12-291-27/+18
| | | | | | | | | | | | not clobber existing service rights of other users
* | | Merge pull request #4333 from chef/vaultMatt Wrock2015-12-224-31/+41
|\ \ \ | | | | | | | | no longer wait on node search to refresh vault but pass created ApiCient instead
| * | | no longer wait on node search to refresh vault but pass created ApiClient ↵Matt Wrock2015-12-224-31/+41
| | | | | | | | | | | | | | | | instead
* | | | Added profile provider for OS XNate Walck2015-12-173-0/+329
|/ / / | | | | | | | | | Fixed resource to use let
* | | Merge pull request #4320 from chef/jk/fix-package-nil-warningJay Mundrawala2015-12-161-5/+1
|\ \ \ | |/ / |/| | Fix failing package unit test
| * | Fix failing package unit testjk/fix-package-nil-warningJohn Keiser2015-12-161-5/+1
| | |
* | | Merge pull request #4277 from chef/mwrock/packageMatt Wrock2015-12-151-7/+17
|\ \ \ | |_|/ |/| | non msi packages must explicitly provide a source attribute on install
| * | fixing post merge test breaksMatt Wrock2015-12-151-2/+2
| | |
| * | non msi packages must explicitly provide a source attribute on installMatt Wrock2015-12-151-5/+15
| |/
* | Add knife/ChefFS support for policyfiles and policy_groups.John Keiser2015-12-151-4/+0
|/
* Use properties in package resourcesjk/package-propertiesJohn Keiser2015-12-153-7/+1
|
* Merge pull request #4309 from chef/lcg/fix-tags-and-set-unlessThom May2015-12-151-1/+25
|\ | | | | tags always an array; fix set_unless
| * really always return an Arraylcg/fix-tags-and-set-unlessLamont Granquist2015-12-141-0/+12
| |
| * tags always an array; fix set_unlessLamont Granquist2015-12-141-1/+13
| | | | | | | | | | | | | | previously if the node had a { normal: { tags: nil } } that we read since the key existed (but was nil) we would not initialize with an array. replacing the code with a call to node.set_unless revealed that set_unless was similarly buggy. fixed both issues by fixing set_unless.
* | why is 'identity' different than 'name_property'?Lamont Granquist2015-12-141-1/+1
| |
* | Don't warn when setting a property to nil unless its value wouldjk/updates2John Keiser2015-12-141-12/+14
|/ | | | | actually change. Gets rid of cases where we are initializing a resource with values from another resource.
* Merge pull request #4273 from chef/jdm/default-auth-1.1Jay Mundrawala2015-12-112-42/+23
|\ | | | | Use signing protocol 1.1 by default
| * Use signing protocol 1.1 by defaultjdm/default-auth-1.1Jay Mundrawala2015-12-102-42/+23
| | | | | | | | | | | | | | | | | | | | | | All supported Chef servers support the 1.1 signing protocol. There is no reason to continue using 1.0, and removing it as the default allows us to remove a bunch of code that tries to upgrade 1.0 to 1.1 when the node name is too long. If the user specifies 1.0 as the auth protocol version from this point on, they will have to guarantee that the node name is not too long.
* | Merge pull request #4248 from chef/tm/fix_reg_tests_2Thom May2015-12-101-0/+3
|\ \ | | | | | | expose HTTP options
| * | expose HTTP optionstm/fix_reg_tests_2Thom May2015-12-091-0/+3
| | | | | | | | | | | | and use it in registration client specs
* | | Moving the proxy export to environment varibles into Chef::Configtyler-ball2015-12-092-181/+5
| | |
* | | Adding proxifier dependency and leveraging its ability to make TCPSocket.new ↵tyler-ball2015-12-093-4/+94
|/ / | | | | | | respect ENV['http_proxy']
* | Merge pull request #4236 from chef/jdm/dsc_resource_timeoutThom May2015-12-081-1/+7
|\ \ | | | | | | Add timeout attribute to dsc_resource
| * | Add timeout attribute to dsc_resourcejdm/dsc_resource_timeoutJay Mundrawala2015-12-031-1/+7
| | | | | | | | | | | | | | | | | | | | | | | | `dsc_resource` was missing a timeout resource. Some resources, such as `xSQLServerSetup` were taking longer than the default of 10 minuites set by mixlib-shellout. Fixes #4232
* | | Merge pull request #4193 from chef/mwrock/packageSalim Alam2015-12-083-36/+572
|\ \ \ | | | | | | | | adds support for installer types inno, nsis, wise and installshield to windows_package resource
| * | | adds support to installer types inno, nsis, wise and installshield top the ↵Matt Wrock2015-12-073-36/+572
| | |/ | |/| | | | | | | windows_package resource
* | | restore handling of dpkg -s returns codes (w/tests)lcg/dpkg-s-return-codesLamont Granquist2015-12-071-3/+38
|/ / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Previously dpkg -s would allow a 0 or 1 exit status and it wasn't entirely clear why this was so either from the code or the tests. This restores throwing an exception if we are outside of the [0,1] range, and then adds tests for both behaviors: - on old ubuntu/debian we get an exit(0) and output on stdout if the file is not installed - on newer ubuntu/debian we get an exit(1) and output on stderr if the file is not installed Added tests for both those cases, and the case where dpkg -s does some other kind of exitcode barf and we should raise.
* | fix registration teststm/fix_reg_testsThom May2015-12-071-4/+2
|/
* Merge pull request #4196 from chef/lcg/dpkg-multipackageLamont Granquist2015-12-023-48/+90
|\ | | | | multipackage dpkg_package and bonus fixes
| * quick spec for subclass directiveLamont Granquist2015-12-021-0/+45
| |