summaryrefslogtreecommitdiff
path: root/spec/unit
Commit message (Collapse)AuthorAgeFilesLines
* Merge pull request #7249 from chef/lcg/cleanup-userLamont Granquist2018-05-211-2/+2
|\ | | | | Cleanup AIX and Solaris user resources.
| * clean up solaris providerLamont Granquist2018-05-101-2/+2
| | | | | | | | Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
* | Merge pull request #7274 from mal/fix-systemd-user-callsThom May2018-05-212-41/+86
|\ \ | | | | | | Fix systemd_unit user context
| * | Contextualise daemon-reload callsMal Graty2018-05-191-39/+84
| | | | | | | | | | | | | | | | | | Ensure daemon-reload is called in the correct context for user units. Signed-off-by: Mal Graty <mal.graty@googlemail.com>
| * | Fix systemd_unit user property usageMal Graty2018-05-192-2/+2
| | | | | | | | | | | | | | | | | | | | | Use Etc.getpwnam rather than Etc.getpwuid, since getpwuid expects a UID and the resource provides a string. Signed-off-by: Mal Graty <mal.graty@googlemail.com>
* | | Merge pull request #7270 from chef/tm/fix_manifest_root_filesThom May2018-05-182-1/+3
|\ \ \ | | | | | | | | Fix manifest entries for root files
| * | | Fix manifest entries for root filestm/fix_manifest_root_filesThom May2018-05-162-1/+3
| |/ / | | | | | | | | | | | | | | | | | | | | | We weren't consistently naming root files; in some places in the spec the names contained the `root_files` segment and in other places they didn't. This now always uses `root_files`. Signed-off-by: Thom May <thom@chef.io>
* | | Merge pull request #7196 from chef/tg/clear_gem_sources_optionThom May2018-05-181-0/+13
|\ \ \ | | | | | | | | Tg/clear gem sources option
| * | | Add test for Chef::Config[:clear_gem_sources]Thomas Gordon2018-04-271-0/+13
| | | | | | | | | | | | | | | | Signed-off-by: Thomas Gordon <nrgetik@users.noreply.github.com>
* | | | Merge pull request #7264 from jeremymv2/jeremymv2/fix_normalize_hashJosh Hudson2018-05-171-0/+65
|\ \ \ \ | | | | | | | | | | object validation for DataHandlerBase#normalize_hash
| * | | | prefer double quotesJeremy J. Miller2018-05-161-1/+1
| | | | | | | | | | | | | | | | | | | | Signed-off-by: Jeremy J. Miller <jm@chef.io>
| * | | | added one more rspec testJeremy J. Miller2018-05-161-4/+11
| | | | | | | | | | | | | | | | | | | | Signed-off-by: Jeremy J. Miller <jm@chef.io>
| * | | | object validation for DataHandlerBase#normalize_hashJeremy J. Miller2018-05-151-0/+58
| | | | | | | | | | | | | | | | | | | | Signed-off-by: Jeremy J. Miller <jm@chef.io>
* | | | | Merge pull request #4903 from nmcspadden/mac_uidThom May2018-05-171-9/+2
|\ \ \ \ \ | | | | | | | | | | | | UID now starts at 501, uses createhomedir instead
| * | | | | Fixing shell_out to have correct syntaxNick McSpadden2018-05-151-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Nick McSpadden <nmcspadden@gmail.com>
| * | | | | Updating spec testNick McSpadden2018-05-151-9/+2
| |/ / / /
* | | | | Add additional resource specsTim Smith2018-05-1697-417/+1169
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Auto generated from the resource inspector Signed-off-by: Tim Smith <tsmith@chef.io>
* | | | | Add additional unit tests for resource actions/propertiesTim Smith2018-05-1513-20/+176
| |_|/ / |/| | | | | | | | | | | | | | | | | | | Fill in some missing coverage here so we can refactor with confidence. Signed-off-by: Tim Smith <tsmith@chef.io>
* | | | [MSYS-809] Fix for command resource does not handle commands with argumentsvasu11052018-05-151-0/+9
|/ / / | | | | | | | | | Signed-off-by: vasu1105 <vasundhara.jagdale@msystechnologies.com>
* | | remote_directory: restore overwrite defaultRoland Moriz2018-05-121-0/+4
| | | | | | | | | | | | Signed-off-by: Roland Moriz <roland@moriz.de>
* | | sadly --prune-tags is not available with old git versions, fixes #7233Roland Moriz2018-05-101-4/+4
| | | | | | | | | | | | Signed-off-by: Roland Moriz <roland@moriz.de>
* | | Switch Node#role? to use the attributes expansion instead of the run list.Noah Kantrowitz2018-05-081-3/+3
| |/ |/| | | | | | | This means it understands nested roles, avoiding a common footgun. Signed-off-by: Noah Kantrowitz <noah@coderanger.net>
* | Fix two new Chefstyle warnings with Rubocop 0.55rubocop_055Tim Smith2018-05-031-1/+1
| | | | | | | | | | | | These should have been caught before. Signed-off-by: Tim Smith <tsmith@chef.io>
* | Merge pull request #7214 from chef/tm/fix_unix_testsThom May2018-05-031-1/+2
|\ \ | | | | | | Ensure we don't run tests on unsupported platforms
| * | Ensure we don't run tests on unsupported platformstm/fix_unix_testsThom May2018-05-021-1/+2
| | | | | | | | | | | | | | | | | | Our more esoteric unices need some more careful hand holding Signed-off-by: Thom May <thom@chef.io>
* | | Merge pull request #6815 from ↵Bryan McLellan2018-05-022-614/+335
|\ \ \ | | | | | | | | | | | | | | | | MsysTechnologiesllc/vasundhara/MSYS_752_windows_task_rewrite [MSYS-752] windows task rewrite using win32-taskscheduler
| * | | [MSYS-752] windows task rewrite using wind32-taskschedulerVasu11052018-04-242-614/+335
| | | | | | | | | | | | | | | | Signed-off-by: Vasu1105 <vasundhara.jagdale@msystechnologies.com>
* | | | Merge pull request #7194 from coderanger/silent-failureThom May2018-05-022-5/+44
|\ \ \ \ | | | | | | | | | | Allow specifying `ignore_failure :quiet` to disable the error spew
| * | | | Feed the Rubocop.Noah Kantrowitz2018-04-262-8/+8
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Some day I will remember which is which on string styles. Signed-off-by: Noah Kantrowitz <noah@coderanger.net>
| * | | | Allow specifying `ignore_failure :quiet` to disable the error spew.Noah Kantrowitz2018-04-262-5/+44
| | | | | | | | | | | | | | | | | | | | Signed-off-by: Noah Kantrowitz <noah@coderanger.net>
* | | | | Merge pull request #7204 from chef/remote_directoryTim Smith2018-04-301-3/+4
|\ \ \ \ \ | |_|_|/ / |/| | | | Convert some of remote_directory to use properties
| * | | | Fix source property to use the name_property not nameTim Smith2018-04-301-1/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Source was building off name not path, which means the name _property never actually worked. I modified an existing test to check this. Signed-off-by: Tim Smith <tsmith@chef.io>
| * | | | Convert some of remote_directory to use propertiesremote_directoryTim Smith2018-04-271-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This isn't all of it, but it's a start Signed-off-by: Tim Smith <tsmith@chef.io>
* | | | | fix application tests on linuxThom May2018-04-302-0/+2
| | | | | | | | | | | | | | | | | | | | Signed-off-by: Thom May <thom@chef.io>
* | | | | Merge pull request #7199 from chef/registry_keyThom May2018-04-301-1/+1
|\ \ \ \ \ | | | | | | | | | | | | Convert :key property in registry_key to use property
| * | | | | Convert :key property in registry_key to use propertyregistry_keyTim Smith2018-04-271-1/+1
| |/ / / / | | | | | | | | | | | | | | | | | | | | | | | | | Convert this to a true property. It required updating the specs a tiny bit since key gets returned as a state value now (as it should). Signed-off-by: Tim Smith <tsmith@chef.io>
* | | | | Allow build_essential resource to be used without setting a namebuild_nameTim Smith2018-04-271-0/+8
|/ / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This is the same as apt_update. It seems odd to use this resource with a name like this: ```ruby build_essential "build-essential" ``` Signed-off-by: Tim Smith <tsmith@chef.io>
* | | | Merge pull request #7129 from rmoriz/rmoriz/git-pruneTim Smith2018-04-271-8/+8
|\ \ \ \ | |/ / / |/| | | prune remotes with git fetch, fixes #3929
| * | | prune remotes with git fetch, fixes #3929Roland Moriz2018-04-061-8/+8
| | | | | | | | | | | | | | | | Signed-off-by: Roland Moriz <roland@moriz.de>
* | | | Merge pull request #7138 from ↵Tim Smith2018-04-262-2/+67
|\ \ \ \ | |_|/ / |/| | | | | | | | | | | RoboticCheese/roboticcheese/package-locking-nomethoderror Fix NoMethodError when (un)locking single packages in apt and zypper
| * | | Fix NoMethodError when (un)locking single packages in apt and zypperJonathan Hartman2018-04-092-2/+67
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | These resources don't coerce `package_name` to an array so would raise an exception when trying to call `name.all?` when `package_name`, and thus `name`, was a string. Signed-off-by: Jonathan Hartman <j@hartman.io>
* | | | Merge pull request #7170 from chef/tm/ub_1404_serviceThom May2018-04-231-161/+41
|\ \ \ \ | | | | | | | | | | [Debian/Ubuntu] Detect init script service levels
| * | | | [Debian/Ubuntu] Detect init script service levelstm/ub_1404_serviceThom May2018-04-231-161/+41
| | |_|/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Debian removed the no-op flag from update-rc.d, and so we're now unlinking services rather than just querying them. Just parsing the scripts ourselves is much safer. Fixes: #7119 Signed-off-by: Thom May <thom@chef.io>
* | | | Fixing specs for powershell_packagesp/fix7166Stuart Preston2018-04-211-19/+19
|/ / / | | | | | | | | | Signed-off-by: Stuart Preston <stuart@chef.io>
* | | add test that methods wind up in important classesLamont Granquist2018-04-131-1/+10
|/ / | | | | | | Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
* | remove invalid coercing unit testsErik Gomez2018-04-061-14/+0
|/ | | | Signed-off-by: Erik Gomez <e@eriknicolasgomez.com>
* Merge pull request #7027 from gengor/CHEF-7026Lamont Granquist2018-04-021-147/+11
|\ | | | | [CHEF-7026] Rewrite portage package provider candidate_version determination and fix tests
| * Fix style issues reported by travis-ci.Gordon Malm2018-03-211-1/+1
| | | | | | | | Signed-off-by: Gordon Malm <gmalm+github-chef-chef-2018@engineyard.com>
| * Fix portage tests for candidate_version rewrite in previous commit.Gordon Malm2018-03-211-147/+11
| | | | | | | | | | | | | | | | Reference: https://github.com/chef/chef/issues/7026 03fffdcd91b71ee0ff346310c5c9616976ca3f1b Signed-off-by: Gordon Malm <gmalm+github-chef-chef-2018@engineyard.com>
* | Merge pull request #7083 from chef/btm/fix-windows-serviceTim Smith2018-03-291-8/+9
|\ \ | | | | | | Avoid lookups for rights of 'LocalSystem' in windows service
View Lorry Controller Status