summaryrefslogtreecommitdiff
path: root/spec
Commit message (Collapse)AuthorAgeFilesLines
* remove hashrocket syntaxlcg/remove-hashrocketsLamont Granquist2018-06-252-37/+37
| | | | Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
* add sensitive featuredheerajd-msys2018-06-151-6/+6
| | | | Signed-off-by: dheerajd-msys <dheeraj.dubey@msystechnologies.com>
* [MSYS-629] Added username,password validation for elavated option (#151)Nimisha Sharad2017-08-091-3/+13
| | | | | | * Added username,password validation for elavated option Signed-off-by: nimisha <nimisha.sharad@clogeny.com>
* Update error messages to indicate user who lacks privilege on WindowsBryan McLellan2017-07-191-1/+1
| | | | | | | In some cases the user running mixlib-shellout needs additional privileges, in other cases the user we're trying to run the process as needs them. Signed-off-by: Bryan McLellan <btm@loftninjas.org>
* Launch process with elevated tokenNimishaS2017-07-191-0/+8
| | | | Signed-off-by: NimishaS <nimisha.sharad@msystechnologies.com>
* spec cleanup and some light refactoringlcg/cleanup2Lamont Granquist2017-02-011-176/+131
| | | | | | | | | | | | | | | - mostly cleans up the windows specs so its easier to read the API out of them and removes some of the very brittle internal testing - refactors the 'which' logic a bit. trying to converge towards the chef/chef version and eventually extracting common code so that do not have to maintain 10+ slightly different copies everywhere. - adds the Mixlib::ShellOut::EmptyWindowsCommand exception because letting CreateProcessW throw a generic SystemCallError is pretty much useless to everyone. Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
* remove largely useless Utils classLamont Granquist2017-01-272-12/+11
| | | | | | | | its just a collection of private methods, they should be private methods retain a method in the class that was being used externally. Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
* Resolve most chefstyle warningsTim Smith2016-10-073-380/+381
| | | | Signed-off-by: Tim Smith <tsmith@chef.io>
* Require Ruby 2.1+deprecateTim Smith2016-08-222-14/+1
| | | | | | Just removes some 1.9 logic from testing. Signed-off-by: Tim Smith <tsmith@chef.io>
* add logging message if attempting to kill critical processsmurawski/agressive_terminationSteven Murawski2016-06-061-5/+7
|
* Keep `kill_process_tree` from killing protected system processes.Steven Murawski2016-05-241-3/+43
|
* convert environment hash keys to stringsEvan Tschuy2015-11-101-0/+8
| | | | | | | by converting the hash keys to strings, we can use symbols in our environment blocks: environment {FOO: 'bar', SHELL: 'zsh'}
* improve the test to more accurately detrermine if child process is killedMatt Wrock2015-10-201-8/+5
|
* kill all windows child processes when the command times outMatt Wrock2015-09-241-0/+13
|
* make tests pass when run by a domain joined userMatt Wrock2015-09-241-2/+2
|
* prevent shellout from attempting to execute a directory on windowsMatt Wrock2015-09-111-0/+2
|
* have failing testMatt Wrock2015-09-111-0/+29
|
* Added comment for bugjdm/testJay Mundrawala2015-08-261-0/+10
|
* Don't use presence of fd to check if fd's were inheritedJay Mundrawala2015-08-261-2/+4
| | | | | | | | | This is breaking on windows with ruby 2.0.0p645. The issue seems to be that there is something at the fd, but it's not the file descriptor from the parent. Instead, we're now testing that both the fd exists and the file contains the contents that were written
* force expect syntaxLamont Granquist2015-05-151-0/+3
|
* rspec3ificationLamont Granquist2015-05-153-283/+350
|
* Merge pull request #94 from chef/lcg/no-getpgrpLamont Granquist2015-05-151-6/+6
|\ | | | | remove getpgrp back to setsid
| * remove getpgrp back to setsidlcg/no-getpgrpLamont Granquist2015-05-151-6/+6
| |
* | Fix 2 ruby warnings for uninitialized variablesdanielsdeleo2015-05-151-0/+3
|/
* Merge pull request #78 from BackSlasher/add-loginLamont Granquist2015-05-141-1/+83
|\ | | | | with_logon now simulates login in unix
| * Added TODO for next majorNitz2015-02-151-0/+3
| |
| * Option "login" now simulates login in unixNitz2015-02-151-1/+80
| | | | | | | | | | | | | | | | | | Added: * Secondary groups from etc files * Set environment variables like `su` would * Set primary group to user's primary group if no other group specified All only if `login` is set
* | Change mixlib spec test to pass on windows by invoking powershell explicitly.ksubrama/test_fixKartik Null Cating-Subramanian2015-04-211-1/+1
| |
* | Add some test coverage for timeouts on windowsJay Mundrawala2014-12-181-99/+115
| |
* | The process no longer inherit LC_ALL from the parent. Update spec to ↵Jay Mundrawala2014-12-171-8/+3
|/ | | | | | | | | refelect that. This fixes a spec that was broken on Windows by 767ea4fc19cb29a55a81a55d1a13643ae3108f4c from PR https://github.com/opscode/mixlib-shellout/pull/58 It seems the Unix spec was updated to reflect this change, but the Windows was not
* Merge pull request #57 from ClogenyTechnologies/kd/processgrp-fixlamont-granquist2014-09-111-11/+14
|\ | | | | aix getpgid fails when called from parent and parent and child have different session
| * removed emulated Errno::ESRCHkaustubh-d2014-09-101-10/+11
| |
| * raise Errno::ESRCH in spec for aixkaustubh-d2014-08-271-1/+1
| |
| * fix getpgid issue on aixkaustubh-d2014-08-271-2/+4
| |
* | control ENV['LC_ALL'] global betterLamont Granquist2014-09-031-10/+9
| | | | | | | | fixes travis failure that did not fail on my desktop
* | remove LC_ALL default environment variableLamont Granquist2014-08-281-13/+8
| | | | | | | | now that we no longer support ruby 1.8.7 its time to retire this
* | aix does not allow resetting real uid once the euid is changed to non-root.kaustubh-d2014-08-273-0/+27
| |
* | remove #clean_parent_file_descriptors which is not required now.kaustubh-d2014-08-251-14/+0
|/
* Simplify live_stream vs live_stdout vs live_stderr based on PR commentsMax Lincoln2014-07-171-22/+34
|
* Support separate live stream for stderrMax Lincoln2014-07-171-4/+78
|
* Correct spelling in spec filesTyler Cipriani2014-05-161-5/+5
|
* Merge pull request #26 from akshaykarle/masterSerdar Sutay2014-05-131-2/+7
|\ | | | | [#MIXLIB-19] Copy stderr to live_stream along with stdout of the subprocess
| * Fix a broken specAkshay Karle2014-01-061-4/+3
| |
| * Add a spec for showing stderr to live_stream.Akshay Karle2014-01-061-0/+6
| |
* | marking some unix_only tests unix_onlyLamont Granquist2014-04-081-2/+2
| |
* | [MIXLIB-6] Add comment describing why the test was changed, per Dan DeLeo.Julian C. Dunn2014-04-011-0/+2
| |
* | [MIXLIB-6] Use conditional to determine whether to use /tmp or /bin as a cwd ↵Julian C. Dunn2014-04-011-1/+1
| | | | | | | | for the test.
* | [MIXLIB-6] Fix test due to /bin being a symlink on Fedora. Use /tmp insteadJulian C. Dunn2014-04-011-1/+1
| |
* | Exclude ESRCH test from windowsdanielsdeleo2014-03-311-1/+1
| |
* | Handle ESRCH when getting pgid of a zombie on OS Xdanielsdeleo2014-03-311-0/+19
| |
View Lorry Controller Status