Reorder connection markers in libpq TAP tests
Commit 7f5b198 introduced TAP tests that use string literals to mark
the presence of a query in server logs. For no explicable reason, the
tests with the marker 'connect2' occur before the tests that use
'connect1' marker.
The attached patch swaps the connection marker strings so that a
reader doesn't have to spend extra deciphering why 'connect2' tests
appear before 'connect1' tests.
Best regards,
Gurjeet https://Gurje.et
Postgres Contributors Team, http://aws.amazon.com
Attachments:
v0-0001-Reorder-connection-markers-in-TAP-tests.patchapplication/octet-stream; name=v0-0001-Reorder-connection-markers-in-TAP-tests.patchDownload+6-7
On Thu, Apr 20, 2023 at 11:00 PM Gurjeet Singh <gurjeet@singh.im> wrote:
Commit 7f5b198 introduced TAP tests that use string literals to mark
the presence of a query in server logs. For no explicable reason, the
tests with the marker 'connect2' occur before the tests that use
'connect1' marker.The attached patch swaps the connection marker strings so that a
reader doesn't have to spend extra deciphering why 'connect2' tests
appear before 'connect1' tests.
Please see attached v2 of the patch. It now includes same fix in
another TAP tests file.
Best regards,
Gurjeet https://Gurje.et
Postgres Contributors Team, http://aws.amazon.com
Attachments:
v2-0001-Reorder-connection-markers-in-TAP-tests.patchapplication/x-patch; name=v2-0001-Reorder-connection-markers-in-TAP-tests.patchDownload+13-14
LGTM I guess this was an unintended leftover from me reordering the tests a
bit in the final stages of getting these patches in.
On Fri, 21 Apr 2023 at 08:38, Gurjeet Singh <gurjeet@singh.im> wrote:
Show quoted text
On Thu, Apr 20, 2023 at 11:00 PM Gurjeet Singh <gurjeet@singh.im> wrote:
Commit 7f5b198 introduced TAP tests that use string literals to mark
the presence of a query in server logs. For no explicable reason, the
tests with the marker 'connect2' occur before the tests that use
'connect1' marker.The attached patch swaps the connection marker strings so that a
reader doesn't have to spend extra deciphering why 'connect2' tests
appear before 'connect1' tests.Please see attached v2 of the patch. It now includes same fix in
another TAP tests file.Best regards,
Gurjeet https://Gurje.et
Postgres Contributors Team, http://aws.amazon.com
On 21 Apr 2023, at 08:38, Gurjeet Singh <gurjeet@singh.im> wrote:
On Thu, Apr 20, 2023 at 11:00 PM Gurjeet Singh <gurjeet@singh.im> wrote:
Commit 7f5b198 introduced TAP tests that use string literals to mark
the presence of a query in server logs. For no explicable reason, the
tests with the marker 'connect2' occur before the tests that use
'connect1' marker.The attached patch swaps the connection marker strings so that a
reader doesn't have to spend extra deciphering why 'connect2' tests
appear before 'connect1' tests.Please see attached v2 of the patch. It now includes same fix in
another TAP tests file.
- 'Potentially unsafe test load_balance not enabled in PG_TEST_EXTRA';
+ 'Potentially unsafe test; load_balance not enabled in PG_TEST_EXTRA';We have this spelling without a ';' in multiple places so I left this alone.
I have applied this version of the patch apart from the above hunk. Thanks!
--
Daniel Gustafsson