@ahcorde i am not really sure what needs to be avoided here... what happens if this if statement is not there, exception? and this is user application does to configure PublisherOptions, that means just switching rmw would break the user application?

I believe if we run this test with rmw_zenoh, an exception will be thrown when rcl will initialize a liveliness or deadline event since we don't support this yet. This happens when we set certain event callbacks in pub/sub options which internally invoke rmw_publisher_event_init/rmw_subscription_event_init. See below.

108: C++ exception with description "Failed to initialize event: provided event_type 7 is not supported by rmw_zenoh_cpp, at /home/yadunund/ros2_rolling/src/ros2/rmw_zenoh/rmw_zenoh_cpp/src/rmw_event.cpp:60" thrown in the test body.

Having said that I think the way to address this would be to set the matched or qos_incompatible callbacks for po.event_callbacks and subscription_options.event_callbacks. instead of the liveliness or deadlines one if rmw_implementation_str == "rmw_zenoh_cpp"

@Yadunund can we replace this with #2648? i think #2648 is more reasonable, and we do not need to bypass the tests for rmw_zenoh like this or any other rmw implementations that do not support this feature.