summaryrefslogtreecommitdiff
path: root/spec
Commit message (Collapse)AuthorAgeFilesLines
* Prevent inspect on PsCredential from printing out plain text passwordjdm/dont-print-pscredJay Mundrawala2015-11-231-1/+8
|
* 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
* Run pedant from chef-zero directly instead of having our ownJohn Keiser2015-11-184-222/+0
|
* WMF 5 RTM and Win 10 Threshold 2 allow the RefreshMode to be enabled.Steven Murawski2015-11-181-41/+62
|
* Add reboot notification to dsc_resourceSteven Murawski2015-11-182-7/+30
|
* base package provider fixesLamont Granquist2015-11-172-366/+499
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * removes the mutation of the new_resource.version * adds package_class_supports_arrays for multipackage providers * cleans up the package resource initializer and sets the package_name correctly through the accessor method. By mutating new_resource.version we were destroying the original intent of what the user was requesting. This is a bug that must be fixed. Generally, subclasses will be able to get the correct information they need through either the version argument they were passed in install_package(name, version) instead, or through their `#target_version_array` method. If this breaks anything then those providers need bugfixes as well (since this is a change to an internal API accessible only to subclassing, and there are backwards compatible and correct ways to get the information, this is *not* a SemVer violating change). See the fix I made to the OpenBSD provider to preserve the same semantics, avoid using new_resource.version where it was inappropriate, and make the code more symmetrical where before substantially different looking code in install_package and remove_package had exactly the same ultimate effect. The package_class_supports_arrays 'DSL' for writing multipackage providers coerces the arguments to the virtual methods (e.g. install_package) into arrays, even if the user is only requesting a single package install. This removes most of the `is_a?(Array)` checks from the implementation subclasses. The cleanup of the initializer is similarly necessary so that we can use Chef 12.5 coercions to make the package_name and version arguments accept string, but always be Arrays. This should eliminate the rest of the need to write `is_a?(Array)` code all over the package provider implementations.
* Restore rspec 3.4 by setting project_source_dirsjk/rspec-3.4John Keiser2015-11-171-0/+4
| | | | to be the cookbooks (see http://rspec.info/blog/2015/11/rspec-3-4-has-been-released/, Better Failure Source Detection, for more info)
* Merge pull request #4040 from chef/tm/live_execute_streamingThom May2015-11-123-36/+99
|\ | | | | Implement live streaming for execute resources
| * respond to review commentsThom May2015-11-092-64/+75
| | | | | | | | | | | | | | | | | | | | the logic is now: if the resource is not sensitive, and if it's explicitly requested to be streamed or if the log level is info or debug, then we'll consider streaming it. If we're configured to send the output to the events stream, we'll do so. Otherwise, if we're not daemonized and have a TTY, we'll go to STDOUT
| * Implement code review comments and fix failing testsThom May2015-11-091-2/+2
| |
| * Implement live streaming for execute resourcesAdam Jacob2015-11-092-26/+78
| | | | | | | | | | | | | | | | | | | | This brings live streaming of execute resource output to the output formatters. It also adds a mechanism for checking to see if an output formatter is in use through the event dispatch system. It adds a new configuration option, "always_stream_execute", which does what it says on the tin.
* | Merge pull request #3991 from chef/jdm/remote-file-sha2Thom May2015-11-121-36/+62
|\ \ | | | | | | Modify remote_file cache_control_data to use sha256 for its name
| * | Update cache_control_data_ specs to use sha256 for path namejdm/remote-file-sha2Jay Mundrawala2015-09-281-36/+62
| | |
* | | Merge pull request #4014 from nsdavidson/issue-3923Thom May2015-11-121-0/+40
|\ \ \ | | | | | | | | Adding ksh resource for #3923
| * | | updating copyright statment in ksh_spec.rbNolan Davidson2015-10-131-1/+1
| | | |
| * | | Adding ksh resource and specsNolan Davidson2015-10-061-0/+40
| | | |
* | | | Merge pull request #4142 from tas50/masterThom May2015-11-121-0/+6
|\ \ \ \ | | | | | | | | | | Use the proper python interpretor for yum-dump.py on Fedora 21+
| * | | | 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.
* | | | | Merge pull request #4064 from josb/masterThom May2015-11-123-9/+13
|\ \ \ \ \ | | | | | | | | | | | | Use the initializer to avoid NoMethodError on nil.include?
| * | | | | Route all tags access through Chef::Node#{tags,tag}Jos Backus2015-10-202-7/+7
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This should avoid node tags not being initialized properly to an empty Array.
| * | | | | Stop initializing node tags hereJos Backus2015-10-202-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | This should be done in one place (Chef::Node#tags).
| * | | | | Add a test for correct tags initializationJos Backus2015-10-201-0/+4
| | | | | |
* | | | | | fix log location resolution in windows serviceMatt Wrock2015-11-091-3/+4
| |_|_|_|/ |/| | | |
* | | | | Merge pull request #4137 from chef/btm/knife-backtraceBryan McLellan2015-11-061-1/+14
|\ \ \ \ \ | | | | | | | | | | | | don't squash Chef::Config[:verbosity] on subsequent instances of Chef::Knife::Bootstrap
| * | | | | Avoid squashing/humanizing the exception when config[:verbosity] == 2btm/knife-backtraceBryan McLellan2015-11-061-1/+14
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | We create an instance of Chef::Knife::Bootstrap in knife-ec2, but we've already processed ARGV so we substitute and process an empty argv. config[:verbosity] comes from Chef::Application::Knife. Normally we merge those options with the ones from the knife subcommand in Chef::Knife.run. Since we don't in this case, we don't set it to the default of 0 that is specified there, leaving it nil. We set Chef::Config[:verbosity] to config[:verbosity] in Chef::Knife#initialize which now becomes nil. This change makes it so we do not update Chef::Config[:verbosity] if config[:verbosity] is nil, so that later when we rescue we don't call humanize_exception if Chef::Config[:verbosity] is 2.
* | | | | | Handle nil run list option in knife bootstrapnil-run-list-bootstrapdanielsdeleo2015-11-051-0/+15
|/ / / / /
* | | | | Fixed knife_spec unit testJay Mundrawala2015-11-051-1/+4
| | | | | | | | | | | | | | | | | | | | | | | | | This is failing on both my and btm's machine. No idea how it passes in other places.
* | | | | Merge pull request #3836 from nathwill/svc-helpers-altThom May2015-11-051-30/+6
|\ \ \ \ \ | |_|/ / / |/| | | | simplify service helpers
| * | | | fix specsNathan Williams2015-10-231-1/+1
| | | | |
| * | | | i have no idea what is happeningNathan Williams2015-10-231-30/+6
| | | | |
* | | | | windows service uses log fileMatt Wrock2015-11-051-46/+82
| | | | | | | | | | | | | | | | | | | | location from config if none is given on commandline
* | | | | Add tests for Chef::HTTP retry behaviordanielsdeleo2015-11-041-1/+103
| | | | |
* | | | | Improve detection of ChefFS-based commands in `knife rehash`ssd/rehash-cheffsSteven Danna2015-10-291-0/+8
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | ChefFS-based commands have a superclass (Chef::ChefFS::Knife) which defines its own inherited method that calls super. This breaks our detection of where the subcommand is defined since the file with the definition is no longer at the top of the call stack. This commit special-cases subclasses with a superclass of Chef::ChefFS::Knife to account for this. Fixes #4089
* | | | | Make Property.derive create derived properties of the same typejk/custom_propertiesJohn Keiser2015-10-281-0/+48
| | | | |
* | | | | Updated --no-color unit tests based on feedback from btm and KartikTyler Fitch2015-10-281-11/+11
| | | | |
* | | | | Update knife bootstrap command to honor --no-color flag in chef-client run ↵Tyler Fitch2015-10-281-4/+11
| | | | | | | | | | | | | | | | | | | | that is part of the bootstrap process.
* | | | | Merge pull request #4081 from chef/lcg/chef-versionLamont Granquist2015-10-285-3/+206
|\ \ \ \ \ | | | | | | | | | | | | RFC-037: add chef_version and ohai_version metadata
| * | | | | add integration test for the client sidelcg/chef-versionLamont Granquist2015-10-261-0/+20
| | | | | |
| * | | | | add YARD docsLamont Granquist2015-10-261-1/+1
| | | | | |
| * | | | | stub #validate! methodLamont Granquist2015-10-261-0/+2
| | | | | |
| * | | | | fix some busted specsLamont Granquist2015-10-262-4/+4
| | | | | |
| * | | | | improve error message, add more testsLamont Granquist2015-10-263-2/+113
| | | | | |
| * | | | | extend metadata to include chef_version and ohai_versionLamont Granquist2015-10-261-1/+71
| | | | | |
* | | | | | Change plural to singularJohn Keiser2015-10-271-7/+7
| | | | | |
* | | | | | test for default valuechasebolt-sensitiveChase Bolt2015-10-271-0/+24
| | | | | |
* | | | | | tests and styleChase Bolt2015-10-271-0/+48
|/ / / / /
* | | | | Use SHA256 instead for registry_key when data is not displayableJay Mundrawala2015-10-241-2/+2
| | | | |
* | | | | fix tests for my changeAubrey Holland2015-10-241-1/+1
| | | | |
* | | | | We shouldn't be subclassing Struct.new - if the file is loaded twice we get ↵tyler-ball2015-10-241-45/+44
| | | | | | | | | | | | | | | | | | | | a superclass mismatch error
* | | | | write spec for --json-attribute-file and move configration under render_templatesawanoboly2015-10-241-6/+33
| | | | |
View Lorry Controller Status