about summary refs log tree commit diff
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.tar.xz
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: