forked frompostgres/postgres
- Notifications
You must be signed in to change notification settings - Fork6
Commit1b9cd69
committed
Add some tests to check the SQL functions of control file
As the recent commit05d4cbf (reverted after asa448e49) has proved,there is zero coverage for the four SQL functions that can scan thecontrol file data:- pg_control_checkpoint()- pg_control_init()- pg_control_recovery()- pg_control_system()This commit adds a minimal coverage for these functions, checking thattheir execution is able to complete. This would have been enough tocatch the problems introduced in the commit mentioned above. Morechecks could be done for each individual fields, but it is unclearwhether this would be better than the other checks in place in thebackend code.Per discussion with Bharath Rupireddy.Discussion:https://postgr.es/m/Y1d2FZmQmyAhPSRG@paquier.xyz1 parentc591300 commit1b9cd69
File tree
2 files changed
+31
-0
lines changed- src/test/regress
- expected
- sql
2 files changed
+31
-0
lines changedLines changed: 25 additions & 0 deletions
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
594 | 594 |
| |
595 | 595 |
| |
596 | 596 |
| |
| 597 | + | |
| 598 | + | |
| 599 | + | |
| 600 | + | |
| 601 | + | |
| 602 | + | |
| 603 | + | |
| 604 | + | |
| 605 | + | |
| 606 | + | |
| 607 | + | |
| 608 | + | |
| 609 | + | |
| 610 | + | |
| 611 | + | |
| 612 | + | |
| 613 | + | |
| 614 | + | |
| 615 | + | |
| 616 | + | |
| 617 | + | |
| 618 | + | |
| 619 | + | |
| 620 | + | |
| 621 | + |
Lines changed: 6 additions & 0 deletions
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
223 | 223 |
| |
224 | 224 |
| |
225 | 225 |
| |
| 226 | + | |
| 227 | + | |
| 228 | + | |
| 229 | + | |
| 230 | + | |
| 231 | + |
0 commit comments
Comments
(0)