Skip to content
Snippets Groups Projects
Commit 81b47321 authored by Simon McVittie's avatar Simon McVittie
Browse files

tests/depot: Assert that pressure-vessel-wrap preserves exit status


This was fixed in steam/pressure-vessel!18.

Signed-off-by: default avatarSimon McVittie <smcv@collabora.com>
parent 06467597
No related branches found
No related tags found
No related merge requests found
......@@ -297,15 +297,12 @@ class TestPressureVessel(unittest.TestCase):
] + adverb
with self.catch('cat /etc/os-release in container'):
# TODO: Ideally this would use check=True,
# but that would need a bubblewrap with
# https://github.com/containers/bubblewrap/issues/336
# fixed, or a pressure-vessel with !18 fixed
completed = self.run_subprocess(
adverb + ['cat', '/etc/os-release'],
cwd=self.tmpdir.name,
stdout=subprocess.PIPE,
stderr=subprocess.PIPE,
check=True,
)
logger.info(
......@@ -326,15 +323,12 @@ class TestPressureVessel(unittest.TestCase):
'w',
) as writer:
with self.catch('run s-r-s-i in container'):
# TODO: Ideally this would use check=True,
# but that would need a bubblewrap with
# https://github.com/containers/bubblewrap/issues/336
# fixed, or a pressure-vessel with !18 fixed
completed = self.run_subprocess(
adverb + ['steam-runtime-system-info', '--verbose'],
cwd=self.tmpdir.name,
stdout=writer,
stderr=subprocess.PIPE,
check=True,
)
if completed.stderr:
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment