remove debug message and remove the test of get_return twice. do not try to get_return twice since by the first call the stdout is all read and output of the second call is not predictable.

Author: Han Wang

dpgen/dispatcher/LazyLocalContext.py

@@ -121,16 +121,13 @@ def check_finish(self, proc):
     def get_return(self, proc):
         ret = proc.poll()
         if ret is None:
-            print('---------------- proc is None ----------------')
             return None, None, None
         else :
             try:
                 o, e = proc.communicate()
-                print('---------------- communicate finishes ----------------')
                 stdout = SPRetObj(o)
                 stderr = SPRetObj(e)
             except ValueError:
-                print('---------------- communicate failed ----------------')
                 stdout = None
                 stderr = None
         return ret, stdout, stderr

tests/dispatcher/test_lazy_local_context.py

@@ -167,8 +167,8 @@ def test_call(self) :
         self.assertEqual(r, 0)
         self.assertEqual(o.read(), b'')
         self.assertEqual(e.read(), b'')
-        r,o,e=self.job.get_return(proc)
-        self.assertEqual(r, 0)
-        self.assertEqual(o, None)
-        self.assertEqual(e, None)
+        # r,o,e=self.job.get_return(proc)
+        # self.assertEqual(r, 0)
+        # self.assertEqual(o, None)
+        # self.assertEqual(e, None)
 

tests/dispatcher/test_local_context.py

@@ -356,8 +356,8 @@ def test_call(self) :
         self.assertEqual(r, 0)
         self.assertEqual(o.read(), b'')
         self.assertEqual(e.read(), b'')
-        r,o,e=self.job.get_return(proc)
-        self.assertEqual(r, 0)
-        self.assertEqual(o, None)
-        self.assertEqual(e, None)
+        # r,o,e=self.job.get_return(proc)
+        # self.assertEqual(r, 0)
+        # self.assertEqual(o, None)
+        # self.assertEqual(e, None)