rbarbey commented on PR #19008: URL: https://github.com/apache/pulsar/pull/19008#issuecomment-1361109528
@eolivelli I've added unit tests for `TableViewImpl` and `TableViewBuilderImpl` (there weren't any before and I took heavy inspiration on similar tests relating to `Producer`s and `Consumer`s. Then I found `TableViewImplTest` in the `pulsar-broker` module which is an integration and there I've added a test that actually shows that encrypted messages can be read inside `TableViewImpl`. I've run the tests locally (using the Maven commands I found in the workflow YAML) and they were passing so they should pass here as well. -- 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]
