peterxcli opened a new pull request, #5941: URL: https://github.com/apache/datafusion-comet/pull/5941
## Which issue does this PR close? No linked issue; this adds test coverage for the existing SQL fixture parser. ## Rationale for this change The parser tests cover query assertion modes but do not directly check the configuration and version metadata that control fixture execution, or the statement records and table names used for diagnostics and cleanup. ## What changes are included in this PR? Add four tests for empty/comment-only fixtures, configuration ordering and whitespace, independent Spark version bounds, and mixed statement/query records. Check multiline SQL, source line numbers, table extraction, and the final record with and without a trailing blank line. ## How are these changes tested? All 12 tests in `SqlFileTestParserSuite` pass locally on Spark 3.5.9 and 4.1.3 with JDK 17, including the four new tests. `make format` (Spark 3.5) and Apache RAT passed. -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
