KenanAdel commented on PR #28475: URL: https://github.com/apache/flink/pull/28475#issuecomment-4794583912
> > Previously, some methods could access argument metadata and values using direct list lookups, which could lead to IndexOutOfBoundsException when a test declared optional arguments but did not provide corresponding entries. This change makes the mock handle missing optional arguments consistently and safely. > > how in this PR we can be sure that it is fixed and will not be broken after some other code change? Thanks for the feedback. I added regression tests for CallContextMock covering missing argument metadata, missing values, and out-of-bounds access The tests ensure the issue won’t be reintroduced in future changes. All tests are passing in CI. -- 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]
