sergiotarxz/Wordpress
1
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

Tests: Correct assertion in Tests_DB::test_prepare_incorrect_arg_count().

Browse Source 
On failure, `wpdb::prepare()` returns either an empty string or `null`, not `false`.

The test only passed accidentally due to `assertEquals()` not performing a strict type check.

Follow-up to [41662].

See #38266.

git-svn-id: https://develop.svn.wordpress.org/trunk@48974 602fd350-edb4-49c9-b593-d223f7449a82
This commit is contained in:
Sergey Biryukov 2020-09-13 15:19:40 +00:00
parent 2d01fac875
commit 91feaf04af
1 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
tests/phpunit/tests/db.php
View File

@ -427,7 +427,7 @@ class Tests_DB extends WP_UnitTestCase {
// phpcs:ignore WordPress.PHP.NoSilencedErrors.Discouraged,WordPress.DB.PreparedSQL
$prepared = @$wpdb->prepare( $query, ...$args );
$this->assertEquals( $expected, $prepared );
$this->assertSame( $expected, $prepared );
}
public function data_prepare_incorrect_arg_count() {
@ -442,7 +442,7 @@ class Tests_DB extends WP_UnitTestCase {
array(
"SELECT * FROM $wpdb->users WHERE id = %%%d AND user_login = %s",
array( 1 ),
false,
'',
),
array(
"SELECT * FROM $wpdb->users WHERE id = %d AND user_login = %s",