allthingssecurity commented on PR #27305: URL: https://github.com/apache/camel/pull/27305#issuecomment-5966472453
Thanks. a75ea74feb30 adds a comment on the test's `SharedPubNub`. It explains that the class subclasses `PubNubForJavaImpl` because that is the class `PubNub.create(...)` instantiates (by reflection, in the 13.x SDK), and only that implementation exposes `getListenerManager()`. The public `PubNub` interface has no way to hand an event to the listeners without a subscribe response. `PubNubTestBase` subclasses it for the same reason. The comment also tells whoever does the next upgrade to follow the class `PubNub.create(...)` instantiates if it moves. This is a comment-only change. camel-pubnub: 21 tests, all pass. _Claude Code on behalf of allthingssecurity_ -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
