V4.5.0

.distignore

@@ -9,14 +9,13 @@
 /.tests
 /.wordpress-org
 /.yarn
-/bin
 /node_modules
 /src/js
 
 # Files
 /.DS_Store
 /.babelrc
-/.codeception.yml
+/codeception.yml
 /.distignore
 /.editorconfig
 /.eslintrc.json

.tests/php/integration/AAAMainTest.php

@@ -26,6 +26,7 @@
 use HCaptcha\Jetpack\JetpackForm;
 use HCaptcha\Main;
 use HCaptcha\ElementorPro\HCaptchaHandler;
+use HCaptcha\Migrations\Migrations;
 use HCaptcha\NF\NF;
 use HCaptcha\Quform\Quform;
 use HCaptcha\Sendinblue\Sendinblue;
@@ -109,15 +110,41 @@ public function test_init(): void {
 		$hcaptcha = hcaptcha();
 
 		// The plugin was loaded by codeception.
-		self::assertSame( - PHP_INT_MAX, has_action( 'plugins_loaded', [ $hcaptcha, 'init_hooks' ] ) );
+		self::assertSame( Main::LOAD_PRIORITY, has_action( 'plugins_loaded', [ $hcaptcha, 'init_hooks' ] ) );
 
-		remove_action( 'plugins_loaded', [ $hcaptcha, 'init_hooks' ], -PHP_INT_MAX );
+		remove_action( 'plugins_loaded', [ $hcaptcha, 'init_hooks' ], Main::LOAD_PRIORITY );
 
 		self::assertFalse( has_action( 'plugins_loaded', [ $hcaptcha, 'init_hooks' ] ) );
 
 		$hcaptcha->init();
 
-		self::assertSame( - PHP_INT_MAX, has_action( 'plugins_loaded', [ $hcaptcha, 'init_hooks' ] ) );
+		self::assertSame( Main::LOAD_PRIORITY, has_action( 'plugins_loaded', [ $hcaptcha, 'init_hooks' ] ) );
+	}
+
+	/**
+	 * Test init() on cron request.
+	 *
+	 * @return void
+	 * @throws ReflectionException ReflectionException.
+	 */
+	public function test_init_on_cron(): void {
+		$hcaptcha = hcaptcha();
+
+		// The plugin was loaded by codeception.
+		self::assertSame( Main::LOAD_PRIORITY, has_action( 'plugins_loaded', [ $hcaptcha, 'init_hooks' ] ) );
+
+		remove_action( 'plugins_loaded', [ $hcaptcha, 'init_hooks' ], Main::LOAD_PRIORITY );
+
+		self::assertFalse( has_action( 'plugins_loaded', [ $hcaptcha, 'init_hooks' ] ) );
+
+		add_filter( 'wp_doing_cron', '__return_true' );
+
+		$hcaptcha->init();
+
+		$migrations = $this->get_protected_property( $hcaptcha, 'migrations' );
+
+		self::assertFalse( has_action( 'plugins_loaded', [ $hcaptcha, 'init_hooks' ] ) );
+		self::assertSame( Migrations::LOAD_PRIORITY, has_action( 'plugins_loaded', [ $migrations, 'migrate' ] ) );
 	}
 
 	/**
@@ -166,9 +193,9 @@ static function () use ( $whitelisted ) {
 			)
 		);
 
-		self::assertSame( - PHP_INT_MAX, has_action( 'plugins_loaded', [ $hcaptcha, 'init_hooks' ] ) );
+		self::assertSame( Main::LOAD_PRIORITY, has_action( 'plugins_loaded', [ $hcaptcha, 'init_hooks' ] ) );
 
-		self::assertSame( - PHP_INT_MAX + 1, has_action( 'plugins_loaded', [ $hcaptcha, 'load_modules' ] ) );
+		self::assertSame( Main::LOAD_PRIORITY + 1, has_action( 'plugins_loaded', [ $hcaptcha, 'load_modules' ] ) );
 
 		self::assertSame( 10, has_filter( 'wp_resource_hints', [ $hcaptcha, 'prefetch_hcaptcha_dns' ] ) );
 		self::assertSame( 10, has_filter( 'wp_headers', [ $hcaptcha, 'csp_headers' ] ) );
@@ -188,11 +215,11 @@ static function () use ( $whitelisted ) {
 		$subject->init_hooks();
 
 		self::assertSame(
-			- PHP_INT_MAX + 1,
+			Main::LOAD_PRIORITY + 1,
 			has_action( 'plugins_loaded', [ $subject, 'load_modules' ] )
 		);
 		self::assertSame(
-			- PHP_INT_MAX,
+			-PHP_INT_MAX,
 			has_filter(
 				'hcap_whitelist_ip',
 				[ $subject, 'whitelist_ip' ]
@@ -285,11 +312,11 @@ static function () {
 		$subject->init_hooks();
 
 		self::assertSame(
-			- PHP_INT_MAX + 1,
+			Main::LOAD_PRIORITY + 1,
 			has_action( 'plugins_loaded', [ $subject, 'load_modules' ] )
 		);
 		self::assertSame(
-			- PHP_INT_MAX,
+			-PHP_INT_MAX,
 			has_filter(
 				'hcap_whitelist_ip',
 				[ $subject, 'whitelist_ip' ]
@@ -341,8 +368,12 @@ public function dp_test_init_and_init_hooks_on_elementor_pro_edit_page(): array
 			],
 			'request2'             => [
 				'on',
-				[ 'REQUEST_URI' => '/elementor?elementor-preview=23' ],
-				[ 'elementor-preview' => 23 ],
+				[],
+				[
+					'preview_id'    => 23,
+					'preview_nonce' => 'some',
+					'preview'       => true,
+				],
 				[],
 				true,
 			],
@@ -613,7 +644,8 @@ public function dp_test_print_inline_styles(): array {
 	/**
 	 * Test login_head().
 	 *
-	 * @noinspection CssUnusedSymbol*/
+	 * @noinspection CssUnusedSymbol
+	 */
 	public function test_login_head(): void {
 		FunctionMocker::replace(
 			'defined',
@@ -930,7 +962,7 @@ function delayedLoad() {
 				'custom_themes'        => $custom_themes ? [ $custom_themes ] : [],
 				// phpcs:ignore WordPress.WP.AlternativeFunctions.json_encode_json_encode
 				'config_params'        => json_encode( $config_params ),
-				'delay'                => - 100,
+				'delay'                => -100,
 				'license'              => 'pro',
 			]
 		);
@@ -1302,11 +1334,38 @@ public function dp_test_load_modules(): array {
 				'back-in-stock-notifier-for-woocommerce/cwginstocknotifier.php',
 				\HCaptcha\BackInStockNotifier\Form::class,
 			],
+			'bbPress Login Form'                => [
+				[ 'bbp_status', 'login' ],
+				'bbpress/bbpress.php',
+				[
+					\HCaptcha\BBPress\Login::class,
+					\HCaptcha\BBPress\LostPassword::class,
+					\HCaptcha\BBPress\Register::class,
+				],
+			],
+			'bbPress Lost Password Form'        => [
+				[ 'bbp_status', 'lost_pass' ],
+				'bbpress/bbpress.php',
+				[
+					\HCaptcha\BBPress\Login::class,
+					\HCaptcha\BBPress\LostPassword::class,
+					\HCaptcha\BBPress\Register::class,
+				],
+			],
 			'bbPress New Topic'                 => [
 				[ 'bbp_status', 'new_topic' ],
 				'bbpress/bbpress.php',
 				NewTopic::class,
 			],
+			'bbPress Register Form'             => [
+				[ 'bbp_status', 'register' ],
+				'bbpress/bbpress.php',
+				[
+					\HCaptcha\BBPress\Login::class,
+					\HCaptcha\BBPress\LostPassword::class,
+					\HCaptcha\BBPress\Register::class,
+				],
+			],
 			'bbPress Reply'                     => [
 				[ 'bbp_status', 'reply' ],
 				'bbpress/bbpress.php',
@@ -1696,7 +1755,7 @@ static function ( $override, $domain, $mofile ) use ( &$override_filter_params )
 
 				return $override;
 			},
-			- PHP_INT_MAX,
+			Main::LOAD_PRIORITY,
 			3
 		);
 

