Pipes output (#10)

* Embedded pipe command returns output as byte string rather than utf-8

* Bug fixes.

Author: harro

README.md

@@ -25,7 +25,7 @@ that do not require a ctrl-c to terminate.
 ## Documentation
 
 ### Setup
-TCLI requires some setup bit can be run straight out of the box using some
+TCLI requires some setup but can be run straight out of the box using some
 fictictous devices and a limited set of commands that produce canned output:
 
 Devices

setup.py

@@ -21,7 +21,7 @@
 from os import path
 from setuptools import setup
 
-__version__ = '0.1.6'
+__version__ = '0.1.7'
 here = path.abspath(path.dirname(__file__))
 
 # Get the long description from the README file

tcli/tcli_lib.py

@@ -1188,7 +1188,7 @@ def _Pipe(self, output, pipe=''):
       result, _ = p.communicate(bytes(output, 'utf-8'))
       logging.debug('Output written to pipe:\n%s', output)
       logging.debug('Output read from pipe:\n%s', result)
-      return result
+      return result.decode('utf-8')
 
     except IOError:
       logging.error('IOerror writing/reading from pipe.')

tcli/tcli_test.py

@@ -1293,15 +1293,15 @@ def testPipe(self):
 
     # Piped function greps for 'boo' in supplied argument.
     self.tcli_obj.pipe = '/bin/grep boo'
-    self.assertEqual(b'boo\nboo\n', self.tcli_obj._Pipe('boo\nboo'))
+    self.assertEqual('boo\nboo\n', self.tcli_obj._Pipe('boo\nboo'))
     self.tcli_obj.pipe = '/bin/grep boo'
-    self.assertEqual(b'boo\n', self.tcli_obj._Pipe('boo\nhoo'))
+    self.assertEqual('boo\n', self.tcli_obj._Pipe('boo\nhoo'))
 
     # Tests that pipes are supported within the piping function.
     self.tcli_obj.pipe = '/bin/grep boo | /bin/grep -v hoo'
-    self.assertEqual(b'boo\n', self.tcli_obj._Pipe('boo\nhoo'))
+    self.assertEqual('boo\n', self.tcli_obj._Pipe('boo\nhoo'))
     self.tcli_obj.pipe = '/bin/grep boo | /bin/grep -v boo'
-    self.assertEqual(b'', self.tcli_obj._Pipe('boo\nhoo'))
+    self.assertEqual('', self.tcli_obj._Pipe('boo\nhoo'))
 
   def testTildeExpandTargets(self):
     """Tests target expansion."""