Commit efbe647
committed
test(e2e/mst): DESCRIBE QUERY now allowed in transactions
The server's MSTCheckRule allowlist has broadened to include DESCRIBE
QUERY (DescribeQueryCommand), mirroring the earlier SHOW COLUMNS change.
It no longer throws inside an active transaction, so the prior
test_describe_query_blocked assertion (DID NOT RAISE) was stale.
Flip it to test_describe_query_not_blocked using _assert_not_blocked
(verifies it succeeds and returns >0 rows) and move DESCRIBE QUERY from
the Blocked to the Allowed list in the class docstring.
Verified against a live DBSQL warehouse: the full TestMstBlockedSql
class (9 tests) passes.
Co-authored-by: Isaac1 parent a2fe99f commit efbe647
1 file changed
Lines changed: 11 additions & 5 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
565 | 565 | | |
566 | 566 | | |
567 | 567 | | |
568 | | - | |
| 568 | + | |
| 569 | + | |
569 | 570 | | |
570 | 571 | | |
571 | 572 | | |
572 | | - | |
| 573 | + | |
573 | 574 | | |
574 | 575 | | |
575 | 576 | | |
576 | 577 | | |
577 | 578 | | |
578 | 579 | | |
| 580 | + | |
579 | 581 | | |
580 | 582 | | |
581 | 583 | | |
| |||
652 | 654 | | |
653 | 655 | | |
654 | 656 | | |
655 | | - | |
656 | | - | |
| 657 | + | |
| 658 | + | |
| 659 | + | |
| 660 | + | |
| 661 | + | |
| 662 | + | |
657 | 663 | | |
658 | | - | |
| 664 | + | |
659 | 665 | | |
660 | 666 | | |
661 | 667 | | |
| |||
0 commit comments