Skip to content

Commit

Permalink
Remove calls to deprecated assertInternalType()
Browse files Browse the repository at this point in the history
  • Loading branch information
psrpinto committed Jan 10, 2024
1 parent 49fc738 commit ddfa196
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 3 deletions.
4 changes: 2 additions & 2 deletions tests/phpunit/testcases/test_meta.php
Original file line number Diff line number Diff line change
Expand Up @@ -62,7 +62,7 @@ function test_update_meta_should_set_meta() {
}

function test_gp_update_meta_updates_an_existing_meta_value() {
$this->assertInternalType( 'int', gp_update_meta( '1', 'key', 'value-1', 'thing' ) );
$this->assertIsInt( gp_update_meta( '1', 'key', 'value-1', 'thing' ) );
$this->assertTrue( gp_update_meta( '1', 'key', 'value-2', 'thing' ) );
$this->assertSame( 'value-2', gp_get_meta( 'thing', '1', 'key' ) );
}
Expand All @@ -84,7 +84,7 @@ function test_delete_meta_with_value_should_delete_only_meta_with_value() {
}

function test_gp_update_meta_does_not_update_if_prev_value_equals_new_value() {
$this->assertInternalType( 'int', gp_update_meta( '1', 'foo', 'foo', 'thing' ) );
$this->assertIsInt( gp_update_meta( '1', 'foo', 'foo', 'thing' ) );
$this->assertTrue( gp_update_meta( '1', 'foo', 'foo', 'thing' ) ); // @todo Is this the correct return value?
}

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -273,7 +273,7 @@ function test_return_types_of_count_by_project_id() {
$this->assertSame( 2, $total );

$all = GP::$original->count_by_project_id( $project->id, 'all' );
$this->assertInternalType( 'object', $all );
$this->assertIsObject( $all );
$this->assertSame( 2, $all->total );
$this->assertSame( 1, $all->hidden );
$this->assertSame( 1, $all->public );
Expand Down

0 comments on commit ddfa196

Please sign in to comment.