summaryrefslogtreecommitdiff
path: root/spec/unit/provider/package
Commit message (Collapse)AuthorAgeFilesLines
* fixed exe package downloadsMatt Wrock2016-02-221-1/+13
|
* flip multiline function arguments aroundlcg/rubocop-0.37.2Lamont Granquist2016-02-197-99/+99
| | | | no enforced trailing comma on arguments...
* deal with 0.37.2 renamed copsLamont Granquist2016-02-186-58/+58
| | | | | | | | | | | | | | | 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?
* autofixing auto-inserted delimiterslcg/chefstyle-batchLamont Granquist2016-02-093-43/+43
|
* auto fixing some rubocopsLamont Granquist2016-02-093-43/+43
| | | | | | | | | Style/NegatedWhile Style/ParenthesesAroundCondition Style/WhileUntilDo Style/WordArray Performance/ReverseEach Style/ColonMethodCall
* autofixing whitespace copsLamont Granquist2016-02-0521-190/+160
| | | | | | | | | | | | | | | | | | | | | | 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.
* Update for chef.io domain in copyright lines too.Noah Kantrowitz2016-02-021-1/+1
|
* Update all auth email address from opscode.com and getchef.com to chef.io.Noah Kantrowitz2016-02-027-8/+8
| | | 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-028-8/+8
| | | Created via git ls-files | xargs perl -pi -e "s/(Copyright.*?), Opscode(,)? Inc(\.)?/\\1, Chef Software Inc./gi"
* Copyright year update for 2016 and massive cleanup.Noah Kantrowitz2016-02-0224-24/+24
| | | Generated via git ls-files | xargs perl -pi -e "s/[Cc]opyright (?:\([Cc]\) )?((?\!$(date +%Y))\\d{4})(-\\d{4})?([, ][ \d]+)*(,|(?= ))/Copyright \\1-$(date +%Y),/g"
* Fix nil with properties:John Keiser2016-01-271-21/+17
| | | | | | 1. Warn when default values are invalid. 2. Never validate nil (on set or get) if there is no default. 3. Emit "will be invalid in Chef 13" warning when setting an invalid nil value.
* zypper multipackage patchLamont Granquist2016-01-251-52/+26
|
* Added support for Zypper to install multiple packages at onceYorgos Saslis2016-01-251-0/+36
| | | | See #3570
* Autofixing Style/PercentLiteralDelimeterslcg/percentliteraldelimetersLamont Granquist2016-01-182-2/+2
| | | | | | See chef/chefstyle#11 for analysis and discussion. We select '{}' since audit of our source code shows that is the most common, and that used to be the dominant learning paradigm (e.g. in ruby 1.9 pickaxe book.
* fix chefstyle failuresLamont Granquist2016-01-151-2/+2
| | | | | | | | autocorrects some merges that will be broken when the next chefstyle is released. also runs chefstyle after the travis tests, because i'd rather know how the code is broken rather than how its not pretty.
* adding functional tests for chocolatey providerchoco_func_testsMatt Wrock2016-01-141-6/+19
|
* Use double quotes by defaultThom May2016-01-1424-523/+523
| | | | | | | This is an entirely mechanically generated (chefstyle -a) change, to go along with chef/chefstyle#5 . We should pick something and use it consistently, and my opinion is that double quotes are the appropriate thing.
* autocorrecting Style/TrailingCommalcg/trailing_commaLamont Granquist2016-01-137-84/+84
| | | | chefstyle -a fixed 1044 occurrances
* Merge pull request #4384 from chef/lcg/really-fix-copsLamont Granquist2016-01-121-8/+8
|\ | | | | really fix cops
| * really fix copslcg/really-fix-copsLamont Granquist2016-01-121-8/+8
| |
* | 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
* | fix some lint copsLamont Granquist2016-01-121-1/+1
|/ | | | | | | | | fixes the following offenses: 46 Lint/EndAlignment 21 Lint/BlockAlignment 3 Lint/SpaceBeforeFirstArg 1 Lint/DefEndAlignment
* chocolatey multipackage providerlcg/chocolatey-packageLamont Granquist2016-01-082-1/+423
|
* 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
| |
* | 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
|/
* Use properties in package resourcesjk/package-propertiesJohn Keiser2015-12-152-2/+0
|
* 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.
* dpkg multipackage and bonus fixesLamont Granquist2015-12-021-39/+40
| | | | | | | | - multipackages dpkg_package - fixes edge conditions in load-current-resource around purging/removing packages that threw errors before - fixes the ability to purge packages that have been removed - adds a lot of functional tests for dpkg_package
* dpkg provider cleanupLamont Granquist2015-11-201-85/+138
| | | | | | | | | | | | | | - :update and :install are now treated the same way and throw the same exceptions - :remove and :purge don't require the source at all, so don't do any checking on that - fix some convoluted side-effecty logic in load_current_resource - load_current_resource now correctly gets the dpkg state on :remove and :purge when the file does not exist (pretty sure the old logic did not) - fixed the FIXME about using en_US.UTF-8 (the default for shell_out!) - just use shell_out! to throw exceptions - clean up all the specs and remove all the instance vars from the code
* Use the proper python interpretor for yum-dump.py on Fedora 21+Tim Smith2015-11-051-0/+6
| | | | | | | | | | | Fedora 21+ use dnf as the primary package manager. Lamont added code in 12.5 to allow for a yum compat mode. This doesn't entirely work though as we need yum-dump.py to correctly run. We were parsing the shabang in the yum binary to find the path to python. On a dnf system the yum binary is a bash script though so we were trying to run yum-dump.py using bash which obviously fails. I added a fallback method to use python if the shebang parsing returns bash. With this in place AND the yum package installed you can use the package resource on Fedora.
* Use much simpler regex for determining the rpm versionIrving Popovetsky2015-09-291-1/+18
|
* fixes from code reviewLamont Granquist2015-08-201-0/+40
|
* add a test for the new provider behaviorLamont Granquist2015-08-201-1/+14
|
* green the broken testsLamont Granquist2015-08-201-23/+43
|
* Merge pull request #3383 from jfly/masterThom May2015-08-131-0/+18
|\ | | | | gem_package should install to the systemwide Ruby when using ChefDK.
| * Added some pretty straightforward tests of is_omnibus?Jeremy Fleischman2015-05-181-0/+18
| |
* | Use the output of dpkg-deb directly, rather than regextm/use-dpkg-debThom May2015-08-121-1/+8
| | | | | | | | Thanks to @kwilczynski for the original implementation
* | Make the install/uninstall tests run more realisticallydanielsdeleo2015-06-091-104/+162
| | | | | | | | | | Call all the methods Provider#run_action would call rather than just poking lower-level implementation.
* | Move 3503 regression tests into main describe blockdanielsdeleo2015-06-091-118/+59
| |
* | Clean up RPM package provider specsdanielsdeleo2015-06-091-76/+173
| | | | | | | | | | | | | | | | | | | | Most egregious problem was stubbing two shellout calls at once, which was confusing and let some shell_out calls slip though the stubbing. Also added a bogus catchall stub to ensure that all calls to shell out have a corresponding stub. Additionally, re-arranged some tests which were not in a sensible test context.