Repository navigation
[no-release-notes] Run feature-specific GMS script test suites - #3491
Conversation
|
|
SummaryCoverage focuses on the application’s themed SQL behavior across supported queries, joins, subqueries, aggregations, compatibility boundaries, and harness lifecycle behavior. It also checks edge cases such as unsupported and multiline queries being skipped correctly while valid nearby queries continue to run, along with suite registration and dependency integration. Safe to merge — the exercised application behaviors completed successfully, including supported query execution, intentional compatibility skips, suite isolation, and dependency integration. No PR-attributable regressions or unresolved failures were identified; the run is low risk. Tests run by Ito
Tip Reply with @itoqa to send us feedback on this test run. |
|
@zachmu DOLT
|
|
Diff SummaryCoverage focuses on end-to-end database behavior for themed script suites, including supported query results, expected error handling, configured incompatibilities, suite registration, isolation, cleanup, and avoiding resource conflicts. The run also checks that upgraded scripts and aggregation flows behave correctly across both normal and edge-case scenarios. Safe to merge — the exercised behaviors remain healthy with no PR-attributable regressions, new failures, or previously flagged failures still failing. Several previously passing checks were not rerun, so they remain a flag for later rather than a merge blocker. Tests run by Ito
Tip Reply with @itoqa to send us feedback on this test run. |
|
Diff SummaryThe run covers PostgreSQL-compatible database behavior across normal role and permission flows, transaction and setting lifecycles, trigger handling, and adversarial type, catalog, and invalid-command edge cases. Core flows are broadly healthy, but a catalog-row compatibility path fails for valid whole-row function calls. Merge with caution — the PR introduces a medium-severity compatibility failure affecting valid catalog-row queries, so the change is not risk-free. Separate medium-severity findings are not attributable to this PR and are flag-for-later observations rather than merge blockers. Tests run by ItoAdditional Findings DetailsThese findings are unrelated to the current changes but were observed during testing. 🟡 Failed insert leaves a false audit record
Evidence Package🟡 Quoted composite types block function creation
Evidence PackageTip Reply with @itoqa to send us feedback on this test run. |
|
Diff SummaryCoverage centers on database scripting behavior: variable-processing flows complete successfully, supported cases remain eligible while incompatible cases are excluded, and expected script functionality remains discoverable. This is primarily happy-path and compatibility-boundary coverage, with no failures observed. Safe to merge — no PR-attributable regressions, new failures, or previously flagged failures remain in this run. The results show the changed behavior is healthy, while unrelated historical findings are flag-for-later observations rather than merge blockers. Tests run by Ito
Tests that are no longer relevantBelow are tests that previously ran and are no longer relevant:
Tip Reply with @itoqa to send us feedback on this test run. |
|
Diff SummaryCoverage spans normal execution of database scripts and views, discovery of registered behavior, compatibility handling, and clear failure reporting for missing definitions. It also exercises edge cases around transactions, invalid views, and table-warning behavior, with the application behavior remaining healthy throughout. Safe to merge — no regressions, new failures, or previously reported PR-related failures were found, and the exercised execution and compatibility behaviors all remain healthy. No merge blocker is indicated. Tests run by Ito
Tip Reply with @itoqa to send us feedback on this test run. |
|
Diff SummaryCoverage spans normal and edge-case database cursor behavior, including movement in multiple directions, zero-row operations, transaction and lifetime handling, catalog visibility, option validation, identifier parsing, and rapid close-and-reopen flows. Adversarial checks also confirm that unsupported rewinding behavior is rejected as intended, with the exercised behavior broadly healthy. Safe to merge — the run found no PR-attributable regressions or new failures, and the reviewed behavior is consistent with the documented contract. Some previously passing areas were not exercised or were removed as obsolete, so they remain follow-up coverage considerations rather than merge blockers. Tests run by Ito
Tests that are no longer relevantBelow are tests that previously ran and are no longer relevant:
Tip Reply with @itoqa to send us feedback on this test run. |


Uses the consolidated script-test runners from GMS #3990, alongside Dolt #12009.