Address comments

Author: minggangw

test/test-rosidl-message-generator.js

@@ -63,6 +63,17 @@ function sourceSetupScript(setupPath) {
 function buildTestMessage() {
   // Build the custom_msg_test package synchronously before running the test
   const customMsgTestPath = path.join(__dirname, 'custom_msg_test');
+
+  // List packages to ensure colcon can discover them
+  require('child_process').spawnSync(
+    'colcon',
+    ['--log-base', '/dev/null', 'list'],
+    {
+      cwd: customMsgTestPath,
+      stdio: 'inherit',
+    }
+  );
+
   const buildResult = require('child_process').spawnSync('colcon', ['build'], {
     cwd: customMsgTestPath,
     stdio: 'inherit',
@@ -77,7 +88,7 @@ function buildTestMessage() {
 
   if (buildResult.status !== 0) {
     console.log(buildResult);
-    // throw new Error(`colcon build failed with exit code ${buildResult.status}`);
+    throw new Error(`colcon build failed with exit code ${buildResult.status}`);
   }
 
   // Source the local_setup.sh to get environment variables