.tests/php/integration/BBPress/LoginTest.php

@@ -0,0 +1,104 @@
+<?php
+/**
+ * LoginTest class file.
+ *
+ * @package HCaptcha\Tests
+ */
+
+namespace HCaptcha\Tests\Integration\BBPress;
+
+use HCaptcha\BBPress\Login;
+use HCaptcha\Helpers\HCaptcha;
+use HCaptcha\Tests\Integration\HCaptchaPluginWPTestCase;
+
+/**
+ * Test Login class.
+ *
+ * @group bbpress
+ * @group bbpress-login
+ */
+class LoginTest extends HCaptchaPluginWPTestCase {
+
+	/**
+	 * Test init_hooks().
+	 *
+	 * @return void
+	 */
+	public function test_init_hooks(): void {
+		$subject = new Login();
+
+		self::assertSame( 10, has_filter( 'do_shortcode_tag', [ $subject, 'do_shortcode_tag' ] ) );
+	}
+
+	/**
+	 * Test do_shortcode_tag().
+	 *
+	 * @return void
+	 * @noinspection PhpUnusedParameterInspection
+	 */
+	public function test_do_shortcode_tag(): void {
+		$tag  = 'bbp-login';
+		$attr = [];
+		$m    = [];
+
+		$placeholder = '===hcaptcha placeholder===';
+		$template    = <<<HTML
+<form action="https://test.test/wp-login.php">
+		$placeholder<button type="submit"/>
+</form>
+HTML;
+
+		$args      = [
+			'action' => 'hcaptcha_login',
+			'name'   => 'hcaptcha_login_nonce',
+			'id'     => [
+				'source'  => [ 'bbpress/bbpress.php' ],
+				'form_id' => 'login',
+			],
+		];
+		$hcaptcha  = $this->get_hcap_form( $args );
+		$signature = HCaptcha::get_signature( Login::class, 'login', true );
+
+		$output   = str_replace( $placeholder, '', $template );
+		$expected = str_replace( $placeholder, $hcaptcha . $signature . "\n", $template );
+
+		$subject = new Login();
+
+		// Wrong tag.
+		self::assertSame( $output, $subject->do_shortcode_tag( $output, 'some', $attr, $m ) );
+
+		// Logged in.
+		wp_set_current_user( 1 );
+
+		self::assertSame( $output, $subject->do_shortcode_tag( $output, $tag, $attr, $m ) );
+
+		// Login limit not exceeded.
+		wp_set_current_user( 0 );
+		add_filter(
+			'hcap_login_limit_exceeded',
+			static function ( $value ) use ( &$limit_exceeded ) {
+				return $limit_exceeded;
+			}
+		);
+
+		$limit_exceeded = false;
+
+		self::assertSame( $output, $subject->do_shortcode_tag( $output, $tag, $attr, $m ) );
+
+		// Status is 'login'.
+		$limit_exceeded = true;
+
+		hcaptcha()->settings()->set( 'bbp_status', 'login' );
+
+		self::assertSame( $expected, $subject->do_shortcode_tag( $output, $tag, $attr, $m ) );
+		self::assertSame( 1, did_action( 'hcap_signature' ) );
+
+		// Status is not 'login'.
+		hcaptcha()->settings()->set( 'bbp_status', 'some' );
+
+		$expected = str_replace( $placeholder, $signature . "\n", $template );
+
+		self::assertSame( $expected, $subject->do_shortcode_tag( $output, $tag, $attr, $m ) );
+		self::assertSame( 2, did_action( 'hcap_signature' ) );
+	}
+}