Replacing TAP test planning with done_testing()
Daniel Gustafsson <daniel@yesql.se>
From: Daniel Gustafsson <daniel@yesql.se>
To: PostgreSQL Hackers <pgsql-hackers@lists.postgresql.org>
Date: 2022-02-09T14:01:36Z
Lists: pgsql-hackers
Attachments
- 0001-Replace-Test-More-plans-with-done_testing.patch (application/octet-stream) patch 0001
Whether or not to explicitly plan the number of TAP tests per suite has been discussed a number of times on this list, often as a side-note in an unrelated thread which adds/modifies a test. The concensus has so far weighed towards not doing manual bookkeeping of test plans but to let Test::More deal with it. So far, no concrete patch to make that happen has been presented though. The attached patch removes all Test::More planning and instead ensures that all tests conclude with a done_testing() call. While there, I also removed a few exit(0) calls from individual tests making them more consistent. Thoughts? -- Daniel Gustafsson https://vmware.com/
Commits
-
Replace Test::More plans with done_testing
- 549ec201d613 15.0 landed