Skip to content

Commit 4e51059

Browse files
committed
PDO MySQL: Fix libmysql option test
The fact that getAttribute() fails for various libmysqlclient-only options is a known issue, and the test was taking that into account. However, the change of the default error mode broke the handling. We need to handle the exceptions now.
1 parent 43741a3 commit 4e51059

File tree

1 file changed

+32
-37
lines changed

1 file changed

+32
-37
lines changed

ext/pdo_mysql/tests/pdo_mysql___construct_options_libmysql.phpt

Lines changed: 32 additions & 37 deletions
Original file line numberDiff line numberDiff line change
@@ -27,57 +27,52 @@ if (MySQLPDOTest::isPDOMySQLnd())
2727
$tmp, gettype($tmp),
2828
$option_desc);
2929
} catch (PDOException $e) {
30-
printf("[%03d] %s\n", $offset, $e->getMessage());
30+
echo "Failed to getAttribute() for $option_desc\n";
3131
}
32-
3332
}
3433

35-
try {
36-
37-
$dsn = MySQLPDOTest::getDSN();
38-
$user = PDO_MYSQL_TEST_USER;
39-
$pass = PDO_MYSQL_TEST_PASS;
34+
$dsn = MySQLPDOTest::getDSN();
35+
$user = PDO_MYSQL_TEST_USER;
36+
$pass = PDO_MYSQL_TEST_PASS;
4037

41-
$valid_options = array();
42-
$valid_options[PDO::MYSQL_ATTR_MAX_BUFFER_SIZE] = 'PDO::MYSQL_ATTR_MAX_BUFFER_SIZE';
43-
$valid_options[PDO::MYSQL_ATTR_INIT_COMMAND] = 'PDO::MYSQL_ATTR_INIT_COMMAND';
44-
$valid_options[PDO::MYSQL_ATTR_READ_DEFAULT_FILE] = 'PDO::MYSQL_ATTR_READ_DEFAULT_FILE';
45-
$valid_options[PDO::MYSQL_ATTR_READ_DEFAULT_GROUP] = 'PDO::MYSQL_ATTR_READ_DEFAULT_GROUP';
38+
$valid_options = array();
39+
$valid_options[PDO::MYSQL_ATTR_MAX_BUFFER_SIZE] = 'PDO::MYSQL_ATTR_MAX_BUFFER_SIZE';
40+
$valid_options[PDO::MYSQL_ATTR_INIT_COMMAND] = 'PDO::MYSQL_ATTR_INIT_COMMAND';
41+
$valid_options[PDO::MYSQL_ATTR_READ_DEFAULT_FILE] = 'PDO::MYSQL_ATTR_READ_DEFAULT_FILE';
42+
$valid_options[PDO::MYSQL_ATTR_READ_DEFAULT_GROUP] = 'PDO::MYSQL_ATTR_READ_DEFAULT_GROUP';
4643

47-
$defaults[PDO::MYSQL_ATTR_MAX_BUFFER_SIZE] = 1048576;
48-
/* TODO getAttribute() does not handle it */
49-
$defaults[PDO::MYSQL_ATTR_INIT_COMMAND] = '';
50-
$defaults[PDO::MYSQL_ATTR_READ_DEFAULT_FILE] = false;
51-
$defaults[PDO::MYSQL_ATTR_READ_DEFAULT_GROUP] = false;
44+
$defaults[PDO::MYSQL_ATTR_MAX_BUFFER_SIZE] = 1048576;
45+
/* TODO getAttribute() does not handle it */
46+
$defaults[PDO::MYSQL_ATTR_INIT_COMMAND] = '';
47+
$defaults[PDO::MYSQL_ATTR_READ_DEFAULT_FILE] = false;
48+
$defaults[PDO::MYSQL_ATTR_READ_DEFAULT_GROUP] = false;
5249

53-
$db = new PDO($dsn, $user, $pass);
54-
foreach ($valid_options as $option => $name) {
55-
/* TODO getAttribute() is pretty poor in supporting the options, suppress errors */
56-
$tmp = @$db->getAttribute($option);
50+
$db = new PDO($dsn, $user, $pass);
51+
foreach ($valid_options as $option => $name) {
52+
try {
53+
$tmp = $db->getAttribute($option);
5754
if ($tmp !== $defaults[$option])
5855
printf("[001] Expecting default value for '%s' of '%s'/%s, getAttribute() reports setting '%s'/%s\n",
5956
$name, $defaults[$option], gettype($defaults[$option]),
6057
$tmp, gettype($tmp));
58+
} catch (PDOException $e) {
59+
echo "Failed to getAttribute() for $name\n";
6160
}
61+
}
6262

63-
set_option_and_check(26, PDO::MYSQL_ATTR_READ_DEFAULT_FILE, true, 'PDO::MYSQL_ATTR_READ_DEFAULT_FILE');
64-
set_option_and_check(27, PDO::MYSQL_ATTR_READ_DEFAULT_FILE, false, 'PDO::MYSQL_ATTR_READ_DEFAULT_FILE');
65-
66-
set_option_and_check(30, PDO::MYSQL_ATTR_MAX_BUFFER_SIZE, -1, 'PDO::MYSQL_ATTR_MAX_BUFFER_SIZE', true);
67-
set_option_and_check(31, PDO::MYSQL_ATTR_MAX_BUFFER_SIZE, PHP_INT_MAX, 'PDO::MYSQL_ATTR_MAX_BUFFER_SIZE');
68-
set_option_and_check(32, PDO::MYSQL_ATTR_MAX_BUFFER_SIZE, 1, 'PDO::MYSQL_ATTR_MAX_BUFFER_SIZE');
69-
63+
set_option_and_check(26, PDO::MYSQL_ATTR_READ_DEFAULT_FILE, true, 'PDO::MYSQL_ATTR_READ_DEFAULT_FILE');
64+
set_option_and_check(27, PDO::MYSQL_ATTR_READ_DEFAULT_FILE, false, 'PDO::MYSQL_ATTR_READ_DEFAULT_FILE');
7065

71-
} catch (PDOException $e) {
72-
printf("[001] %s, [%s] %s\n",
73-
$e->getMessage(),
74-
(is_object($db)) ? $db->errorCode() : 'n/a',
75-
(is_object($db)) ? implode(' ', $db->errorInfo()) : 'n/a');
76-
}
66+
set_option_and_check(30, PDO::MYSQL_ATTR_MAX_BUFFER_SIZE, -1, 'PDO::MYSQL_ATTR_MAX_BUFFER_SIZE', true);
67+
set_option_and_check(31, PDO::MYSQL_ATTR_MAX_BUFFER_SIZE, PHP_INT_MAX, 'PDO::MYSQL_ATTR_MAX_BUFFER_SIZE');
68+
set_option_and_check(32, PDO::MYSQL_ATTR_MAX_BUFFER_SIZE, 1, 'PDO::MYSQL_ATTR_MAX_BUFFER_SIZE');
7769

78-
print "done!";
70+
print "done!\n";
7971
?>
8072
--EXPECT--
81-
[001] Expecting default value for 'PDO::MYSQL_ATTR_INIT_COMMAND' of ''/string, getAttribute() reports setting ''/boolean
82-
[026] Expecting '1'/boolean got ''/boolean' for options 'PDO::MYSQL_ATTR_READ_DEFAULT_FILE'
73+
Failed to getAttribute() for PDO::MYSQL_ATTR_INIT_COMMAND
74+
Failed to getAttribute() for PDO::MYSQL_ATTR_READ_DEFAULT_FILE
75+
Failed to getAttribute() for PDO::MYSQL_ATTR_READ_DEFAULT_GROUP
76+
Failed to getAttribute() for PDO::MYSQL_ATTR_READ_DEFAULT_FILE
77+
Failed to getAttribute() for PDO::MYSQL_ATTR_READ_DEFAULT_FILE
8378
done!

0 commit comments

Comments
 (0)