summaryrefslogtreecommitdiff
path: root/spec/functional/resource
Commit message (Collapse)AuthorAgeFilesLines
* enable HTTP winrm listener for dsc functional testsdsc_listenerMatt Wrock2016-02-231-0/+10
|
* flip multiline function arguments aroundlcg/rubocop-0.37.2Lamont Granquist2016-02-193-3/+3
| | | | no enforced trailing comma on arguments...
* deal with 0.37.2 renamed copsLamont Granquist2016-02-182-5/+5
| | | | | | | | | | | | | | | 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?
* skip uid check for aix in user add functional testsMatt Wrock2016-02-181-16/+9
|
* change shell to /bin/sh in user_add tests to fix aixMatt Wrock2016-02-161-1/+1
|
* Merge pull request #4465 from artursitarski/change-shell-for-locked-userThom May2016-02-161-0/+3
|\ | | | | Change shell for locked user.
| * Updated specartursitarski2016-02-021-0/+3
| |
* | pull rubocop 0.37.2 into chefstyleLamont Granquist2016-02-121-1/+1
| | | | | | | | | | this is from the same ruleset that we had, but the new code catches more conditions.
* | do not include source parameter when removing a chocolatey package and ↵no_webMatt Wrock2016-02-111-4/+10
| | | | | | | | ensure source is used on all functional tests
* | Merge pull request #4541 from chef/lcg/chefstyle-batchLamont Granquist2016-02-1116-35/+35
|\ \ | | | | | | fix a batch of cops
| * | autofixing auto-inserted delimiterslcg/chefstyle-batchLamont Granquist2016-02-0910-21/+21
| | |
| * | auto fixing some rubocopsLamont Granquist2016-02-0916-35/+35
| | | | | | | | | | | | | | | | | | | | | | | | | | | Style/NegatedWhile Style/ParenthesesAroundCondition Style/WhileUntilDo Style/WordArray Performance/ReverseEach Style/ColonMethodCall
* | | Respect TMPDIR in functional testsJohn Keiser2016-02-103-11/+11
|/ /
* | autofixing whitespace copsLamont Granquist2016-02-0525-216/+205
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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
* | auto-fixing some easy copschefstyle/fixesLamont Granquist2016-02-031-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | 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.
* | fixes #4515 and use FormatMessage to get correct error messages from native ↵invalidMatt Wrock2016-02-031-1/+1
| | | | | | | | API instead of retyping them with our fingers
* | Update all auth email address from opscode.com and getchef.com to chef.io.Noah Kantrowitz2016-02-0220-21/+21
| | | | | | 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-0223-23/+23
| | | | | | 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-0235-35/+35
| | | | | | 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"
* | Mark windows_service_spec as brokenJay Mundrawala2016-02-021-1/+2
| | | | | | | | | | This test is causing the rspec tests in appveyor to exit with 116
* | Use OHAI_SYSTEM where possibleJay Mundrawala2016-02-022-9/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Rerunning ohai wastes a lot of time. This should get us back to within our time limits in appveyor even with the perf regression in Ohai. Don't rerun ohai for unit/regsitry_helper_spec Ohai is slow. We don't need to rerun this, it's already run in our spec_helper. Don't keep rerunning ohai for reboot_pending_spec Running Ohai is slow. This should know almost 2 mins off the runtime of the tests. Don't need to run ohai for registry_spec crypto_spec does not need its own ohai cmdlet_spec does not need its own ohai functional registry_helper_spec does not need its own ohai Use spec ohai system for knife configure_spec Use OHAI_SYSTEM for dsc_script_spec use OHAI_SYSTEM for dsc_resource_spec
* | Don't rerun ohai for ohai_spec when not neededjdm/ohai-func-specJay Mundrawala2016-02-011-3/+1
|/ | | | Running ohai is slow
* Set range of system user/group id to max of 200Jay Mundrawala2016-01-261-3/+3
| | | | | | | | | | | | | | According to https://abcofaix.wordpress.com/tag/usermod/, the value can be up to 200. We were seeing tests on our aix testers fail with ``` 1) Chef::Provider::User::Useradd action :create when the user does not exist beforehand when a system user is specified ensures the user has the properties of a system user Failure/Error: expect(pw_entry.uid.to_i).to be < uid_min.to_i expected: < 100 got: 101 # ./spec/functional/resource/user/useradd_spec.rb:306:in `block (5 levels) in <top (required)>' ```
* wrlinux is a derivative of RHEL and as such the default creation of user ↵tball/nexus7_more_failurestyler-ball2016-01-251-1/+1
| | | | home directory is expected
* Using random names in the group_spec so we don't blow away a real accounttyler-ball2016-01-211-3/+3
|
* Test failures for the Cisco Nexus 7 platform in Manhattan. It cannot have ↵tyler-ball2016-01-212-8/+8
| | | | uppercase users
* Sometimes chocolately doesn't appear on the path, so adding two fixes.tball/win2008r2_test_failurestyler-ball2016-01-201-0/+3
| | | | | First, making powershell raise if there is an exception installing. Second, add choco to the path if it isn't already there.
* die die die if functional test fails to install chocolateywhy_choco_no_installMatt Wrock2016-01-201-2/+2
|
* Autofixing Style/PercentLiteralDelimeterslcg/percentliteraldelimetersLamont Granquist2016-01-182-13/+13
| | | | | | 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-31/+31
| | | | | | | | 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.
* fix candidate version filteringMatt Wrock2016-01-141-6/+32
|
* adding some functional tests for chocolatey_packageMatt Wrock2016-01-141-0/+95
|
* Use double quotes by defaultThom May2016-01-1434-521/+521
| | | | | | | 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-7/+7
| | | | chefstyle -a fixed 1044 occurrances
* Merge pull request #4384 from chef/lcg/really-fix-copsLamont Granquist2016-01-121-6/+6
|\ | | | | really fix cops
| * really fix copslcg/really-fix-copsLamont Granquist2016-01-121-6/+6
| |
* | Merge pull request #4375 from chef/no_version_detectMatt Wrock2016-01-121-12/+3
|\ \ | |/ |/| 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-12/+3
| | | | | | | | installers
* | Merge pull request #4381 from chef/lcg/useless-to-sLamont Granquist2016-01-121-1/+1
|\ \ | | | | | | chefstyle: fix Lint/StringConversionInInterpolation
| * | chefstyle: fix Lint/StringConversionInInterpolationLamont Granquist2016-01-121-1/+1
| |/ | | | | | | useless use of `"#{foo.to_s}"`
* | fix some lint copsLamont Granquist2016-01-122-3/+3
|/ | | | | | | | | fixes the following offenses: 46 Lint/EndAlignment 21 Lint/BlockAlignment 3 Lint/SpaceBeforeFirstArg 1 Lint/DefEndAlignment
* Convert all uses of Chef::REST to Chef::ServerAPItm/no_more_restThom May2016-01-111-1/+1
| | | | | | | 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 #4340 from juliandunn/reenable-aix-mount-testBryan McLellan2015-12-311-20/+8
|\ | | | | Re-enable AIX mount functional test
| * Re-enable AIX mount functional test by using the NameFS type whichJulian C. Dunn2015-12-231-20/+8
| | | | | | | | works inside both LPARs and WPARs.
* | Merge pull request #4321 from chef/service_userMatt Wrock2015-12-291-2/+5
|\ \ | |/ |/| fix run_as_user of windows_service
| * fixes #3521 correcting format of user name passed to policy apis and does ↵Matt Wrock2015-12-291-2/+5
| | | | | | | | not clobber existing service rights of other users
* | Filter out service test that cannot run inside a WPARshain/disable_aix_mount_testsScott Hain2015-12-181-2/+2
| |
* | Disabled mount tests on AIX since they only work in LPARs and we test in ↵Scott Hain2015-12-181-1/+2
|/ | | | WPARs. In the future we should find a way to test this code. One cannot create a ramdisk in a WPAR, since it shares filesystem with the LPAR host.
* merge cleanupmwrock/packageMatt Wrock2015-12-151-1/+1
|
* Merge pull request #4258 from chef/jdm/dpkg-spec-fixesLamont Granquist2015-12-081-22/+25
|\ | | | | Fix dpkg specs to run on debian 6