s5dsn-eqee opened a new pull request, #2713:
URL: https://github.com/apache/datafusion-sqlparser-rs/pull/2713
Docs: Assert exact parser errors in some negative tests
Motivation: when a negative test asserts the exact error, a change to the
parser branch that rejects the input shows up as a failing test. That helps
a developer or an agent see which branches a PR changed, straight from the
test diff. A bare `is_err()` passes no matter which branch rejected the
input.
The usual objection is maintenance: when error messages change, updating
many exact assertions by hand is slow and tedious. With an LLM, updating
them is now a few minutes of work, so the extra signal is worth the cost.
--
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]
