Fix test failure

Author: minggangw

test/test-action-client.js

@@ -317,6 +317,28 @@ describe('rclnodejs action client', function () {
       typeof nativeLoader.actionConfigureFeedbackSubFilterAddGoalId ===
       'function';
 
+    // Probe whether the RMW actually supports content filtering, matching
+    // the same check the ActionClient constructor performs.
+    let isContentFilterSupported = false;
+    before(function () {
+      if (isFeedbackFilterSupported()) {
+        try {
+          const Fibonacci_ = rclnodejs.require(fibonacci);
+          const probeSub = node.createSubscription(
+            Fibonacci_.FeedbackMessage,
+            'fibonacci/_action/feedback',
+            () => {}
+          );
+          isContentFilterSupported =
+            typeof probeSub.isContentFilterSupported === 'function' &&
+            probeSub.isContentFilterSupported();
+          probeSub.destroy();
+        } catch {
+          isContentFilterSupported = false;
+        }
+      }
+    });
+
     it('Test option defaults to false', function () {
       let client = new rclnodejs.ActionClient(node, fibonacci, 'fibonacci');
       assert.strictEqual(client._enableFeedbackMsgOptimization, false);
@@ -327,8 +349,8 @@ describe('rclnodejs action client', function () {
       let client = new rclnodejs.ActionClient(node, fibonacci, 'fibonacci', {
         enableFeedbackMsgOptimization: true,
       });
-      // If native API is available, it should be enabled; otherwise disabled
-      if (isFeedbackFilterSupported()) {
+      // Only enabled when native API exists AND the RMW supports content filtering
+      if (isFeedbackFilterSupported() && isContentFilterSupported) {
         assert.strictEqual(client._enableFeedbackMsgOptimization, true);
       } else {
         assert.strictEqual(client._enableFeedbackMsgOptimization, false);