From aa8cfda4599e48d12b59008de50f7e91fa6d57b2 Mon Sep 17 00:00:00 2001 From: Kaspars Dambis Date: Fri, 14 Feb 2025 17:19:54 +0200 Subject: [PATCH] Test the filter impact to the new helper --- tests/class-two-factor-core.php | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/tests/class-two-factor-core.php b/tests/class-two-factor-core.php index e8303086..0a114b26 100644 --- a/tests/class-two-factor-core.php +++ b/tests/class-two-factor-core.php @@ -1558,13 +1558,16 @@ public function test_all_sessions_destroyed_when_enabling_2fa_by_admin() { public function test_can_filter_registered_providers_for_user() { $user = self::factory()->user->create_and_get(); + $providers = Two_Factor_Core::get_providers(); $this->assertEquals( - Two_Factor_Core::get_providers(), + $providers, Two_Factor_Core::get_supported_providers_for_user( $user ), 'All providers are available by default' ); + $this->assertTrue( $providers['Two_Factor_Email']::is_supported_for_user( $user ), 'Email provider is supported by default' ); + add_filter( 'two_factor_providers_for_user', function( $providers, $user ) { @@ -1582,6 +1585,8 @@ function( $providers, $user ) { 'Email provider can be disabled for a user' ); + $this->assertFalse( $providers['Two_Factor_Email']::is_supported_for_user( $user ), 'Email provider is disabled if not supported' ); + remove_all_filters( 'two_factor_providers_for_user' ); }