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

Merge branch 'wip/t18232' into 'master'

tests/depot: Assert that bugs were fixed

See merge request steam/steamlinuxruntime!6
parents 06467597 8d09ce63
No related branches found
No related tags found
No related merge requests found
......@@ -204,8 +204,7 @@ class TestPressureVessel(unittest.TestCase):
'i386-linux-gnu-capsule-capture-libs',
'x86_64-linux-gnu-capsule-capture-libs',
'pressure-vessel-wrap',
# TODO: Add this when a version with --version has been integrated
# 'steam-runtime-system-info',
'steam-runtime-system-info',
):
with self.catch('--version', exe=exe):
completed = self.run_subprocess(
......@@ -297,15 +296,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 +322,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