Fix(tests): Update expected SQL in StatementGenerator test #470

Closed
opened 2026-04-05 16:19:57 +02:00 by MrUnknownDE · 0 comments
Owner

Originally created by @simlarsen on 5/27/2025

The expected SQL statement in the toTableCreateStatement test was outdated and did not include the PARTITION BY clause, which is now correctly generated by the StatementGenerator.

This commit updates the expected statement to match the actual output, resolving the test failure.

Title of this pull request?

Small Description?

Pull Request Checklist:

  • Please make sure all jobs pass before requesting a review.
  • Put closes #XXXX in your comment to auto-close the issue that your PR fixes (if such).
  • Have you lint your code locally before submission?
  • Did you write tests where appropriate?

Screenshots (if appropriate):

*Originally created by @simlarsen on 5/27/2025* The expected SQL statement in the `toTableCreateStatement` test was outdated and did not include the `PARTITION BY` clause, which is now correctly generated by the StatementGenerator. This commit updates the expected statement to match the actual output, resolving the test failure. ### Title of this pull request? ### Small Description? ### Pull Request Checklist: - [ ] Please make sure all jobs pass before requesting a review. - [ ] Put `closes #XXXX` in your comment to auto-close the issue that your PR fixes (if such). - [ ] Have you lint your code locally before submission? - [ ] Did you write tests where appropriate? ### Related Issue? ### Screenshots (if appropriate):
Sign in to join this conversation.
No Label
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: github/oneuptime#470