From 4ab90db99afafcf3d6ab91deeaf35e5f4874502b Mon Sep 17 00:00:00 2001
From: mahesh
Date: Thu, 13 Apr 2017 09:41:43 +0530
Subject: added test case to check for stray processes

---
 yaksh/evaluator_tests/test_c_cpp_evaluation.py | 10 ++++++++++
 1 file changed, 10 insertions(+)

(limited to 'yaksh/evaluator_tests/test_c_cpp_evaluation.py')

diff --git a/yaksh/evaluator_tests/test_c_cpp_evaluation.py b/yaksh/evaluator_tests/test_c_cpp_evaluation.py
index 304f1cb..d3147f5 100644
--- a/yaksh/evaluator_tests/test_c_cpp_evaluation.py
+++ b/yaksh/evaluator_tests/test_c_cpp_evaluation.py
@@ -4,6 +4,7 @@ import os
 import shutil
 import tempfile
 from textwrap import dedent
+from psutil import Process
 
 # Local import
 from yaksh.grader import Grader
@@ -151,6 +152,11 @@ class CAssertionEvaluationTestCases(EvaluatorBaseTest):
         # Then
         self.assertFalse(result.get("success"))
         self.assert_correct_output(self.timeout_msg, result.get("error"))
+        parent_proc = Process(os.getpid()).children()
+        if parent_proc:
+            self.assertFalse(any(Process(parent_proc[0].pid)\
+                .children(recursive=True)))
+
 
     def test_file_based_assert(self):
         # Given
@@ -401,6 +407,10 @@ class CppStdIOEvaluationTestCases(EvaluatorBaseTest):
         # Then
         self.assertFalse(result.get("success"))
         self.assert_correct_output(self.timeout_msg, result.get("error"))
+        parent_proc = Process(os.getpid()).children()
+        if parent_proc:
+            self.assertFalse(any(Process(parent_proc[0].pid)\
+                .children(recursive=True)))
 
     def test_only_stdout(self):
         # Given
-- 
cgit 


From c431c255738b741698adffcbf894930ae298b8c0 Mon Sep 17 00:00:00 2001
From: mahesh
Date: Thu, 27 Apr 2017 12:13:22 +0530
Subject: added tests to check killing of stray processes in hook evaluator

---
 yaksh/evaluator_tests/test_c_cpp_evaluation.py | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

(limited to 'yaksh/evaluator_tests/test_c_cpp_evaluation.py')

diff --git a/yaksh/evaluator_tests/test_c_cpp_evaluation.py b/yaksh/evaluator_tests/test_c_cpp_evaluation.py
index d3147f5..79227f5 100644
--- a/yaksh/evaluator_tests/test_c_cpp_evaluation.py
+++ b/yaksh/evaluator_tests/test_c_cpp_evaluation.py
@@ -977,7 +977,10 @@ class CppHookEvaluationTestCases(EvaluatorBaseTest):
         # Then
         self.assertFalse(result.get('success'))
         self.assert_correct_output(self.timeout_msg, result.get('error'))
-
+        parent_proc = Process(os.getpid()).children()
+        if parent_proc:
+            self.assertFalse(any(Process(parent_proc[0].pid)\
+                .children(recursive=True)))
 
 if __name__ == '__main__':
     unittest.main()
-- 
cgit 


From 4a8f9a117f6a784fe722b8b381368c41be86024a Mon Sep 17 00:00:00 2001
From: mahesh
Date: Thu, 27 Apr 2017 12:51:10 +0530
Subject: made pep8 changes

---
 yaksh/evaluator_tests/test_c_cpp_evaluation.py | 14 +++++++-------
 1 file changed, 7 insertions(+), 7 deletions(-)

(limited to 'yaksh/evaluator_tests/test_c_cpp_evaluation.py')

diff --git a/yaksh/evaluator_tests/test_c_cpp_evaluation.py b/yaksh/evaluator_tests/test_c_cpp_evaluation.py
index 79227f5..b15f766 100644
--- a/yaksh/evaluator_tests/test_c_cpp_evaluation.py
+++ b/yaksh/evaluator_tests/test_c_cpp_evaluation.py
@@ -154,9 +154,8 @@ class CAssertionEvaluationTestCases(EvaluatorBaseTest):
         self.assert_correct_output(self.timeout_msg, result.get("error"))
         parent_proc = Process(os.getpid()).children()
         if parent_proc:
-            self.assertFalse(any(Process(parent_proc[0].pid)\
-                .children(recursive=True)))
-
+            children_procs = Process(parent_proc[0].pid)
+            self.assertFalse(any(children_procs.children(recursive=True)))
 
     def test_file_based_assert(self):
         # Given
@@ -409,8 +408,8 @@ class CppStdIOEvaluationTestCases(EvaluatorBaseTest):
         self.assert_correct_output(self.timeout_msg, result.get("error"))
         parent_proc = Process(os.getpid()).children()
         if parent_proc:
-            self.assertFalse(any(Process(parent_proc[0].pid)\
-                .children(recursive=True)))
+            children_procs = Process(parent_proc[0].pid)
+            self.assertFalse(any(children_procs.children(recursive=True)))
 
     def test_only_stdout(self):
         # Given
@@ -979,8 +978,9 @@ class CppHookEvaluationTestCases(EvaluatorBaseTest):
         self.assert_correct_output(self.timeout_msg, result.get('error'))
         parent_proc = Process(os.getpid()).children()
         if parent_proc:
-            self.assertFalse(any(Process(parent_proc[0].pid)\
-                .children(recursive=True)))
+            children_procs = Process(parent_proc[0].pid)
+            self.assertFalse(any(children_procs.children(recursive=True)))
+
 
 if __name__ == '__main__':
     unittest.main()
-- 
cgit