Address comments

Author: minggangw

lib/interface_loader.js

@@ -148,6 +148,9 @@ let interfaceLoader = {
         }
       }
     }
+    throw new Error(
+      `The message required does not exist: ${packageName}, ${type}, ${messageName} at ${generator.generatedRoot}`
+    );
   },
 
   loadInterface(packageName, type, messageName) {
@@ -160,7 +163,6 @@ let interfaceLoader = {
       }
       throw new Error(`The message required does not exist: ${type}`);
     }
-
     if (packageName && type && messageName) {
       let filePath = path.join(
         generator.generatedRoot,
@@ -179,6 +181,7 @@ let interfaceLoader = {
         );
       }
     }
+    // We cannot parse `packageName`, `type` and `messageName` from the string passed.
     throw new Error(
       `The message required does not exist: ${packageName}, ${type}, ${messageName} at ${generator.generatedRoot}`
     );

rosidl_gen/index.js

@@ -55,11 +55,12 @@ async function generateInPath(path) {
 }
 
 function generateInPathSyncWorker(targetPath) {
+  console.log('generateInPathSyncWorker' + targetPath);
   try {
     // Use child_process.spawnSync for truly synchronous execution
     const result = require('child_process').spawnSync(
       'node',
-      [path.join(__dirname, 'generate-worker.js')],
+      [path.join(__dirname, 'generate_worker.js')],
       {
         env: { ...process.env, WORKER_TARGET_PATH: targetPath },
         encoding: 'utf8',

test/overlay_test_ws/install/geometry_msgs/share/geometry_msgs/msg/Testing.msg

@@ -149,7 +149,8 @@ describe('Rclnodejs createMessage() testing', function () {
     it('expecting exception when passing ' + testData.toString(), function () {
       assert.throws(
         () => {
-          rclnodejs.createMessage(testData);
+          const t = rclnodejs.createMessage(testData);
+          console.log(t);
         },
         function (e) {
           return e instanceof Error;