Fix. Integration. EDD integration fixed. (#770)

* Fix. Integration. EDD integration fixed.

* Upd. Code. Unit-test for an integration added: `TestEasyDigitalDownloads`.

Author: Glomberg

inc/cleantalk-integrations-by-hook.php

@@ -135,7 +135,7 @@
         'ajax'    => false
     ),
     'EasyDigitalDownloads' => array(
-        'hook'    => array('edd_pre_process_register_form', 'edd_insert_user_args'),
+        'hook'    => array('edd_pre_process_register_form', 'edd_insert_user_args', 'edd_customer_pre_create'),
         'setting' => 'forms__registrations_test',
         'ajax'    => false
     ),

lib/Cleantalk/Antispam/Integrations/EasyDigitalDownloads.php

@@ -13,8 +13,9 @@ public function getDataForChecking($argument)
         $this->user_data = $argument;
 
         if (
-            Post::get('edd_action') === "user_register" ||
-            !empty($argument['user_email'])
+            Post::getString('edd_action') === "user_register" ||
+            !empty($argument['user_email']) ||
+            Post::getString('edd-process-checkout-nonce')
         ) {
             /**
              * Filter for POST

tests/Antispam/IntegrationsByHook/TestEasyDigitalDownloads.php

@@ -0,0 +1,70 @@
+<?php
+
+namespace Cleantalk\Antispam\Integrations;
+
+use Cleantalk\ApbctWP\Variables\Get;
+use Cleantalk\ApbctWP\Variables\Post;
+use PHPUnit\Framework\TestCase;
+
+class TestEasyDigitalDownloads extends TestCase
+{
+    private $integration;
+
+    protected function setUp(): void
+    {
+        parent::setUp();
+        $this->integration = new EasyDigitalDownloads();
+    }
+
+    protected function tearDown(): void
+    {
+        // Clean up global state
+        $_POST = [];
+        $_GET = [];
+        Post::getInstance()->variables = [];
+        Get::getInstance()->variables = [];
+        parent::tearDown();
+    }
+
+    public function testGetDataForCheckingRegisterPage()
+    {
+        $_POST['edd_action'] = 'user_register';
+        $_POST['edd_user_login'] = 'John';
+        $_POST['edd_user_email'] = 'john.doe@example.com';
+
+        $result = $this->integration->getDataForChecking(null);
+
+        $this->assertIsArray($result);
+        $this->assertEquals('john.doe@example.com', $result['email']);
+        $this->assertEquals('', $result['nickname']); // NickName not detected at that integration
+        $this->assertTrue($result['register']);
+    }
+
+    public function testGetDataForCheckingRegisterDuringCheckout()
+    {
+        $_POST['edd-process-checkout-nonce'] = 'user_register';
+        $_POST['edd_first'] = 'John';
+        $_POST['edd_last'] = 'Doe';
+        $_POST['edd_email'] = 'john.doe@example.com';
+
+        $result = $this->integration->getDataForChecking(null);
+
+        $this->assertIsArray($result);
+        $this->assertEquals('john.doe@example.com', $result['email']);
+        $this->assertEquals('', $result['nickname']); // NickName not detected at that integration
+        $this->assertTrue($result['register']);
+    }
+
+    public function testGetDataForCheckingRegisterCommon()
+    {
+        $_POST['edd_user_login'] = 'John';
+        $_POST['edd_user_email'] = 'john.doe@example.com';
+
+        $result = $this->integration->getDataForChecking(['user_email' => 'any_email_no_sense']);
+
+        $this->assertIsArray($result);
+        $this->assertEquals('john.doe@example.com', $result['email']);
+        $this->assertEquals('', $result['nickname']); // NickName not detected at that integration
+        $this->assertTrue($result['register']);
+    }
+}