Skip to content

Commit f7c5799

Browse files
committed
GA: let's try again
1 parent 8b8068d commit f7c5799

2 files changed

Lines changed: 12 additions & 2 deletions

File tree

.github/workflows/ci.yml

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -121,6 +121,8 @@ jobs:
121121
run: |
122122
apt-get update
123123
DEBIAN_FRONTEND=noninteractive apt-get install -y \
124+
build-essential \
125+
cmake \
124126
curl \
125127
ca-certificates \
126128
git \

open-codegen/test/test_ros2.py

Lines changed: 10 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -100,14 +100,22 @@ def ros2_test_env(cls):
100100
@staticmethod
101101
def _bash(command, cwd, env=None, timeout=180, check=True):
102102
"""Run a bash command and return the completed process."""
103-
return subprocess.run(
103+
result = subprocess.run(
104104
["/bin/bash", "-lc", command],
105105
cwd=cwd,
106106
env=env,
107107
text=True,
108108
capture_output=True,
109109
timeout=timeout,
110-
check=check)
110+
check=False)
111+
if check and result.returncode != 0:
112+
raise AssertionError(
113+
"Command failed with exit code "
114+
f"{result.returncode}: {command}\n"
115+
f"stdout:\n{result.stdout}\n"
116+
f"stderr:\n{result.stderr}"
117+
)
118+
return result
111119

112120
def test_ros2_package_generation(self):
113121
"""Verify the ROS2 package files are generated."""

0 commit comments

Comments
 (0)