From 8045d328ada426da124cfd9cac5ed98d697e481b Mon Sep 17 00:00:00 2001
From: jette <jette@schedmd.com>
Date: Mon, 5 Aug 2013 10:42:21 -0700
Subject: [PATCH] Fix test for poe timing bug

---
 testsuite/expect/test1.33 | 11 ++++++++++-
 1 file changed, 10 insertions(+), 1 deletion(-)

diff --git a/testsuite/expect/test1.33 b/testsuite/expect/test1.33
index 107b94d7ee9..2389c64b933 100755
--- a/testsuite/expect/test1.33
+++ b/testsuite/expect/test1.33
@@ -64,7 +64,14 @@ if {[test_bluegene]} {
 #
 # Delete left-over scripts and build new ones
 #
+# POE BUG: If the application exits immediately then pmdv12 hangs until
+# slurm kills it with a timeout, thus the sleep below is required:
+# F S   UID   PID  PPID  C PRI  NI ADDR SZ WCHAN TTY   TIME CMD
+# 4 R  1001 17141 17137 99  80   0 - 33410 ?    ?  00:00:02 pmdv12
+# 0 Z  1001 17152 17141  0  80   0 -     0 exit ?  00:00:00 test1.33.exit.b <defunct>
+#
 make_bash_script $exit_script "
+  $bin_sleep 1
   RC=`expr \$SLURM_PROCID + 10`
   exit \$RC
 "
@@ -97,7 +104,9 @@ expect {
 			# since with poe we can only grab the poe's return code
 			# we check here to make sure it is the correct number
 			catch wait result
-			incr sum [lindex $result 3]
+			set rc [lindex $result 3]
+			incr sum $rc
+			send_user "\npoe return code is $rc\n"
 		} else {
 			wait
 		}
-- 
GitLab