emacs-27 2065316 1/3: Make comint-tests more robust (bug#38813)

[Mattias Engdegård]

branch: emacs-27
commit 206531674947789841a61bd428dd6babce933a0d
Author: Mattias Engdegård <address@hidden>
Commit: Mattias Engdegård <address@hidden>

    Make comint-tests more robust (bug#38813)
    
    * test/lisp/comint-tests.el (comint-test-no-password-function)
    (comint-test-password-function-with-value)
    (comint-test-password-function-with-nil):
    Call accept-process-output as many times as needed, with a slightly
    more generous timeout (100 ms), after sending the Password: prompt to
    the process, since there must be time for some back-and-forth
    communication.  Also clear the process-query-on-exit flag, since it
    doesn't go well with noninteractive tests.
---
 test/lisp/comint-tests.el | 12 ++++++------
 1 file changed, 6 insertions(+), 6 deletions(-)

diff --git a/test/lisp/comint-tests.el b/test/lisp/comint-tests.el
index c041345..4c1c5cc 100644
--- a/test/lisp/comint-tests.el
+++ b/test/lisp/comint-tests.el
@@ -64,10 +64,10 @@ alter normal password flow."
         (with-temp-buffer
           (make-comint-in-buffer "test-comint-password" (current-buffer) cat)
           (let ((proc (get-buffer-process (current-buffer))))
+            (set-process-query-on-exit-flag proc nil)
             (comint-send-string proc "Password: ")
-            (accept-process-output proc 0 1 t)
             (comint-send-eof)
-            (accept-process-output proc 0 1 t)
+            (while (accept-process-output proc 0.1 nil t))
             (should (string-equal (buffer-substring-no-properties (point-min) 
(point-max))
                                   "Password: PaSsWoRd123\n"))
             (when (process-live-p proc)
@@ -87,10 +87,10 @@ flow.  Hook function returns alternative password."
         (with-temp-buffer
           (make-comint-in-buffer "test-comint-password" (current-buffer) cat)
           (let ((proc (get-buffer-process (current-buffer))))
+            (set-process-query-on-exit-flag proc nil)
             (comint-send-string proc "Password: ")
-            (accept-process-output proc 0 1 t)
             (comint-send-eof)
-            (accept-process-output proc 0 1 t)
+            (while (accept-process-output proc 0.1 nil t))
             (should (string-equal (buffer-substring-no-properties (point-min) 
(point-max))
                                   "Password: MaGiC-PaSsWoRd789\n"))
             (when (process-live-p proc)
@@ -110,10 +110,10 @@ password flow if it returns a nil value."
         (with-temp-buffer
           (make-comint-in-buffer "test-comint-password" (current-buffer) cat)
           (let ((proc (get-buffer-process (current-buffer))))
+            (set-process-query-on-exit-flag proc nil)
             (comint-send-string proc "Password: ")
-            (accept-process-output proc 0 1 t)
             (comint-send-eof)
-            (accept-process-output proc 0 1 t)
+            (while (accept-process-output proc 0.1 nil t))
             (should (string-equal (buffer-substring-no-properties (point-min) 
(point-max))
                                   "Password: PaSsWoRd456\n"))
             (when (process-live-p proc)