Meganexxx opened a new pull request, #13095:
URL: https://github.com/apache/seatunnel/pull/13095

   ### Purpose
   
   Part of #11007.
   
   Move missing and empty `rules` validation to 
`AssertSinkFactory.optionRule()` using `mapNotEmpty`.
   
   Keep the runtime check for rules that produce no effective assertions, and 
preserve a clear error when `AssertSink` is constructed directly without 
`rules`.
   
   ### User-facing changes
   
   Invalid configurations with empty `rules` now fail earlier during 
configuration validation. Valid configurations and assertion behavior remain 
unchanged.
   
   ### Testing
   
   - Added tests for missing, empty, and valid `rules`.
   - Added tests for direct sink construction and effectively empty rules.
   - Red-Green regression: 2 expected test failures without `mapNotEmpty`.
   - Module tests: `mvn -pl seatunnel-connectors-v2/connector-assert test`.


-- 
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]

Reply via email to