Handle None values for .stdout and .stderr on ``ProcessRe…

…sult.__str__()``

Fixes saltstack#8

Signed-off-by: Pedro Algarvio <palgarvio@vmware.com>

changelog/8.bugfix.rst

@@ -0,0 +1 @@
+Handle ``None`` values for ``.stdout`` and ``.stderr`` on ``ProcessResult.__str__()``

src/pytestshellutils/downgraded/utils/processes.py

@@ -146,13 +146,13 @@ def __str__(self) -> str:
             message += '\n Command Line: {0}'.format(self.cmdline)
         if self.returncode is not None:
             message += '\n Returncode: {0}'.format(self.returncode)
-        if self.stdout.strip() or self.stderr.strip():
+        if self.stdout and self.stdout.strip() or self.stderr and self.stderr.strip():
             message += '\n Process Output:'
-        if self.stdout.strip():
+        if self.stdout and self.stdout.strip():
             message += '\n   >>>>> STDOUT >>>>>\n{0}\n   <<<<< STDOUT <<<<<'.format(
                 self.stdout
             )
-        if self.stderr.strip():
+        if self.stderr and self.stderr.strip():
             message += '\n   >>>>> STDERR >>>>>\n{0}\n   <<<<< STDERR <<<<<'.format(
                 self.stderr
             )

src/pytestshellutils/utils/processes.py

@@ -148,11 +148,11 @@ def __str__(self) -> str:
             message += f"\n Command Line: {self.cmdline}"
         if self.returncode is not None:
             message += f"\n Returncode: {self.returncode}"
-        if self.stdout.strip() or self.stderr.strip():
+        if (self.stdout and self.stdout.strip()) or (self.stderr and self.stderr.strip()):
             message += "\n Process Output:"
-        if self.stdout.strip():
+        if self.stdout and self.stdout.strip():
             message += f"\n   >>>>> STDOUT >>>>>\n{self.stdout}\n   <<<<< STDOUT <<<<<"
-        if self.stderr.strip():
+        if self.stderr and self.stderr.strip():
             message += f"\n   >>>>> STDERR >>>>>\n{self.stderr}\n   <<<<< STDERR <<<<<"
         if self.data:
             message += "\n Parsed JSON Data:\n"