From f801c3a2a76455ff896ce40b73fef74da18d5f25 Mon Sep 17 00:00:00 2001 From: Kaspars Dambis Date: Fri, 14 Feb 2025 15:59:01 +0200 Subject: [PATCH] Check for known things instead --- tests/class-two-factor-core.php | 9 +++------ 1 file changed, 3 insertions(+), 6 deletions(-) diff --git a/tests/class-two-factor-core.php b/tests/class-two-factor-core.php index 268432a5..a8ad1816 100644 --- a/tests/class-two-factor-core.php +++ b/tests/class-two-factor-core.php @@ -1568,7 +1568,7 @@ public function test_can_filter_registered_providers_for_user() { add_filter( 'two_factor_providers_for_user', function( $providers, $user ) { - $this->assertInstanceOf( WP_User::class, $user ); + $this->assertInstanceOf( WP_User::class, $user, 'A user referenced is passed to the filter' ); return array_diff_key( $providers, array( 'Two_Factor_Email' => null ) ); }, @@ -1576,11 +1576,8 @@ function( $providers, $user ) { 2 ); - $providers = Two_Factor_Core::get_providers(); - unset( $providers['Two_Factor_Email'] ); - - $this->assertEquals( - $providers, + $this->assertNotContains( + 'Two_Factor_Email', Two_Factor_Core::get_supported_providers_for_user( $user ), 'Email provider can be disabled for a user' );