Revert "Fix more tests with $sth handles"

This reverts commit bd1d2f6.

According to DBI documentation $sth->finish() should not be called after
fetching all data.

Author: pali

t/25lockunlock.t

@@ -40,7 +40,6 @@ my ($row, $errstr);
 $errstr= '';
 $row = $sth->fetchrow_arrayref;
 $errstr= $sth->errstr;
-$sth->finish;
 ok !defined($row), "Fetch should have failed";
 ok !defined($errstr), "Fetch should have failed";
 

t/35prepare.t

@@ -13,7 +13,7 @@ use vars qw($test_dsn $test_user $test_password);
 $dbh = DbiTestConnect($test_dsn, $test_user, $test_password,
     { RaiseError => 1, PrintError => 0 });
 
-plan tests => 50;
+plan tests => 49;
 
 ok(defined $dbh, "Connected to database");
 
@@ -94,7 +94,6 @@ ok($sth= $dbh->prepare("SELECT 1"), "Prepare - Testing bug #20153");
 ok($sth->execute(), "Execute - Testing bug #20153");
 ok($sth->fetchrow_arrayref(), "Fetch - Testing bug #20153");
 ok(!($sth->fetchrow_arrayref()),"Not Fetch - Testing bug #20153");
-ok($sth->finish);
 
 # Install a handler so that a warning about unfreed resources gets caught
 $SIG{__WARN__} = sub { die @_ };

t/40numrows.t

@@ -10,7 +10,7 @@ require 'lib.pl';
 my ($dbh, $sth, $aref);
 $dbh = DbiTestConnect($test_dsn, $test_user, $test_password,
                       { RaiseError => 1, PrintError => 0, AutoCommit => 0 });
-plan tests => 2*33 + 1;
+plan tests => 2*32 + 1;
 
 for my $mariadb_server_prepare (0, 1) {
 
@@ -85,8 +85,6 @@ is scalar @$aref, 3, 'Verified rows should be 3';
 
 is $sth->rows, 3, 'rows still should be 3';
 
-ok $sth->finish;
-
 ok $dbh->do("DROP TEMPORARY TABLE dbd_mysql_t40numrows"), "drop table dbd_mysql_t40numrows";
 
 }