From 59258252b501159c2988de3a2d87a18dc17e98b1 Mon Sep 17 00:00:00 2001 From: Chris Down Date: Mon, 17 Feb 2020 10:12:02 -0500 Subject: 45452: builtins: kill: Add `kill ''` regression test with explicit sigspec The version without a sigspec can't be added yet because it would still kill the test runner even in expected-to-fail mode; see workers/45449 for discussion. For the same reason, we use a signal which is non-fatal by default and unlikely to be sent by someone else, SIGURG, to do the expected-to-fail case prior to the fix. --- ChangeLog | 3 +++ Test/B11kill.ztst | 9 +++++++++ 2 files changed, 12 insertions(+) diff --git a/ChangeLog b/ChangeLog index 4d48d787c..162bde6b0 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,5 +1,8 @@ 2020-02-18 Chris Down + * 45452: Test/B11kill.ztst: builtins: kill: Add `kill ''` + regression test with explicit sigspec + * 45451: Test/B11kill.ztst: builtins: kill: Add basic test suite 2020-02-17 brian m. carlson diff --git a/Test/B11kill.ztst b/Test/B11kill.ztst index 26d7a11fe..957a7b385 100644 --- a/Test/B11kill.ztst +++ b/Test/B11kill.ztst @@ -58,3 +58,12 @@ ) 1:kill with sigspec only ?(eval):kill:2: not enough arguments + +# Regression tests: `kill ''` should not result in `kill 0`. + + ( + trap 'exit 11' URG + kill -URG '' + ) +1f:kill with empty pid and sigspec should not send signal to current process group +?(eval):kill:3: illegal pid: -- cgit 1.4.1