From acb6eac23834d268b4ed964a9d4af4bd75d6838b Mon Sep 17 00:00:00 2001
From: John-Mark Gurney <jmg@funkthat.com>
Date: Thu, 31 Oct 2019 14:18:26 -0700
Subject: [PATCH] clean up the tests better, shame there isn't a context
 handler for processes...

---
 ntunnel.py | 11 +++++++++++
 1 file changed, 11 insertions(+)

diff --git a/ntunnel.py b/ntunnel.py
index 6ef35e4..28a5e67 100644
--- a/ntunnel.py
+++ b/ntunnel.py
@@ -779,6 +779,13 @@ class TestMain(unittest.TestCase):
 		lsock.close()
 		await lsock.wait_closed()
 
+		# Kill off the client
+		client.terminate()
+
+		stdout, stderr = await client.communicate()
+		#print('s:', repr((stdout, stderr)))
+		# XXX - figure out how to clean up client properly
+
 	@async_test
 	async def test_end2end(self):
 		# Generate necessar keys
@@ -862,6 +869,10 @@ class TestMain(unittest.TestCase):
 		lsock.close()
 		await lsock.wait_closed()
 
+		server.terminate()
+		client.terminate()
+		# XXX - more clean up testing
+
 	@async_test
 	async def test_genkey(self):
 		# that it can generate a key