github-mirror/fish-shell
2
0
Fork 0
mirror of https://github.com/fish-shell/fish-shell.git synced 2025-01-19 04:42:45 +08:00
Code Issues Actions Packages Projects Releases Wiki Activity

Add a test for background procs in cmdsubs

Browse Source 
This adds a test to ensure that if a long running background process is
launched from a command substitution, that process does not cause the
cmdsub to hang. That could easily happen if we just wait for the pipe to
close; this is verifying that we are also checking for the job to complete.
This commit is contained in:
ridiculousfish 2021-01-03 16:42:30 -08:00
parent 0729c2be4c
commit 534bc66a43
1 changed files with 20 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

23
tests/checks/cmdsub-limit.fish
View File

@ -1,8 +1,25 @@
# RUN: %fish %s
# This tests various corner cases involving command substitution. Most
# importantly the limits on the amount of data we'll substitute.
# This tests various corner cases involving command substitution.
# Test cmdsubs which spawn background processes - see #7559.
# If this hangs, it means that fish keeps trying to read from the write
# end of the cmdsub pipe (which has escaped).
# FIXME: we need to mark full job control for sleep to get its own pgid;
# otherwise $last_pid will return fish's pgid! It's always been so!
status job-control full
echo (command sleep 1000000 & ; set -g sleep_pid $last_pid ; echo local)
# CHECK: local
echo $sleep_pid
# CHECK: {{[1-9]\d*}}
kill $sleep_pid ; echo $status
# CHECK: 0
status job-control interactive
# Test limiting the amount of data we'll substitute.
set fish_read_limit 512
function subme
@ -10,7 +27,7 @@ function subme
echo $x
end
# Command sub just under the limit should succeed
# Command sub just under the limit should succeed.
set a (subme 511)
set --show a
#CHECK: $a: set in global scope, unexported, with 1 elements