On 17.03.22 19:04, Fabien COELHO wrote:
Indeed! The missing part was put back in v17.

Some unrelated notes on this v17 patch:

-use Test::More;
+use Test::More tests => 41;

The test counts are not needed/wanted anymore.


+
+\set ECHO none
+

This seems inappropriate.


+--
+-- autocommit
+--

+--
+-- test ON_ERROR_ROLLBACK
+--

This test file already contains tests for autocommit and ON_ERROR_ROLLBACK. If you want to change those, please add yours into the existing sections, not make new ones. I'm not sure if your tests add any new coverage, or if it is just duplicate.


Reply via email to