snazy commented on PR #2680: URL: https://github.com/apache/polaris/pull/2680#issuecomment-3443583203
> Just so I'm clear: is your suggestion to move the QuarkusIntegrationTest annotated tests back into `polaris-runtime-service`? Yes and no ;) Have non-Quarkus integration tests (like `OpaAuthorizerIT`) in the "main" OPA module and have a minimal set of OPA integration tests in polaris-runtime-service just to verify the integration with Quarkus. I suspect we don't need that many ITs in runtime-service, probably more or less just a "smoke test". I'm not really familiar with what's worth being integration tested in the non-Quarkus ITs. Test coverage for the OPA factory and the authorizer itself is pretty good. -- 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]
