Support "/dev/null" as a target when creating Eshell handles

Previously, you could only use this when setting the handle afterwards.

* lisp/eshell/esh-io.el (eshell-set-output-handle): Don't catch
'eshell-null-device' here...
(eshell-get-target): ... catch it here.

* test/lisp/eshell/eshell-tests.el
(eshell-test/eshell-command/output-dev-null): New test (bug#72857).

1 files changed, 8 insertions, 0 deletions

diff --git a/test/lisp/eshell/eshell-tests.el b/test/lisp/eshell/eshell-tests.el
index e5aeee5123e..50a748cd56e 100644
--- a/test/lisp/eshell/eshell-tests.el
+++ b/test/lisp/eshell/eshell-tests.el
@@ -150,6 +150,14 @@ This test uses a pipeline for the command."
   (eshell-command "echo hi" 'eshell-command-output)
   (should (equal eshell-command-output "hi")))
 
+(ert-deftest eshell-test/eshell-command/output-dev-null ()
+  "Test that the `eshell-command' function handles /dev/null properly."
+  (ert-with-temp-directory eshell-directory-name
+    (let ((eshell-history-file-name nil))
+      (with-temp-buffer
+        (eshell-command "echo hi" "/dev/null")
+        (should (equal (buffer-string) ""))))))
+
 (ert-deftest eshell-test/command-running-p ()
   "Modeline should show no command running"
   (with-temp-eshell