]> granicus.if.org Git - php/commitdiff
oci8: Fix two tests that fail with stricter internal checks
authorCameron Porter <camporter1@gmail.com>
Tue, 3 Mar 2020 04:11:55 +0000 (22:11 -0600)
committerNikita Popov <nikita.ppv@gmail.com>
Tue, 3 Mar 2020 11:13:44 +0000 (12:13 +0100)
Closes GH-5230

ext/oci8/tests/array_bind_009.phpt
ext/oci8/tests/array_bind_013.phpt

index b00efc0c24311b825efea9436323d42de00d1446..b1edfa84748723cdb2267eb83686757b9a61d214 100644 (file)
@@ -8,10 +8,10 @@ oci_bind_array_by_name() and invalid values 9
 require __DIR__.'/connect.inc';
 
 var_dump(oci_bind_array_by_name($c, ":c1", $array, 5, 5, SQLT_CHR));
-
-echo "Done\n";
 ?>
 --EXPECTF--
-Warning: oci_bind_array_by_name(): supplied resource is not a valid oci8 statement resource in %s on line %d
-bool(false)
-Done
+Fatal error: Uncaught TypeError: oci_bind_array_by_name(): supplied resource is not a valid oci8 statement resource in %s:%d
+Stack trace:
+#0 %s: oci_bind_array_by_name(%s)
+#1 {main}
+  thrown in %s on line %d
\ No newline at end of file
index 0d9ddabc1501c3fe9e7e8eab46159ff8bb1ade19..a04bccf30423b19209cd5e6c89bf67c0707059cf 100644 (file)
@@ -11,7 +11,12 @@ $statement = oci_parse($c, 'SELECT user FROM v$session');
 
 $array = array();
 
-var_dump(oci_bind_array_by_name($statement, ":c1", $array, 5, -10, SQLT_CHR, -10));
+try {
+    var_dump(oci_bind_array_by_name($statement, ":c1", $array, 5, -10, SQLT_CHR, -10));
+} catch (ArgumentCountError $exception) {
+    echo $exception->getMessage() . "\n";
+}
+
 var_dump(oci_bind_array_by_name($statement, ":c1", $array, 5, -10));
 var_dump(oci_bind_array_by_name($statement, ":c1", $array, 5, -1));
 var_dump(oci_bind_array_by_name($statement, ":c1", $array, 5, 0));
@@ -23,8 +28,7 @@ var_dump($array);
 echo "Done\n";
 ?>
 --EXPECTF--
-Warning: oci_bind_array_by_name() expects at most 6 parameters, 7 given in %s on line %d
-NULL
+oci_bind_array_by_name() expects at most 6 parameters, 7 given
 
 Warning: oci_bind_array_by_name(): You must provide max length value for empty arrays in %s on line %d
 bool(false)