Skip to content

Commit

Permalink
Merge pull request #306 from mulkieran/print-stderrdata
Browse files Browse the repository at this point in the history
Print out stdout and stderr from checkmetadata result
  • Loading branch information
mulkieran authored Jan 23, 2025
2 parents f27085e + a40cd87 commit 207298f
Showing 1 changed file with 11 additions and 3 deletions.
14 changes: 11 additions & 3 deletions testlib/infra.py
Original file line number Diff line number Diff line change
Expand Up @@ -252,7 +252,7 @@ def _check_encryption_information_consistency(self, pool_object_path, metadata):
elif features is not None:
self.assertNotIn("Encryption", metadata["features"])

def run_check(self, stop_time):
def run_check(self, stop_time): # pylint: disable=too-many-locals
"""
Run the check.
Expand Down Expand Up @@ -297,8 +297,16 @@ def run_check(self, stop_time):
],
stdout=subprocess.PIPE,
) as proc:
(stdoutdata, _) = proc.communicate()
self.assertEqual(proc.returncode, 0, stdoutdata)
(stdoutdata, stderrdata) = proc.communicate()
self.assertEqual(
proc.returncode,
0,
(
f'stdout: {stdoutdata.decode("utf-8")}'
"; "
f'stderr: {stderrdata.decode("utf-8")}'
),
)
except FileNotFoundError as err:
raise RuntimeError(f"{stratisd_tools} not found") from err

Expand Down

0 comments on commit 207298f

Please sign in to comment.