Skip to content

Commit

Permalink
Fix mysqli_get_client_stats test
Browse files Browse the repository at this point in the history
MySQL removed RENAME DATABASE in 18300001c1dbbfddf9a0adcbaeea68956102bdd0
(Sept 2007, 5.1.23). As this briefest existance is very insignificant lets remove it.

It also breaks when testing against MariaDB.

As the alternate path in this test covers all supported MySQL and MariaDB
versions and a signifant portion of unsupported versions lets keep it simple.

Closes GH-6459.
  • Loading branch information
grooverdan authored and cmb69 committed Nov 27, 2020
1 parent b5ce6df commit 4a10037
Showing 1 changed file with 8 additions and 20 deletions.
28 changes: 8 additions & 20 deletions ext/mysqli/tests/mysqli_get_client_stats.phpt
Original file line number Diff line number Diff line change
Expand Up @@ -666,7 +666,7 @@ mysqli.allow_local_infile=1
$info = $new_info;


// CREATE, ALTER, RENAME, DROP DATABASE
// CREATE, ALTER, DROP DATABASE
if (mysqli_query($link, "CREATE DATABASE mysqli_get_client_stats")) {

if (!is_array($new_info = mysqli_get_client_stats()) || empty($new_info))
Expand All @@ -685,25 +685,13 @@ mysqli.allow_local_infile=1
mysqli_get_client_stats_assert_eq('non_result_set_queries', $new_info, (string)($info['non_result_set_queries'] + 1), $test_counter, 'CREATE DATABASE');
$info = $new_info;

if (mysqli_get_server_version($link) > 51700) {
if (!mysqli_query($link, "RENAME DATABASE mysqli_get_client_stats TO mysqli_get_client_stats_"))
printf("[%03d] RENAME DATABASE failed, [%d] %s\n", ++$test_counter,
mysqli_errno($link), mysqli_error($link));

if (!is_array($new_info = mysqli_get_client_stats()) || empty($new_info))
printf("[%03d] Expecting array/any_non_empty, got %s/%s\n",
++$test_counter, gettype($new_info), $new_info);
mysqli_get_client_stats_assert_eq('non_result_set_queries', $new_info, (string)($info['non_result_set_queries'] + 1), $test_counter, 'CREATE DATABASE');
$info = $new_info;
} else {
if (!mysqli_query($link, "CREATE DATABASE mysqli_get_client_stats_"))
printf("[%03d] CREATE DATABASE failed, [%d] %s\n", ++$test_counter,
mysqli_errno($link), mysqli_error($link));
if (!is_array($new_info = mysqli_get_client_stats()) || empty($new_info))
printf("[%03d] Expecting array/any_non_empty, got %s/%s\n",
++$test_counter, gettype($new_info), $new_info);
$info = $new_info;
}
if (!mysqli_query($link, "CREATE DATABASE mysqli_get_client_stats_"))
printf("[%03d] CREATE DATABASE failed, [%d] %s\n", ++$test_counter,
mysqli_errno($link), mysqli_error($link));
if (!is_array($new_info = mysqli_get_client_stats()) || empty($new_info))
printf("[%03d] Expecting array/any_non_empty, got %s/%s\n",
++$test_counter, gettype($new_info), $new_info);
$info = $new_info;

if (!mysqli_query($link, "DROP DATABASE mysqli_get_client_stats_"))
printf("[%03d] DROP DATABASE failed, [%d] %s\n", ++$test_counter,
Expand Down

0 comments on commit 4a10037

Please sign in to comment.