summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorChris Down <chris@chrisdown.name>2020-02-17 10:12:02 -0500
committerDaniel Shahaf <danielsh@apache.org>2020-02-18 12:59:52 +0000
commit59258252b501159c2988de3a2d87a18dc17e98b1 (patch)
tree92b81715909e302c0f9ab508f9484edd0e5f3a9e
parentcb5d8bfe8621c8029f97647ec543ce6e7c41fa58 (diff)
downloadzsh-59258252b501159c2988de3a2d87a18dc17e98b1.tar.gz
zsh-59258252b501159c2988de3a2d87a18dc17e98b1.zip
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.
-rw-r--r--ChangeLog3
-rw-r--r--Test/B11kill.ztst9
2 files changed, 12 insertions, 0 deletions
diff --git a/ChangeLog b/ChangeLog
index 4d48d787c..162bde6b0 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,5 +1,8 @@
2020-02-18 Chris Down <chris@chrisdown.name>
+ * 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 <sandals@crustytoothpaste.net>
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: