Fixing failing tests due to new duplicity checks in Mautic

tests/Api/CompaniesTest.php

@@ -84,6 +84,15 @@ public function testAddAndRemove()
 
     public function testBatchEndpoints()
     {
-        $this->standardTestBatchEndpoints();
+        $batch = [];
+
+        // Company name must be unique
+        for ($i = 0; $i < 3; ++$i) {
+            $company = $this->testPayload;
+            $company['companyname'] = 'test'.$i;
+            $batch[]                = $company;
+        }
+
+        $this->standardTestBatchEndpoints($batch);
     }
 }

tests/Api/ContactsTest.php

@@ -11,6 +11,7 @@
 namespace Mautic\Tests\Api;
 
 use Mautic\Api\Contacts;
+use Mautic\QueryBuilder\QueryBuilder;
 
 class ContactsTest extends AbstractCustomFieldsTest
 {
@@ -365,6 +366,17 @@ public function testEditPatchFormError()
 
     public function testEditPut()
     {
+        $qb = new QueryBuilder();
+        $qb->addWhere($qb->getWhereBuilder()->eq('email', $this->testPayload['email']));
+        $response = $this->api->getCustomList($qb, 0, 1);
+        $this->assertErrors($response);
+
+        // Making sure that if the contact exists, it won't try to create it. Otherwise we'll get an error about duplicated email.
+        if (isset($response[$this->api->listName()]) && count($response[$this->api->listName()])) {
+            $response = $this->api->delete(array_pop($response[$this->api->listName()])['id']);
+            $this->assertErrors($response);
+        }
+
         $this->standardTestEditPut();
     }
 

tests/Api/MauticApiTestCase.php

@@ -320,7 +320,7 @@ public function standardTestEditPut()
         $response = $this->api->edit(10000, $this->testPayload, true);
         $this->assertPayload($response);
 
-        //now delete the category
+        //now delete the entity
         $response = $this->api->delete($response[$this->api->itemName()]['id']);
         $this->assertErrors($response);
     }