nastra commented on code in PR #12555: URL: https://github.com/apache/iceberg/pull/12555#discussion_r2011779235
########## core/src/test/java/org/apache/iceberg/rest/auth/TestOAuth2Manager.java: ########## @@ -130,24 +136,26 @@ void catalogSessionNoOAuth2Properties() { @Test void catalogSessionTokenProvided() { Map<String, String> properties = Map.of(OAuth2Properties.TOKEN, "test"); - try (OAuth2Manager manager = new OAuth2Manager("test"); - OAuth2Util.AuthSession catalogSession = manager.catalogSession(client, properties)) { + try (OAuth2Manager manager = new OAuth2Manager("test").withClient(client).withClient(client); Review Comment: ```suggestion try (OAuth2Manager manager = new OAuth2Manager("test").withClient(client); ``` -- 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: issues-unsubscr...@iceberg.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: issues-unsubscr...@iceberg.apache.org For additional commands, e-mail: issues-h...@iceberg.apache.org