konevsa/fish
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity
fish/tests/checks/status-value.fish

48 lines
1.1 KiB
Fish
Raw Permalink Normal View History

builtins: Don't crash for negative return values Another from the "why are we asserting instead of doing something sensible" department. The alternative is to make exit() and return() compute their own exit code, but tbh I don't want any *other* builtin to hit this either? Fixes #9659 (cherry picked from commit a16abf22d9292f232ec7d57b3cf42e2e93ffbb0a)
2023-03-14 10:50:22 +01:00
# RUN: %fish -C 'set -g fish %fish' %s
Rationalize $status and errors Prior to this fix, fish was rather inconsistent in when $status gets set in response to an error. For example, a failed expansion like "$foo[" would not modify $status. This makes the following inter-related changes: 1. String expansion now directly returns the value to set for $status on error. The value is always used. 2. parser_t::eval() now directly returns the proc_status_t, which cleans up a lot of call sites. 3. We expose a new function exec_subshell_for_expand() which ignores $status but returns errors specifically related to subshell expansion. 4. We reify the notion of "expansion breaking" errors. These include command-not-found, expand syntax errors, and others. The upshot is we are more consistent about always setting $status on errors.
2020-01-23 17:34:46 -08:00
# Empty commands should be 123
set empty_var
$empty_var
echo $status
# CHECK: 123
# CHECKERR: {{.*}} The expanded command was empty.
# CHECKERR: $empty_var
Add more tests These were correct, but littlecheck escapes quotes!
2022-08-11 18:20:40 +02:00
# CHECKERR: ^~~~~~~~~^
Rationalize $status and errors Prior to this fix, fish was rather inconsistent in when $status gets set in response to an error. For example, a failed expansion like "$foo[" would not modify $status. This makes the following inter-related changes: 1. String expansion now directly returns the value to set for $status on error. The value is always used. 2. parser_t::eval() now directly returns the proc_status_t, which cleans up a lot of call sites. 3. We expose a new function exec_subshell_for_expand() which ignores $status but returns errors specifically related to subshell expansion. 4. We reify the notion of "expansion breaking" errors. These include command-not-found, expand syntax errors, and others. The upshot is we are more consistent about always setting $status on errors.
2020-01-23 17:34:46 -08:00
# Failed expansions
echo "$abc["
echo $status
# CHECK: 121
# CHECKERR: {{.*}} Invalid index value
# CHECKERR: echo "$abc["
Clamp error carets to the end instead of refusing to print This skipped printing a "^" line if the start or length of the error was longer than the source. That seems like the correc thing at first glance, however it means that the caret line isn't skipped *if the file goes on*. So, for example ```fish echo "$abc[" ``` by itself, in a file or via `fish -c`, would not print an error, but ```fish echo "$abc[" true ``` would. That's not a great way to print errors. So instead we just.. imagine the start was at most at the end. The underlying issue why `echo "$abc["` causes this is that `wcstol` didn't move the end pointer for the index value (because there is no number there). I'd fix this, but apparently some of our recursive variable calls absolutely rely on this position value.
2022-08-11 19:24:41 +02:00
# CHECKERR: ^
Rationalize $status and errors Prior to this fix, fish was rather inconsistent in when $status gets set in response to an error. For example, a failed expansion like "$foo[" would not modify $status. This makes the following inter-related changes: 1. String expansion now directly returns the value to set for $status on error. The value is always used. 2. parser_t::eval() now directly returns the proc_status_t, which cleans up a lot of call sites. 3. We expose a new function exec_subshell_for_expand() which ignores $status but returns errors specifically related to subshell expansion. 4. We reify the notion of "expansion breaking" errors. These include command-not-found, expand syntax errors, and others. The upshot is we are more consistent about always setting $status on errors.
2020-01-23 17:34:46 -08:00
# Failed wildcards
echo *gibberishgibberishgibberish*
echo $status
# CHECK: 124
# CHECKERR: {{.*}} No matches for wildcard '*gibberishgibberishgibberish*'. {{.*}}
# CHECKERR: echo *gibberishgibberishgibberish*
Add acceptable errors to tests
2022-08-11 17:22:03 +02:00
# CHECKERR: ^~~~~~~~~~~~~~~~~~~~~~~~~~~~^
builtins: Don't crash for negative return values Another from the "why are we asserting instead of doing something sensible" department. The alternative is to make exit() and return() compute their own exit code, but tbh I don't want any *other* builtin to hit this either? Fixes #9659 (cherry picked from commit a16abf22d9292f232ec7d57b3cf42e2e93ffbb0a)
2023-03-14 10:50:22 +01:00
$fish -c 'exit -5'
Log original exit code used when a builtin returns a negative exit code
2024-01-03 16:17:49 +01:00
# CHECKERR: warning: builtin exit returned invalid exit code -5
builtins: Don't crash for negative return values Another from the "why are we asserting instead of doing something sensible" department. The alternative is to make exit() and return() compute their own exit code, but tbh I don't want any *other* builtin to hit this either? Fixes #9659 (cherry picked from commit a16abf22d9292f232ec7d57b3cf42e2e93ffbb0a)
2023-03-14 10:50:22 +01:00
echo $status
# CHECK: 251
$fish -c 'exit -1'
Log original exit code used when a builtin returns a negative exit code
2024-01-03 16:17:49 +01:00
# CHECKERR: warning: builtin exit returned invalid exit code -1
builtins: Don't crash for negative return values Another from the "why are we asserting instead of doing something sensible" department. The alternative is to make exit() and return() compute their own exit code, but tbh I don't want any *other* builtin to hit this either? Fixes #9659 (cherry picked from commit a16abf22d9292f232ec7d57b3cf42e2e93ffbb0a)
2023-03-14 10:50:22 +01:00
echo $status
# CHECK: 255
# (we avoid 0, so this is turned into 255 again)
$fish -c 'exit -256'
Log original exit code used when a builtin returns a negative exit code
2024-01-03 16:17:49 +01:00
# CHECKERR: warning: builtin exit returned invalid exit code -256
builtins: Don't crash for negative return values Another from the "why are we asserting instead of doing something sensible" department. The alternative is to make exit() and return() compute their own exit code, but tbh I don't want any *other* builtin to hit this either? Fixes #9659 (cherry picked from commit a16abf22d9292f232ec7d57b3cf42e2e93ffbb0a)
2023-03-14 10:50:22 +01:00
echo $status
# CHECK: 255
$fish -c 'exit -512'
Log original exit code used when a builtin returns a negative exit code
2024-01-03 16:17:49 +01:00
# CHECKERR: warning: builtin exit returned invalid exit code -512
builtins: Don't crash for negative return values Another from the "why are we asserting instead of doing something sensible" department. The alternative is to make exit() and return() compute their own exit code, but tbh I don't want any *other* builtin to hit this either? Fixes #9659 (cherry picked from commit a16abf22d9292f232ec7d57b3cf42e2e93ffbb0a)
2023-03-14 10:50:22 +01:00
echo $status
# CHECK: 255
Reference in New Issue Copy Permalink