summaryrefslogtreecommitdiff
path: root/spec/support/shared
Commit message (Collapse)AuthorAgeFilesLines
* Enable the data collector automaticallydanielsdeleo2016-11-021-3/+12
| | | | Signed-off-by: Daniel DeLeo <dan@chef.io>
* Add the `--field-separator` flag to knife show commandsTom Duffield2016-10-281-1/+4
| | | | | | | | | | | | | 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>
* Fix a few bad copyrightsTim Smith2016-10-201-1/+1
| | | | | | I think this happened when we merged the Opscode and Chef copyrights Signed-off-by: Tim Smith <tsmith@chef.io>
* add check for the centos why-run buglcg/linux-user-check-lockLamont Granquist2016-08-241-0/+3
| | | | Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
* rewrite linu_user provider check_lockLamont Granquist2016-08-241-38/+16
| | | | | | | | | | | | this removes the kinda brittle kinda insane parsing of the installed rpm version. now we're just less strict about the exit 1 code and fail on the output not being what is expected. this also likely fixes a bug in rhel in why-run mode where shell_out! would have failed on the user-not-found and raised an exception that would have ended the run. the code before certainly looks buggy. Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
* sort out the -m / -M mess a bit betterLamont Granquist2016-08-231-9/+9
| | | | | | -M is not valid for usermod Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
* clean up everythingLamont Granquist2016-08-231-5/+5
| | | | Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
* user provider manage_home behavior and refactorLamont Granquist2016-08-191-32/+20
| | | | | | | | | split out the user provider into 1:1 resources. fix the behavior of the linux user provider to implement manage_home behavior correctly. Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
* fix Style/BlockDelimiters, Style/MultilineBlockLayout and 0.42.0 engine upgradeLamont Granquist2016-08-1711-48/+52
| | | | Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
* fixing Style/ConstantName copLamont Granquist2016-08-161-4/+4
|
* fixing Lint/Void copLamont Granquist2016-08-161-2/+2
|
* fix Lint/UselessAccessModifier copLamont Granquist2016-08-161-2/+0
|
* autofixing Style/WhileUntilModifier copLamont Granquist2016-08-162-6/+2
|
* clear test directory before file resource examplesMatt Wrock2016-08-161-0/+1
|
* Revert "Merge pull request #5123 from chef/lcg/resolv-replace"John Keiser2016-08-081-25/+0
| | | | | This reverts commit e8877cd363642ed6757c48d1ed5ab35509d87e22, reversing changes made to 162d988d3bcfce2773ef2831fa33e416f53731f6.
* Cache ChefFS childrentm/cache_fs_childrenThom May2016-07-261-0/+4
| | | | Signed-off-by: Thom May <thom@chef.io>
* fix the spec filterLamont Granquist2016-07-191-1/+1
| | | | | | these platform filters really need some serious cleanup along the lines of the ruby version cleanup that i did, the number of these helpers is getting rediculous...
* only work around resolver caching on linuxLamont Granquist2016-07-191-2/+9
| | | | | it should be the only O/S which needs it due to the way its glibc was written. it is the only O/S that we've recieved any bugs for.
* replace glibc resolver with ruby resolverLamont Granquist2016-07-191-0/+18
| | | | also wires up file providers to reload /etc/reoslv.conf when it changes.
* Add most common knife commands to integration teststm/knife_integration_testsThom May2016-06-221-3/+10
| | | | Signed-off-by: Thom May <thom@chef.io>
* Revert "Run in fips mode if node is fips enabled"Thom May2016-04-271-8/+1
|
* Merge pull request #4642 from chef/ksubrama/fips_acceptanceKartik Null Cating-Subramanian2016-04-263-5/+5
|\ | | | | Run the integration tests in FIPS mode
| * Too much log output and unnecessary warnings! Suppress that shit.Kartik Null Cating-Subramanian2016-04-253-5/+5
| |
* | run in fips mode if node is fips enabledMatt Wrock2016-04-251-1/+8
|/
* chefstyle Style/AndOr fixesLamont Granquist2016-03-141-1/+1
| | | | | 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-042-2/+2
|\ | | | | Autofixing new Perf cops in 0.37.2
| * remove unused block argumentslcg/chefstyle-perfLamont Granquist2016-02-231-1/+1
| |
| * Autofixing new Perf cops in 0.37.2Lamont Granquist2016-02-231-1/+1
| | | | | | | | | | | | | | | | | | | | 6 Performance/Casecmp 18 Performance/Detect 1 Performance/RangeInclude 27 Performance/RedundantBlockCall 6 Performance/RedundantMatch 5 Performance/RedundantMerge 18 Performance/StringReplacement
* | better chef+ruby rspec constraintsLamont Granquist2016-03-032-6/+6
| | | | | | | | | | | | | | | | lets us use Gem::Dependency constraints directly as rspec constraints for matching Chef::Version and RUBY_VERSION. copies some magic sauce from bundler that does something similar
* | Revert "Merge pull request #4465 from ↵tm/revert_4465Thom May2016-03-021-4/+4
| | | | | | | | | | | | | | artursitarski/change-shell-for-locked-user" This reverts commit 94494e761763b877c3ef3b52eee9ac9df2186340, reversing changes made to a95a87d50c9e5af2fbd583afcf972880656d1a4b.
* | Add test for solo.dJay Mundrawala2016-02-241-0/+70
|/ | | | | | | This commit refactors the spec to use shared examples as the test for client.d and solo.d is the same. knife.d is a little special so it cannot use this.
* flip multiline function arguments aroundlcg/rubocop-0.37.2Lamont Granquist2016-02-192-4/+4
| | | | no enforced trailing comma on arguments...
* deal with 0.37.2 renamed copsLamont Granquist2016-02-183-9/+9
| | | | | | | | | | | | | | | 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 #4465 from artursitarski/change-shell-for-locked-userThom May2016-02-161-4/+4
|\ | | | | Change shell for locked user.
| * Updated specartursitarski2016-02-021-4/+4
| |
* | convert MultilineOperationIndentation style to "indented"Lamont Granquist2016-02-121-1/+1
| | | | | | | | this looks nicer.
* | pull rubocop 0.37.2 into chefstyleLamont Granquist2016-02-121-2/+2
| | | | | | | | | | this is from the same ruleset that we had, but the new code catches more conditions.
* | autofixing auto-inserted delimiterslcg/chefstyle-batchLamont Granquist2016-02-093-5/+5
| |
* | auto fixing some rubocopsLamont Granquist2016-02-097-14/+14
| | | | | | | | | | | | | | | | | | Style/NegatedWhile Style/ParenthesesAroundCondition Style/WhileUntilDo Style/WordArray Performance/ReverseEach Style/ColonMethodCall
* | autofixing whitespace copsLamont Granquist2016-02-0523-96/+78
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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
* | Update all auth email address from opscode.com and getchef.com to chef.io.Noah Kantrowitz2016-02-0217-26/+26
| | | | | | 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-0214-14/+14
| | | | | | 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-0223-23/+23
| | | | | | 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"
* | suppress reporter failures by using validate_utf8=falselcg/suppress-reporter-failuresLamont Granquist2016-01-261-0/+6
|/ | | | | | | - doesn't turn off the utf8 validation in the rest of the rest API - has to use a different rest object because we have to pass options in the constructor - the handling of Chef::Config in Chef::Client is just awful
* Autofixing Style/PercentLiteralDelimeterslcg/percentliteraldelimetersLamont Granquist2016-01-182-4/+4
| | | | | | 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.
* Use double quotes by defaultThom May2016-01-1424-211/+211
| | | | | | | 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-1313-26/+26
| | | | chefstyle -a fixed 1044 occurrances
* Move server and repository fs objects to their own directoriesJohn Keiser2016-01-131-8/+7
|
* really fix copslcg/really-fix-copsLamont Granquist2016-01-121-6/+6
|
* chefstyle: fix Lint/StringConversionInInterpolationLamont Granquist2016-01-121-1/+1
| | | | useless use of `"#{foo.to_s}"`
View Lorry Controller Status