From b1ba60bb3970db4357fc96b30ec406638592053c Mon Sep 17 00:00:00 2001
From: Javier Giorello <giorello@ebizmarts.com>
Date: Fri, 22 May 2020 08:40:45 -0300
Subject: [PATCH 1/9] Added exception to prevent order rollback and filter to
 avoid unnecessary DB calls

---
 .../Ebizmarts/MailChimp/Model/Observer.php    | 45 ++++++++++---------
 1 file changed, 25 insertions(+), 20 deletions(-)

diff --git a/app/code/community/Ebizmarts/MailChimp/Model/Observer.php b/app/code/community/Ebizmarts/MailChimp/Model/Observer.php
index 987ce823e..69b206a55 100755
--- a/app/code/community/Ebizmarts/MailChimp/Model/Observer.php
+++ b/app/code/community/Ebizmarts/MailChimp/Model/Observer.php
@@ -440,27 +440,32 @@ public function newOrder(Varien_Event_Observer $observer)
             $this->removeCampaignData();
             $items = $order->getAllItems();
 
-            foreach ($items as $item) {
-                if ($this->isBundleItem($item) || $this->isConfigurableItem($item)) {
-                    continue;
-                }
-
-                $mailchimpStoreId = $helper->getMCStoreId($storeId);
-                $productId = $item->getProductId();
-                $dataProduct = $this->getMailchimpEcommerceSyncDataModel()->getEcommerceSyncDataItem(
-                    $productId,
-                    Ebizmarts_MailChimp_Model_Config::IS_PRODUCT,
-                    $mailchimpStoreId
-                );
-
-                $isMarkedAsDeleted = $dataProduct->getMailchimpSyncDeleted();
-
-                if (!$isMarkedAsDeleted) {
-                    $apiProducts = $this->makeApiProduct();
-                    $apiProducts->setMailchimpStoreId($mailchimpStoreId);
-                    $apiProducts->setMagentoStoreId($storeId);
-                    $apiProducts->update($productId);
+            try {
+                foreach ($items as $item) {
+                    if ($this->isBundleItem($item) || $this->isConfigurableItem($item)) {
+                        continue;
+                    }
+    
+                    $mailchimpStoreId = $helper->getMCStoreId($storeId);
+                    $productId = $item->getProductId();
+                    $dataProduct = $this->getMailchimpEcommerceSyncDataModel()->getEcommerceSyncDataItem(
+                        $productId,
+                        Ebizmarts_MailChimp_Model_Config::IS_PRODUCT,
+                        $mailchimpStoreId
+                    );
+    
+                    $isMarkedAsDeleted = $dataProduct->getMailchimpSyncDeleted();
+                    $isMarkedAsModified = $dataProduct->getMailchimpSyncModified();
+    
+                    if (!$isMarkedAsDeleted && !$isMarkedAsModified) {
+                        $apiProducts = $this->makeApiProduct();
+                        $apiProducts->setMailchimpStoreId($mailchimpStoreId);
+                        $apiProducts->setMagentoStoreId($storeId);
+                        $apiProducts->update($productId);
+                    }
                 }
+            } catch (Exception $e) {
+                $helper->logError($e->getMessage());
             }
         }
 

From a03dd6535112bbd1f43b5e3972c4cfb282e62844 Mon Sep 17 00:00:00 2001
From: Javier Giorello <giorello@ebizmarts.com>
Date: Fri, 22 May 2020 08:41:52 -0300
Subject: [PATCH 2/9] Change parameter to avoid the creation of eCommerce rows
 with wrong data in order creation

---
 .../Ebizmarts/MailChimp/Model/Api/ItemSynchronizer.php          | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/app/code/community/Ebizmarts/MailChimp/Model/Api/ItemSynchronizer.php b/app/code/community/Ebizmarts/MailChimp/Model/Api/ItemSynchronizer.php
index f471b46f2..be4081140 100755
--- a/app/code/community/Ebizmarts/MailChimp/Model/Api/ItemSynchronizer.php
+++ b/app/code/community/Ebizmarts/MailChimp/Model/Api/ItemSynchronizer.php
@@ -177,7 +177,7 @@ protected function addSyncDataToken($id, $token)
 
     protected function markSyncDataAsModified($id)
     {
-        $this->_updateSyncData($id, null, null, 1);
+        $this->_updateSyncData($id, null, null, 1, null, null, null, true);
     }
 
     protected function markSyncDataAsDeleted($id, $syncedFlag = null)

From 1721e41f55ad2c8f81520f026f3162784b30e8c6 Mon Sep 17 00:00:00 2001
From: Javier Giorello <giorello@ebizmarts.com>
Date: Fri, 22 May 2020 10:38:11 -0300
Subject: [PATCH 3/9] Whitespace removed (phpcs).

---
 app/code/community/Ebizmarts/MailChimp/Model/Observer.php | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/app/code/community/Ebizmarts/MailChimp/Model/Observer.php b/app/code/community/Ebizmarts/MailChimp/Model/Observer.php
index 69b206a55..06c43ae04 100755
--- a/app/code/community/Ebizmarts/MailChimp/Model/Observer.php
+++ b/app/code/community/Ebizmarts/MailChimp/Model/Observer.php
@@ -445,7 +445,7 @@ public function newOrder(Varien_Event_Observer $observer)
                     if ($this->isBundleItem($item) || $this->isConfigurableItem($item)) {
                         continue;
                     }
-    
+
                     $mailchimpStoreId = $helper->getMCStoreId($storeId);
                     $productId = $item->getProductId();
                     $dataProduct = $this->getMailchimpEcommerceSyncDataModel()->getEcommerceSyncDataItem(
@@ -453,7 +453,7 @@ public function newOrder(Varien_Event_Observer $observer)
                         Ebizmarts_MailChimp_Model_Config::IS_PRODUCT,
                         $mailchimpStoreId
                     );
-    
+
                     $isMarkedAsDeleted = $dataProduct->getMailchimpSyncDeleted();
                     $isMarkedAsModified = $dataProduct->getMailchimpSyncModified();
     

From 9a41f1e24c0cbad57db3383eea7f6180d58b90c4 Mon Sep 17 00:00:00 2001
From: Javier Giorello <giorello@ebizmarts.com>
Date: Fri, 22 May 2020 10:49:01 -0300
Subject: [PATCH 4/9] Whitespace removed (phpcs).

---
 app/code/community/Ebizmarts/MailChimp/Model/Observer.php | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/app/code/community/Ebizmarts/MailChimp/Model/Observer.php b/app/code/community/Ebizmarts/MailChimp/Model/Observer.php
index 06c43ae04..e29fa957f 100755
--- a/app/code/community/Ebizmarts/MailChimp/Model/Observer.php
+++ b/app/code/community/Ebizmarts/MailChimp/Model/Observer.php
@@ -456,7 +456,7 @@ public function newOrder(Varien_Event_Observer $observer)
 
                     $isMarkedAsDeleted = $dataProduct->getMailchimpSyncDeleted();
                     $isMarkedAsModified = $dataProduct->getMailchimpSyncModified();
-    
+
                     if (!$isMarkedAsDeleted && !$isMarkedAsModified) {
                         $apiProducts = $this->makeApiProduct();
                         $apiProducts->setMailchimpStoreId($mailchimpStoreId);

From e23b9bf2a0f6157c79e6cfdfc0638d582c699477 Mon Sep 17 00:00:00 2001
From: Javier Giorello <giorello@ebizmarts.com>
Date: Mon, 25 May 2020 10:54:47 -0300
Subject: [PATCH 5/9] Tests case added for markAsModified in Observer

---
 .../MailChimp/Model/ObserverTest.php          | 154 +++++++++++++++++-
 1 file changed, 153 insertions(+), 1 deletion(-)

diff --git a/dev/tests/mailchimp/tests/app/Ebizmarts/MailChimp/Model/ObserverTest.php b/dev/tests/mailchimp/tests/app/Ebizmarts/MailChimp/Model/ObserverTest.php
index 29d545f75..7c8e550c6 100755
--- a/dev/tests/mailchimp/tests/app/Ebizmarts/MailChimp/Model/ObserverTest.php
+++ b/dev/tests/mailchimp/tests/app/Ebizmarts/MailChimp/Model/ObserverTest.php
@@ -383,6 +383,7 @@ public function testNewOrder()
         $customerFirstname = 'John';
         $customerLastname = 'Smith';
         $isMarkedAsDeleted = 0;
+        $isMarkedAsDeleted = 0;
         $type = Ebizmarts_MailChimp_Model_Config::IS_PRODUCT;
 
 
@@ -453,7 +454,7 @@ public function testNewOrder()
             ->getMock();
 
         $dataProductMock = $this->getMockBuilder(Ebizmarts_MailChimp_Model_Ecommercesyncdata::class)
-            ->setMethods(array('getMailchimpSyncDeleted', 'getEcommerceSyncDataItem'))
+            ->setMethods(array('getMailchimpSyncDeleted', 'getEcommerceSyncDataItem', 'getMailchimpSyncModified'))
             ->getMock();
 
         $observerMock->expects($this->once())->method('makeHelper')->willReturn($helperMock);
@@ -511,6 +512,157 @@ public function testNewOrder()
         $dataProductMock->expects($this->once())
             ->method('getMailchimpSyncDeleted')
             ->willReturn($isMarkedAsDeleted);
+        
+        $dataProductMock->expects($this->once())
+            ->method('getMailchimpSyncModified')
+            ->willReturn($isMarkedAsDeleted);
+
+        $observerMock->expects($this->once())
+            ->method('getMailchimpEcommerceSyncDataModel')
+            ->willReturn($dataProductMock);
+
+        $dataProductMock->expects($this->once())
+            ->method('getEcommerceSyncDataItem')
+            ->with($productId, $type, $mailchimpStoreId)
+            ->willReturn($dataProductMock);
+
+        $observerMock->newOrder($eventObserverMock);
+    }
+
+    public function testNewOrderNotModified()
+    {
+        $storeId = 1;
+        $post = 1;
+        $productId = 1;
+        $mailchimpStoreId = 'a1s2d3f4g5h6j7k8l9n0';
+        $customerEmail = 'email@example.com';
+        $customerFirstname = 'John';
+        $customerLastname = 'Smith';
+        $isMarkedAsDeleted = 0;
+        $isMarkedAsDeleted = 1;
+        $type = Ebizmarts_MailChimp_Model_Config::IS_PRODUCT;
+
+
+        $itemMock = $this->getMockBuilder(Mage_Sales_Model_Order_Item::class)
+            ->disableOriginalConstructor()
+            ->setMethods(array('getProductType', 'getProductId'))
+            ->getMock();
+
+        $orderMock = $this->getMockBuilder(Mage_Sales_Model_Order::class)
+            ->disableOriginalConstructor()
+            ->setMethods(
+                array(
+                    'getStoreId', 'getCustomerEmail', 'getCustomerFirstname',
+                    'getCustomerLastname', 'getAllItems'
+                )
+            )
+            ->getMock();
+
+        $eventObserverMock = $this->getMockBuilder(Varien_Event_Observer::class)
+            ->disableOriginalConstructor()
+            ->setMethods(array('getEvent'))
+            ->getMock();
+
+        $eventMock = $this->getMockBuilder(Varien_Event::class)
+            ->disableOriginalConstructor()
+            ->setMethods(array('getOrder'))
+            ->getMock();
+
+        $observerMock = $this->getMockBuilder(Ebizmarts_MailChimp_Model_Observer::class)
+            ->disableOriginalConstructor()
+            ->setMethods(
+                array(
+                    'makeHelper', 'removeCampaignData', 'getMailchimpEcommerceSyncDataModel', 'isBundleItem',
+                    'isConfigurableItem'
+                )
+            )
+            ->getMock();
+
+        $helperMock = $this->getMockBuilder(Ebizmarts_MailChimp_Helper_Data::class)
+            ->disableOriginalConstructor()
+            ->setMethods(
+                array(
+                    'getMageApp', 'isEcomSyncDataEnabled',
+                    'isSubscriptionEnabled', 'loadListSubscriber',
+                    'saveEcommerceSyncData', 'getMCStoreId'
+                )
+            )
+            ->getMock();
+
+        $mageAppMock = $this->getMockBuilder(Mage_Core_Model_App::class)
+            ->disableOriginalConstructor()
+            ->setMethods(array('getRequest'))
+            ->getMock();
+
+        $requestMock = $this->getMockBuilder(Mage_Core_Controller_Request_Http::class)
+            ->disableOriginalConstructor()
+            ->setMethods(array('getPost'))
+            ->getMock();
+
+        $subscriberMock = $this->getMockBuilder(Mage_Newsletter_Model_Subscriber::class)
+            ->disableOriginalConstructor()
+            ->setMethods(array('getCustomerId', 'setSubscriberFirstname', 'setSubscriberLastname', 'subscribe'))
+            ->getMock();
+
+        $dataProductMock = $this->getMockBuilder(Ebizmarts_MailChimp_Model_Ecommercesyncdata::class)
+            ->setMethods(array('getMailchimpSyncDeleted', 'getEcommerceSyncDataItem', 'getMailchimpSyncModified'))
+            ->getMock();
+
+        $observerMock->expects($this->once())->method('makeHelper')->willReturn($helperMock);
+
+        $helperMock->expects($this->once())->method('getMageApp')->willReturn($mageAppMock);
+        $mageAppMock->expects($this->once())->method('getRequest')->willReturn($requestMock);
+
+        $requestMock->expects($this->once())->method('getPost')->with('mailchimp_subscribe')->willReturn($post);
+
+        $eventObserverMock->expects($this->once())->method('getEvent')->willReturn($eventMock);
+
+        $eventMock->expects($this->once())->method('getOrder')->willReturn($orderMock);
+
+        $orderMock->expects($this->once())->method('getStoreId')->willReturn($storeId);
+
+        $helperMock->expects($this->once())->method('isEcomSyncDataEnabled')->with($storeId)->willReturn(true);
+
+        $helperMock->expects($this->once())->method('isSubscriptionEnabled')->with($storeId)->willReturn(true);
+
+        $orderMock->expects($this->once())->method('getCustomerEmail')->willReturn($customerEmail);
+
+        $helperMock
+            ->expects($this->once())
+            ->method('loadListSubscriber')
+            ->with($post, $customerEmail)
+            ->willReturn($subscriberMock);
+
+        $subscriberMock->expects($this->once())->method('getCustomerId')->willReturn(false);
+
+        $orderMock->expects($this->once())->method('getCustomerFirstname')->willReturn($customerFirstname);
+
+        $subscriberMock->expects($this->once())->method('setSubscriberFirstname')->with($customerFirstname);
+
+        $orderMock->expects($this->once())->method('getCustomerLastname')->willReturn($customerLastname);
+
+        $subscriberMock->expects($this->once())->method('setSubscriberLastname')->with($customerLastname);
+
+        $subscriberMock->expects($this->once())->method('subscribe')->with($customerEmail);
+
+        $observerMock->expects($this->once())->method('removeCampaignData');
+
+        $orderMock->expects($this->once())->method('getAllItems')->willReturn(array($itemMock));
+
+        $observerMock->expects($this->once())->method('isBundleItem')->with($itemMock)->willReturn(false);
+        $observerMock->expects($this->once())->method('isConfigurableItem')->with($itemMock)->willReturn(false);
+
+        $itemMock->expects($this->once())->method('getProductId')->willReturn($productId);
+
+        $helperMock->expects($this->once())->method('getMCStoreId')->with($storeId)->willReturn($mailchimpStoreId);
+
+        $dataProductMock->expects($this->once())
+            ->method('getMailchimpSyncDeleted')
+            ->willReturn($isMarkedAsDeleted);
+        
+        $dataProductMock->expects($this->once())
+            ->method('getMailchimpSyncModified')
+            ->willReturn($isMarkedAsDeleted);
 
         $observerMock->expects($this->once())
             ->method('getMailchimpEcommerceSyncDataModel')

From 8074a77d30542b5b49345dc0659a925a2f937218 Mon Sep 17 00:00:00 2001
From: Javier Giorello <giorello@ebizmarts.com>
Date: Tue, 26 May 2020 09:04:33 -0300
Subject: [PATCH 6/9] Tests improved

---
 .../MailChimp/Model/ObserverTest.php          | 739 +++++-------------
 1 file changed, 196 insertions(+), 543 deletions(-)

diff --git a/dev/tests/mailchimp/tests/app/Ebizmarts/MailChimp/Model/ObserverTest.php b/dev/tests/mailchimp/tests/app/Ebizmarts/MailChimp/Model/ObserverTest.php
index 7c8e550c6..564ab8474 100755
--- a/dev/tests/mailchimp/tests/app/Ebizmarts/MailChimp/Model/ObserverTest.php
+++ b/dev/tests/mailchimp/tests/app/Ebizmarts/MailChimp/Model/ObserverTest.php
@@ -21,29 +21,18 @@ public function testProductAttributeUpdateIsUsingCorrectStoreId()
         /**
          * @var \Ebizmarts_MailChimp_Model_Observer $modelMock
          */
-        $modelMock = $this->getMockBuilder(Ebizmarts_MailChimp_Model_Observer::class)
-            ->disableOriginalConstructor()
-            ->setMethods(array('makeHelper', 'makeApiProduct', 'getMailchimpEcommerceSyncDataModel'))
-            ->getMock();
+        $modelMock = $this->getMailchimpObserverMock();
 
         $apiProductsMock = $this->getMockBuilder(Ebizmarts_MailChimp_Model_Api_Products::class)
             ->disableOriginalConstructor()
             ->setMethods(array('update'))
             ->getMock();
 
-        $helperMock = $this->getMockBuilder(Ebizmarts_MailChimp_Helper_Data::class)
-            ->disableOriginalConstructor()
-            ->setMethods(array('getAllMailChimpStoreIds', 'isEcommerceEnabled'))
-            ->getMock();
+        $helperMock = $this->getHelperMock();
 
-        $eventMock = $this->getMockBuilder(Varien_Event::class)
-            ->disableOriginalConstructor()
-            ->setMethods(array('getProductIds'))
-            ->getMock();
+        $eventMock = $this->getEventObserverMock();
 
-        $dataProductMock = $this->getMockBuilder(Ebizmarts_MailChimp_Model_Ecommercesyncdata::class)
-            ->setMethods(array('getMailchimpSyncDeleted', 'getEcommerceSyncDataItem'))
-            ->getMock();
+        $dataProductMock = $this->getEcommerceModelMock();
 
         $eventMock->expects($this->once())->method('getProductIds')->willReturn(array(12, 34));
 
@@ -85,26 +74,17 @@ public function testSaveCampaignDataCallsCorrectFunctions()
         /**
          * @var \Ebizmarts_MailChimp_Model_Observer $modelMock
          */
-        $modelMock = $this->getMockBuilder(Ebizmarts_MailChimp_Model_Observer::class)
-            ->disableOriginalConstructor()
-            ->setMethods(array("_getCampaignCookie", "_getLandingCookie"))
-            ->getMock();
+        $modelMock = $this->getMailchimpObserverMock();
 
         $modelMock->expects($this->once())->method("_getCampaignCookie")->willReturn("abcd123");
         $modelMock->expects($this->once())->method("_getLandingCookie")->willReturn("abcd");
 
-        $orderMock = $this->getMockBuilder(Mage_Sales_Model_Order::class)
-            ->setMethods(array("setMailchimpCampaignId", "getMailchimpLandingPage", "setMailchimpLandingPage"))
-            ->disableOriginalConstructor()
-            ->getMock();
+        $orderMock = $this->getOrderMock();
         $orderMock->expects($this->once())->method("setMailchimpCampaignId")->with("abcd123");
         $orderMock->expects($this->once())->method("getMailchimpLandingPage")->willReturn(null);
         $orderMock->expects($this->once())->method("setMailchimpLandingPage")->with("abcd");
 
-        $eventMock = $this->getMockBuilder(Varien_Event::class)
-            ->disableOriginalConstructor()
-            ->setMethods(array("getOrder"))
-            ->getMock();
+        $eventMock = $this->getEventObserverMock();
         $eventMock->expects($this->once())->method("getOrder")->willReturn($orderMock);
 
         $eventObserverMock = $this->makeEventObserverMock($eventMock, 1);
@@ -119,10 +99,7 @@ public function testSaveCampaignDataCallsCorrectFunctions()
      */
     public function makeEventObserverMock($eventMock, $callCount)
     {
-        $eventObserverMock = $this->getMockBuilder(Varien_Event_Observer::class)
-            ->disableOriginalConstructor()
-            ->setMethods(array('getEvent'))
-            ->getMock();
+        $eventObserverMock = $this->getObserverMock();
 
         $eventObserverMock->expects($this->exactly($callCount))->method('getEvent')->willReturn($eventMock);
 
@@ -133,30 +110,15 @@ public function testHandleSubscriberDeletion()
     {
         $storeId = 1;
 
-        $eventObserverMock = $this->getMockBuilder(Varien_Event_Observer::class)
-            ->disableOriginalConstructor()
-            ->setMethods(array('getEvent'))
-            ->getMock();
+        $eventObserverMock = $this->getObserverMock();
 
-        $eventMock = $this->getMockBuilder(Varien_Event::class)
-            ->disableOriginalConstructor()
-            ->setMethods(array('getSubscriber'))
-            ->getMock();
+        $eventMock = $this->getEventObserverMock();
 
-        $observerMock = $this->getMockBuilder(Ebizmarts_MailChimp_Model_Observer::class)
-            ->disableOriginalConstructor()
-            ->setMethods(array('makeHelper', 'makeApiSubscriber'))
-            ->getMock();
+        $observerMock = $this->getMailchimpObserverMock();
 
-        $helperMock = $this->getMockBuilder(Ebizmarts_MailChimp_Helper_Data::class)
-            ->disableOriginalConstructor()
-            ->setMethods(array('isSubscriptionEnabled'))
-            ->getMock();
+        $helperMock = $this->getHelperMock();
 
-        $subscriberMock = $this->getMockBuilder(Mage_Newsletter_Model_Subscriber::class)
-            ->disableOriginalConstructor()
-            ->setMethods(array('getStoreId'))
-            ->getMock();
+        $subscriberMock = $this->getSubscriberMock();
 
         $apiSubscriberMock = $this->getMockBuilder(Ebizmarts_MailChimp_Model_Api_Subscribers::class)
             ->disableOriginalConstructor()
@@ -196,20 +158,11 @@ public function testCustomerSaveAfter()
             ->setMethods(array('getId', 'getOrigData', 'getEmail', 'getStoreId', 'getMailchimpStoreView'))
             ->getMock();
 
-        $eventObserverMock = $this->getMockBuilder(Varien_Event_Observer::class)
-            ->disableOriginalConstructor()
-            ->setMethods(array('getEvent'))
-            ->getMock();
+        $eventObserverMock = $this->getObserverMock();
 
-        $eventMock = $this->getMockBuilder(Varien_Event::class)
-            ->disableOriginalConstructor()
-            ->setMethods(array('getCustomer'))
-            ->getMock();
+        $eventMock = $this->getEventObserverMock();
 
-        $requestMock = $this->getMockBuilder(Mage_Core_Controller_Request_Http::class)
-            ->disableOriginalConstructor()
-            ->setMethods(array('getParams'))
-            ->getMock();
+        $requestMock = $this->getRequestMock();
 
         $observerMock = $this->getMockBuilder(Ebizmarts_MailChimp_Model_Observer::class)
             ->disableOriginalConstructor()
@@ -221,25 +174,16 @@ public function testCustomerSaveAfter()
             )
             ->getMock();
 
-        $helperMock = $this->getMockBuilder(Ebizmarts_MailChimp_Helper_Data::class)
-            ->disableOriginalConstructor()
-            ->setMethods(array('isSubscriptionEnabled', 'isEcomSyncDataEnabled', 'getMCStoreId'))
-            ->getMock();
+        $helperMock = $this->getHelperMock();
 
         $apiSubscriberMock = $this->getMockBuilder(Ebizmarts_MailChimp_Model_Api_Subscribers::class)
             ->disableOriginalConstructor()
             ->setMethods(array('deleteSubscriber', 'updateSubscriber', 'update'))
             ->getMock();
 
-        $subscriberMock = $this->getMockBuilder(Mage_Newsletter_Model_Subscriber::class)
-            ->disableOriginalConstructor()
-            ->setMethods(array('getId'))
-            ->getMock();
+        $subscriberMock = $this->getSubscriberMock();
 
-        $subscriberMockTwo = $this->getMockBuilder(Mage_Newsletter_Model_Subscriber::class)
-            ->disableOriginalConstructor()
-            ->setMethods(array('loadByCustomer', 'setSubscriberEmail', 'save'))
-            ->getMock();
+        $subscriberMockTwo = $this->getSubscriberMock();
 
         $apiCustomerMock = $this->getMockBuilder(Ebizmarts_MailChimp_Model_Api_Customers::class)
             ->disableOriginalConstructor()
@@ -303,25 +247,13 @@ public function testCustomerAddressSaveBefore()
         $customerId = 1;
         $customerEmail = 'customer@email.com';
 
-        $eventObserverMock = $this->getMockBuilder(Varien_Event_Observer::class)
-            ->disableOriginalConstructor()
-            ->setMethods(array('getEvent'))
-            ->getMock();
+        $eventObserverMock = $this->getObserverMock();
 
-        $eventMock = $this->getMockBuilder(Varien_Event::class)
-            ->disableOriginalConstructor()
-            ->setMethods(array('getCustomerAddress'))
-            ->getMock();
+        $eventMock = $this->getEventObserverMock();
 
-        $observerMock = $this->getMockBuilder(Ebizmarts_MailChimp_Model_Observer::class)
-            ->disableOriginalConstructor()
-            ->setMethods(array('makeHelper', 'getCustomerModel', 'makeApiSubscriber', 'makeApiCustomer'))
-            ->getMock();
+        $observerMock = $this->getMailchimpObserverMock();
 
-        $helperMock = $this->getMockBuilder(Ebizmarts_MailChimp_Helper_Data::class)
-            ->disableOriginalConstructor()
-            ->setMethods(array('isSubscriptionEnabled', 'isEcomSyncDataEnabled'))
-            ->getMock();
+        $helperMock = $this->getHelperMock();
 
         $customerMock = $this->getMockBuilder(Mage_Customer_Model_Customer::class)
             ->disableOriginalConstructor()
@@ -386,76 +318,22 @@ public function testNewOrder()
         $isMarkedAsDeleted = 0;
         $type = Ebizmarts_MailChimp_Model_Config::IS_PRODUCT;
 
-
-        $itemMock = $this->getMockBuilder(Mage_Sales_Model_Order_Item::class)
-            ->disableOriginalConstructor()
-            ->setMethods(array('getProductType', 'getProductId'))
-            ->getMock();
-
-        $orderMock = $this->getMockBuilder(Mage_Sales_Model_Order::class)
-            ->disableOriginalConstructor()
-            ->setMethods(
-                array(
-                    'getStoreId', 'getCustomerEmail', 'getCustomerFirstname',
-                    'getCustomerLastname', 'getAllItems'
-                )
-            )
-            ->getMock();
-
-        $eventObserverMock = $this->getMockBuilder(Varien_Event_Observer::class)
-            ->disableOriginalConstructor()
-            ->setMethods(array('getEvent'))
-            ->getMock();
-
-        $eventMock = $this->getMockBuilder(Varien_Event::class)
-            ->disableOriginalConstructor()
-            ->setMethods(array('getOrder'))
-            ->getMock();
-
-        $observerMock = $this->getMockBuilder(Ebizmarts_MailChimp_Model_Observer::class)
-            ->disableOriginalConstructor()
-            ->setMethods(
-                array(
-                    'makeHelper', 'removeCampaignData', 'getMailchimpEcommerceSyncDataModel', 'isBundleItem',
-                    'isConfigurableItem', 'makeApiProduct'
-                )
-            )
-            ->getMock();
-
-        $helperMock = $this->getMockBuilder(Ebizmarts_MailChimp_Helper_Data::class)
-            ->disableOriginalConstructor()
-            ->setMethods(
-                array(
-                    'getMageApp', 'isEcomSyncDataEnabled',
-                    'isSubscriptionEnabled', 'loadListSubscriber',
-                    'saveEcommerceSyncData', 'getMCStoreId'
-                )
-            )
-            ->getMock();
-
-        $mageAppMock = $this->getMockBuilder(Mage_Core_Model_App::class)
-            ->disableOriginalConstructor()
-            ->setMethods(array('getRequest'))
-            ->getMock();
-
-        $requestMock = $this->getMockBuilder(Mage_Core_Controller_Request_Http::class)
-            ->disableOriginalConstructor()
-            ->setMethods(array('getPost'))
-            ->getMock();
-
-        $subscriberMock = $this->getMockBuilder(Mage_Newsletter_Model_Subscriber::class)
-            ->disableOriginalConstructor()
-            ->setMethods(array('getCustomerId', 'setSubscriberFirstname', 'setSubscriberLastname', 'subscribe'))
-            ->getMock();
+        $itemMock = $this->getOrderItemMock();
+        $orderMock = $this->getOrderMock();
+        $eventObserverMock = $this->getObserverMock();
+        $eventMock = $this->getEventObserverMock();
+        $observerMock = $this->getMailchimpObserverMock();
+        $helperMock = $this->getHelperMock();
+        $mageAppMock = $this->getMageAppMock();
+        $requestMock = $this->getRequestMock();
+        $subscriberMock = $this->getSubscriberMock();
 
         $apiProductsMock = $this->getMockBuilder(Ebizmarts_MailChimp_Model_Api_Products::class)
             ->disableOriginalConstructor()
             ->setMethods(array('update'))
             ->getMock();
 
-        $dataProductMock = $this->getMockBuilder(Ebizmarts_MailChimp_Model_Ecommercesyncdata::class)
-            ->setMethods(array('getMailchimpSyncDeleted', 'getEcommerceSyncDataItem', 'getMailchimpSyncModified'))
-            ->getMock();
+        $dataProductMock = $this->getEcommerceModelMock();
 
         $observerMock->expects($this->once())->method('makeHelper')->willReturn($helperMock);
 
@@ -542,71 +420,16 @@ public function testNewOrderNotModified()
         $isMarkedAsDeleted = 1;
         $type = Ebizmarts_MailChimp_Model_Config::IS_PRODUCT;
 
-
-        $itemMock = $this->getMockBuilder(Mage_Sales_Model_Order_Item::class)
-            ->disableOriginalConstructor()
-            ->setMethods(array('getProductType', 'getProductId'))
-            ->getMock();
-
-        $orderMock = $this->getMockBuilder(Mage_Sales_Model_Order::class)
-            ->disableOriginalConstructor()
-            ->setMethods(
-                array(
-                    'getStoreId', 'getCustomerEmail', 'getCustomerFirstname',
-                    'getCustomerLastname', 'getAllItems'
-                )
-            )
-            ->getMock();
-
-        $eventObserverMock = $this->getMockBuilder(Varien_Event_Observer::class)
-            ->disableOriginalConstructor()
-            ->setMethods(array('getEvent'))
-            ->getMock();
-
-        $eventMock = $this->getMockBuilder(Varien_Event::class)
-            ->disableOriginalConstructor()
-            ->setMethods(array('getOrder'))
-            ->getMock();
-
-        $observerMock = $this->getMockBuilder(Ebizmarts_MailChimp_Model_Observer::class)
-            ->disableOriginalConstructor()
-            ->setMethods(
-                array(
-                    'makeHelper', 'removeCampaignData', 'getMailchimpEcommerceSyncDataModel', 'isBundleItem',
-                    'isConfigurableItem'
-                )
-            )
-            ->getMock();
-
-        $helperMock = $this->getMockBuilder(Ebizmarts_MailChimp_Helper_Data::class)
-            ->disableOriginalConstructor()
-            ->setMethods(
-                array(
-                    'getMageApp', 'isEcomSyncDataEnabled',
-                    'isSubscriptionEnabled', 'loadListSubscriber',
-                    'saveEcommerceSyncData', 'getMCStoreId'
-                )
-            )
-            ->getMock();
-
-        $mageAppMock = $this->getMockBuilder(Mage_Core_Model_App::class)
-            ->disableOriginalConstructor()
-            ->setMethods(array('getRequest'))
-            ->getMock();
-
-        $requestMock = $this->getMockBuilder(Mage_Core_Controller_Request_Http::class)
-            ->disableOriginalConstructor()
-            ->setMethods(array('getPost'))
-            ->getMock();
-
-        $subscriberMock = $this->getMockBuilder(Mage_Newsletter_Model_Subscriber::class)
-            ->disableOriginalConstructor()
-            ->setMethods(array('getCustomerId', 'setSubscriberFirstname', 'setSubscriberLastname', 'subscribe'))
-            ->getMock();
-
-        $dataProductMock = $this->getMockBuilder(Ebizmarts_MailChimp_Model_Ecommercesyncdata::class)
-            ->setMethods(array('getMailchimpSyncDeleted', 'getEcommerceSyncDataItem', 'getMailchimpSyncModified'))
-            ->getMock();
+        $itemMock = $this->getOrderItemMock();
+        $orderMock = $this->getOrderMock();
+        $eventObserverMock = $this->getObserverMock();
+        $eventMock = $this->getEventObserverMock();
+        $observerMock = $this->getMailchimpObserverMock();
+        $helperMock = $this->getHelperMock();
+        $mageAppMock = $this->getMageAppMock();
+        $requestMock = $this->getRequestMock();
+        $subscriberMock = $this->getSubscriberMock();
+        $dataProductMock = $this->getEcommerceModelMock();
 
         $observerMock->expects($this->once())->method('makeHelper')->willReturn($helperMock);
 
@@ -692,20 +515,11 @@ public function testAddColumnToSalesOrderGridCollection()
         $condition = 'mc.related_id=main_table.entity_id AND type = ' . Ebizmarts_MailChimp_Model_Config::IS_ORDER;
         $direction = 'ASC';
 
-        $eventObserverMock = $this->getMockBuilder(Varien_Event_Observer::class)
-            ->disableOriginalConstructor()
-            ->setMethods(array('getOrderGridCollection'))
-            ->getMock();
+        $eventObserverMock = $this->getObserverMock();
 
-        $observerMock = $this->getMockBuilder(Ebizmarts_MailChimp_Model_Observer::class)
-            ->disableOriginalConstructor()
-            ->setMethods(array('makeHelper', 'getCoreResource', 'getRegistry', 'removeRegistry'))
-            ->getMock();
+        $observerMock = $this->getMailchimpObserverMock();
 
-        $helperMock = $this->getMockBuilder(Ebizmarts_MailChimp_Helper_Data::class)
-            ->disableOriginalConstructor()
-            ->setMethods(array('getMonkeyInGrid', 'isEcomSyncDataEnabledInAnyScope'))
-            ->getMock();
+        $helperMock = $this->getHelperMock();
 
         $orderGridCollectionMock = $this->getMockBuilder(Mage_Sales_Model_Resource_Order_Grid_Collection::class)
             ->disableOriginalConstructor()
@@ -788,33 +602,14 @@ public function testSubscriberSaveBefore($data)
         $storeId = 1;
         $subscriberSource = null;
 
-        $eventObserverMock = $this->getMockBuilder(Varien_Event_Observer::class)
-            ->disableOriginalConstructor()
-            ->setMethods(array('getEvent'))
-            ->getMock();
+        $eventObserverMock = $this->getObserverMock();
+        $observerMock = $this->getMailchimpObserverMock();
 
-        $observerMock = $this->getMockBuilder(Ebizmarts_MailChimp_Model_Observer::class)
-            ->disableOriginalConstructor()
-            ->setMethods(array('makeHelper', 'addSuccessIfRequired', 'isMailchimpSave'))
-            ->getMock();
-
-        $helperMock = $this->getMockBuilder(Ebizmarts_MailChimp_Helper_Data::class)
-            ->disableOriginalConstructor()
-            ->setMethods(
-                array('isSubscriptionEnabled', 'isEcomSyncDataEnabledInAnyScope',
-                'isSubscriptionConfirmationEnabled', 'getStoreId', 'isUseMagentoEmailsEnabled')
-            )
-            ->getMock();
+        $helperMock = $this->getHelperMock();
 
-        $eventMock = $this->getMockBuilder(Varien_Event::class)
-            ->disableOriginalConstructor()
-            ->setMethods(array('getSubscriber'))
-            ->getMock();
+        $eventMock = $this->getEventObserverMock();
 
-        $subscriberMock = $this->getMockBuilder(Mage_Newsletter_Model_Subscriber::class)
-            ->disableOriginalConstructor()
-            ->setMethods(array('getSubscriberSource', 'getIsStatusChanged', 'getStatus', 'setStatus', 'getStoreId'))
-            ->getMock();
+        $subscriberMock = $this->getSubscriberMock();
 
         $eventObserverMock->expects($this->once())->method('getEvent')->willReturn($eventMock);
 
@@ -864,38 +659,16 @@ public function testSubscriberSaveAfterUseMagentoEmail()
         $params = array();
         $subscriberSource = null;
 
-        $eventObserverMock = $this->getMockBuilder(Varien_Event_Observer::class)
-            ->disableOriginalConstructor()
-            ->setMethods(array('getEvent'))
-            ->getMock();
-
-        $observerMock = $this->getMockBuilder(Ebizmarts_MailChimp_Model_Observer::class)
-            ->disableOriginalConstructor()
-            ->setMethods(
-                array('makeHelper', 'getRequest', 'createEmailCookie', 'makeApiSubscriber',
-                    'getStoreViewIdBySubscriber', 'isEmailConfirmationRequired', 'isMailchimpSave')
-            )
-            ->getMock();
+        $eventObserverMock = $this->getObserverMock();
+        $observerMock = $this->getMailchimpObserverMock();
 
-        $helperMock = $this->getMockBuilder(Ebizmarts_MailChimp_Helper_Data::class)
-            ->disableOriginalConstructor()
-            ->setMethods(array('isSubscriptionEnabled', 'saveInterestGroupData', 'isUseMagentoEmailsEnabled'))
-            ->getMock();
+        $helperMock = $this->getHelperMock();
 
-        $eventMock = $this->getMockBuilder(Varien_Event::class)
-            ->disableOriginalConstructor()
-            ->setMethods(array('getSubscriber'))
-            ->getMock();
+        $eventMock = $this->getEventObserverMock();
 
-        $subscriberMock = $this->getMockBuilder(Mage_Newsletter_Model_Subscriber::class)
-            ->disableOriginalConstructor()
-            ->setMethods(array('getSubscriberSource'))
-            ->getMock();
+        $subscriberMock = $this->getSubscriberMock();
 
-        $requestMock = $this->getMockBuilder(Mage_Core_Controller_Request_Http::class)
-            ->disableOriginalConstructor()
-            ->setMethods(array('getParams'))
-            ->getMock();
+        $requestMock = $this->getRequestMock();
 
         $apiSubscriberMock = $this->getMockBuilder(Ebizmarts_MailChimp_Model_Api_Subscribers::class)
             ->disableOriginalConstructor()
@@ -941,39 +714,17 @@ public function testSubscriberSaveAfterEmailConfirmation()
         $params = array();
         $subscriberSource = Ebizmarts_MailChimp_Model_Subscriber::SUBSCRIBE_CONFIRMATION;
 
-        $eventObserverMock = $this->getMockBuilder(Varien_Event_Observer::class)
-            ->disableOriginalConstructor()
-            ->setMethods(array('getEvent'))
-            ->getMock();
+        $eventObserverMock = $this->getObserverMock();
 
-        $observerMock = $this->getMockBuilder(Ebizmarts_MailChimp_Model_Observer::class)
-            ->disableOriginalConstructor()
-            ->setMethods(
-                array('makeHelper', 'getRequest', 'createEmailCookie', 'makeApiSubscriber',
-                    'getStoreViewIdBySubscriber', 'isMagentoSubscription', 'isEmailConfirmationRequired',
-                    'isMailchimpSave')
-            )
-            ->getMock();
+        $observerMock = $this->getMailchimpObserverMock();
 
-        $helperMock = $this->getMockBuilder(Ebizmarts_MailChimp_Helper_Data::class)
-            ->disableOriginalConstructor()
-            ->setMethods(array('isSubscriptionEnabled', 'saveInterestGroupData', 'isUseMagentoEmailsEnabled'))
-            ->getMock();
+        $helperMock = $this->getHelperMock();
 
-        $eventMock = $this->getMockBuilder(Varien_Event::class)
-            ->disableOriginalConstructor()
-            ->setMethods(array('getSubscriber'))
-            ->getMock();
+        $eventMock = $this->getEventObserverMock();
 
-        $subscriberMock = $this->getMockBuilder(Mage_Newsletter_Model_Subscriber::class)
-            ->disableOriginalConstructor()
-            ->setMethods(array('getSubscriberSource', 'getIsStatusChanged'))
-            ->getMock();
+        $subscriberMock = $this->getSubscriberMock();
 
-        $requestMock = $this->getMockBuilder(Mage_Core_Controller_Request_Http::class)
-            ->disableOriginalConstructor()
-            ->setMethods(array('getParams'))
-            ->getMock();
+        $requestMock = $this->getRequestMock();
 
         $apiSubscriberMock = $this->getMockBuilder(Ebizmarts_MailChimp_Model_Api_Subscribers::class)
             ->disableOriginalConstructor()
@@ -1035,31 +786,13 @@ public function testLoadCustomerToQuoteOnCheckout()
         $campaignId = 'gf45f4gg';
         $landingCookie = 'http%3A//127.0.0.1/MASTER1939m4m/%3Fmc_cid%3Dgf45f4gg%26mc_eid%3D7dgasydg';
 
-        $eventObserverMock = $this->getMockBuilder(Varien_Event_Observer::class)
-            ->disableOriginalConstructor()
-            ->setMethods(array('getEvent'))
-            ->getMock();
+        $eventObserverMock = $this->getObserverMock();
 
-        $eventMock = $this->getMockBuilder(Varien_Event::class)
-            ->disableOriginalConstructor()
-            ->setMethods(array('getQuote'))
-            ->getMock();
+        $eventMock = $this->getEventObserverMock();
 
-        $observerMock = $this->getMockBuilder(Ebizmarts_MailChimp_Model_Observer::class)
-            ->disableOriginalConstructor()
-            ->setMethods(
-                array(
-                    'makeHelper', 'isCustomerLoggedIn', 'getRequestActionName',
-                    'getEmailFromPopUp', 'getEmailCookie',
-                    '_getCampaignCookie', '_getLandingCookie'
-                )
-            )
-            ->getMock();
+        $observerMock = $this->getMailchimpObserverMock();
 
-        $helperMock = $this->getMockBuilder(Ebizmarts_MailChimp_Helper_Data::class)
-            ->disableOriginalConstructor()
-            ->setMethods(array('isEcomSyncDataEnabled', 'isAbandonedCartEnabled'))
-            ->getMock();
+        $helperMock = $this->getHelperMock();
 
         $quoteMock = $this->getMockBuilder(Mage_Sales_Model_Quote::class)
             ->disableOriginalConstructor()
@@ -1114,31 +847,13 @@ public function testLoadCustomerToQuote($cookieData)
         $campaignId = 'gf45f4gg';
         $landingCookie = 'http%3A//127.0.0.1/MASTER1939m4m/%3Fmc_cid%3Dgf45f4gg%26mc_eid%3D7dgasydg';
 
-        $eventObserverMock = $this->getMockBuilder(Varien_Event_Observer::class)
-            ->disableOriginalConstructor()
-            ->setMethods(array('getEvent'))
-            ->getMock();
+        $eventObserverMock = $this->getObserverMock();
 
-        $eventMock = $this->getMockBuilder(Varien_Event::class)
-            ->disableOriginalConstructor()
-            ->setMethods(array('getQuote'))
-            ->getMock();
+        $eventMock = $this->getEventObserverMock();
 
-        $observerMock = $this->getMockBuilder(Ebizmarts_MailChimp_Model_Observer::class)
-            ->disableOriginalConstructor()
-            ->setMethods(
-                array(
-                    'makeHelper', 'isCustomerLoggedIn', 'getRequestActionName',
-                    'getEmailFromPopUp', 'getEmailCookie',
-                    '_getCampaignCookie', '_getLandingCookie'
-                )
-            )
-            ->getMock();
+        $observerMock = $this->getMailchimpObserverMock();
 
-        $helperMock = $this->getMockBuilder(Ebizmarts_MailChimp_Helper_Data::class)
-            ->disableOriginalConstructor()
-            ->setMethods(array('isEcomSyncDataEnabled', 'isAbandonedCartEnabled'))
-            ->getMock();
+        $helperMock = $this->getHelperMock();
 
         $quoteMock = $this->getMockBuilder(Mage_Sales_Model_Quote::class)
             ->disableOriginalConstructor()
@@ -1197,44 +912,22 @@ public function testItemCancel()
         $isMarkedAsDeleted = 0;
         $type = Ebizmarts_MailChimp_Model_Config::IS_PRODUCT;
 
-        $observerMock = $this->getMockBuilder(Varien_Event_Observer::class)
-            ->disableOriginalConstructor()
-            ->setMethods(array('getEvent'))
-            ->getMock();
+        $observerMock = $this->getObserverMock();
 
-        $eventObserverMock = $this->getMockBuilder(Varien_Event::class)
-            ->disableOriginalConstructor()
-            ->setMethods(array('getItem'))
-            ->getMock();
+        $eventObserverMock = $this->getEventObserverMock();
 
-        $mailchimpObserverMock = $this->getMockBuilder(Ebizmarts_MailChimp_Model_Observer::class)
-            ->disableOriginalConstructor()
-            ->setMethods(
-                array(
-                    'makeHelper', 'makeApiProduct', 'isBundleItem', 'isConfigurableItem',
-                    'getMailchimpEcommerceSyncDataModel'
-                )
-            )
-            ->getMock();
+        $mailchimpObserverMock = $this->getMailchimpObserverMock();
 
-        $itemMock = $this->getMockBuilder(Mage_Sales_Model_Order_Item::class)
-            ->disableOriginalConstructor()
-            ->setMethods(array('getStoreId', 'getProductId'))
-            ->getMock();
+        $itemMock = $this->getOrderItemMock();
 
-        $helperMock = $this->getMockBuilder(Ebizmarts_MailChimp_Helper_Data::class)
-            ->disableOriginalConstructor()
-            ->setMethods(array('isEcomSyncDataEnabled', 'getMCStoreId'))
-            ->getMock();
+        $helperMock = $this->getHelperMock();
 
         $apiProductsMock = $this->getMockBuilder(Ebizmarts_MailChimp_Model_Api_Products::class)
             ->disableOriginalConstructor()
             ->setMethods(array('update'))
             ->getMock();
 
-        $dataProductMock = $this->getMockBuilder(Ebizmarts_MailChimp_Model_Ecommercesyncdata::class)
-            ->setMethods(array('getMailchimpSyncDeleted', 'getEcommerceSyncDataItem'))
-            ->getMock();
+        $dataProductMock = $this->getEcommerceModelMock();
 
         $observerMock->expects($this->once())->method('getEvent')->willReturn($eventObserverMock);
 
@@ -1285,15 +978,9 @@ public function testHandleCustomerGroupsIsSubscribed()
         $customerId = 15;
         $subscriberEmail = 'luciaines+testHandelCustomerGroups@ebizmarts.com';
 
-        $mailchimpObserverMock = $this->getMockBuilder(Ebizmarts_MailChimp_Model_Observer::class)
-            ->disableOriginalConstructor()
-            ->setMethods(array('makeHelper', 'getSubscriberModel'))
-            ->getMock();
+        $mailchimpObserverMock = $this->getMailchimpObserverMock();
 
-        $helperMock = $this->getMockBuilder(Ebizmarts_MailChimp_Helper_Data::class)
-            ->disableOriginalConstructor()
-            ->setMethods(array('saveInterestGroupData'))
-            ->getMock();
+        $helperMock = $this->getHelperMock();
 
         $subbscriberModelMock = $this->getMockBuilder(Ebizmarts_MailChimp_Model_Subscriber::class)
             ->setMethods(array('loadByEmail'))
@@ -1338,15 +1025,9 @@ public function testHandleCustomerGroupsIsNotSubcribedFromAdmin()
         $storeId = 1;
         $groups = array('d46296f47c' => array('3dd23446e4' => '3dd23446e4', 'a6c3c332bf' => 'a6c3c332bf'));
 
-        $mailchimpObserverMock = $this->getMockBuilder(Ebizmarts_MailChimp_Model_Observer::class)
-            ->disableOriginalConstructor()
-            ->setMethods(array('makeHelper', 'getSubscriberModel', 'getWarningMessageAdminHtmlSession'))
-            ->getMock();
+        $mailchimpObserverMock = $this->getMailchimpObserverMock();
 
-        $helperMock = $this->getMockBuilder(Ebizmarts_MailChimp_Helper_Data::class)
-            ->disableOriginalConstructor()
-            ->setMethods(array('saveInterestGroupData', 'getInterestGroupsIfAvailable'))
-            ->getMock();
+        $helperMock = $this->getHelperMock();
 
         $subbscriberModelMock = $this->getMockBuilder(Ebizmarts_MailChimp_Model_Subscriber::class)
             ->setMethods(array('loadByEmail'))
@@ -1381,15 +1062,9 @@ public function testHandleCustomerGroupsIsNotSubcribedFromFrontEnd()
         $customerId = 15;
         $subscriberEmail = 'luciaines+testHandelCustomerGroups@ebizmarts.com';
 
-        $mailchimpObserverMock = $this->getMockBuilder(Ebizmarts_MailChimp_Model_Observer::class)
-            ->disableOriginalConstructor()
-            ->setMethods(array('makeHelper', 'getSubscriberModel'))
-            ->getMock();
+        $mailchimpObserverMock = $this->getMailchimpObserverMock();
 
-        $helperMock = $this->getMockBuilder(Ebizmarts_MailChimp_Helper_Data::class)
-            ->disableOriginalConstructor()
-            ->setMethods(array('saveInterestGroupData'))
-            ->getMock();
+        $helperMock = $this->getHelperMock();
 
         $subbscriberModelMock = $this->getMockBuilder(Ebizmarts_MailChimp_Model_Subscriber::class)
             ->setMethods(array('loadByEmail'))
@@ -1420,29 +1095,19 @@ public function testAddOrderViewMonkey()
         $html = '';
         $storeId = 1;
 
-        $mailchimpObserverMock = $this->getMockBuilder(Ebizmarts_MailChimp_Model_Observer::class)
-            ->setMethods(array('makeHelper'))
-            ->getMock();
+        $mailchimpObserverMock = $this->getMailchimpObserverMock();
 
-        $helperMock = $this->getMockBuilder(Ebizmarts_MailChimp_Helper_Data::class)
-            ->setMethods(array('isEcomSyncDataEnabled'))
-            ->getMock();
+        $helperMock = $this->getHelperMock();
 
-        $observerMock = $this->getMockBuilder(Varien_Event_Observer::class)
-            ->setMethods(array('getBlock', 'getTransport'))
-            ->getMock();
+        $observerMock = $this->getObserverMock();
 
         $blockMock = $this->getMockBuilder(Mage_Core_Block_Abstract::class)
             ->setMethods(array('getNameInLayout', 'getOrder', 'getChild'))
             ->getMock();
 
-        $transportMock = $this->getMockBuilder(Varien_Event::class)
-            ->setMethods(array('getHtml', 'setHtml'))
-            ->getMock();
+        $transportMock = $this->getEventObserverMock();
 
-        $orderMock = $this->getMockBuilder(Mage_Sales_Model_Order::class)
-            ->setMethods(array('getStoreId'))
-            ->getMock();
+        $orderMock = $this->getOrderMock();
 
         $childMock = $this->getMockBuilder(Mage_Core_Helper_String::class)
             ->setMethods(array('toHtml'))
@@ -1479,16 +1144,11 @@ public function testCleanProductImagesCacheAfter()
         $configValues = array(array(Ebizmarts_MailChimp_Model_Config::PRODUCT_IMAGE_CACHE_FLUSH, 1));
         $default = 'default';
 
-        $mailchimpObserverMock = $this->getMockBuilder(Ebizmarts_MailChimp_Model_Observer::class)
-            ->setMethods(array('makeHelper'))
-            ->getMock();
+        $mailchimpObserverMock = $this->getMailchimpObserverMock();
 
-        $helperMock = $this->getMockBuilder(Ebizmarts_MailChimp_Helper_Data::class)
-            ->setMethods(array('saveMailchimpConfig', 'addAdminWarning'))
-            ->getMock();
+        $helperMock = $this->getHelperMock();
 
-        $observerMock = $this->getMockBuilder(Varien_Event_Observer::class)
-            ->getMock();
+        $observerMock = $this->getObserverMock();
 
 
         $mailchimpObserverMock->expects($this->once())->method('makeHelper')->willReturn($helperMock);
@@ -1520,23 +1180,11 @@ public function testSaveConfigBeforeInheritList()
         $message = 'The audience configuration was automatically modified to show the audience associated '
             . 'to the selected Mailchimp store.';
 
-        $mailchimpObserverMock = $this->getMockBuilder(Ebizmarts_MailChimp_Model_Observer::class)
-            ->disableOriginalConstructor()
-            ->setMethods(array('makeHelper', 'getAdminSession', 'isListXorStoreInherited'))
-            ->getMock();
+        $mailchimpObserverMock = $this->getMailchimpObserverMock();
 
-        $helperMock = $this->getMockBuilder(Ebizmarts_MailChimp_Helper_Data::class)
-            ->disableOriginalConstructor()
-            ->setMethods(
-                array('getCurrentScope', 'getIfConfigExistsForScope', 'saveMailchimpConfig', 'getGeneralList',
-                'getListIdByApiKeyAndMCStoreId', 'getMCStoreId')
-            )
-            ->getMock();
+        $helperMock = $this->getHelperMock();
 
-        $observerMock = $this->getMockBuilder(Varien_Event_Observer::class)
-            ->disableOriginalConstructor()
-            ->setMethods(array('getObject'))
-            ->getMock();
+        $observerMock = $this->getObserverMock();
 
         $configMock = $this->getMockBuilder(Mage_Adminhtml_Model_Config::class)
             ->disableOriginalConstructor()
@@ -1608,20 +1256,11 @@ public function testSaveConfigBeforeInheritStore()
         $message = 'The Mailchimp store configuration was not modified. There is a Mailchimp audience configured '
             . 'for this scope. Both must be set to inherit at the same time.';
 
-        $mailchimpObserverMock = $this->getMockBuilder(Ebizmarts_MailChimp_Model_Observer::class)
-            ->disableOriginalConstructor()
-            ->setMethods(array('makeHelper', 'getAdminSession', 'isListXorStoreInherited'))
-            ->getMock();
+        $mailchimpObserverMock = $this->getMailchimpObserverMock();
 
-        $helperMock = $this->getMockBuilder(Ebizmarts_MailChimp_Helper_Data::class)
-            ->disableOriginalConstructor()
-            ->setMethods(array('getCurrentScope', 'getIfConfigExistsForScope', 'saveMailchimpConfig', 'getMCStoreId'))
-            ->getMock();
+        $helperMock = $this->getHelperMock();
 
-        $observerMock = $this->getMockBuilder(Varien_Event_Observer::class)
-            ->disableOriginalConstructor()
-            ->setMethods(array('getObject'))
-            ->getMock();
+        $observerMock = $this->getObserverMock();
 
         $configMock = $this->getMockBuilder(Mage_Adminhtml_Model_Config::class)
             ->disableOriginalConstructor()
@@ -1673,18 +1312,9 @@ public function testProductSaveAfter()
         $isMarkedAsDeleted = 1;
         $status = array($productId => 1);
 
-        $mailchimpObserverMock = $this->getMockBuilder(Ebizmarts_MailChimp_Model_Observer::class)
-            ->setMethods(
-                array(
-                    'makeHelper', 'makeApiProduct', 'getCatalogProductStatusModel',
-                    'getMailchimpEcommerceSyncDataModel'
-                )
-            )
-            ->getMock();
+        $mailchimpObserverMock = $this->getMailchimpObserverMock();
 
-        $helperMock = $this->getMockBuilder(Ebizmarts_MailChimp_Helper_Data::class)
-            ->setMethods(array('getMageApp', 'isEcommerceEnabled', 'getMCStoreId'))
-            ->getMock();
+        $helperMock = $this->getHelperMock();
 
         $productMock = $this->getMockBuilder(Ebizmarts_MailChimp_Model_Api_Products::class)
             ->setMethods(array('getId'))
@@ -1694,30 +1324,21 @@ public function testProductSaveAfter()
             ->setMethods(array())
             ->getMock();
 
-        $mageCoreModelAppMock = $this->getMockBuilder(Mage_Core_Model_App::class)
-            ->setMethods(array('getStores'))
-            ->getMock();
+        $mageCoreModelAppMock = $this->getMageAppMock();
 
-        $dataProductMock = $this->getMockBuilder(Ebizmarts_MailChimp_Model_Ecommercesyncdata::class)
-            ->setMethods(array('delete', 'getMailchimpSyncDeleted', 'getEcommerceSyncDataItem'))
-            ->getMock();
+        $dataProductMock = $this->getEcommerceModelMock();
 
         $productStatusMock = $this->getMockBuilder(Mage_Catalog_Model_Resource_Product_Status::class)
             ->setMethods(array('getProductStatus'))
             ->getMock();
 
-        $observerMock = $this->getMockBuilder(Varien_Event_Observer::class)
-            ->setMethods(array('getEvent'))
-            ->getMock();
+        $observerMock = $this->getObserverMock();
 
         $storeMock = $this->getMockBuilder(ArrayObject::class)
             ->setMethods(array('getIterator'))
             ->getMock();
 
-        $eventObserverMock = $this->getMockBuilder(Varien_Event::class)
-            ->disableOriginalConstructor()
-            ->setMethods(array('getProduct'))
-            ->getMock();
+        $eventObserverMock = $this->getEventObserverMock();
 
         $mailchimpObserverMock->expects($this->once())
             ->method('makeHelper')
@@ -1797,37 +1418,20 @@ public function testNewCreditMemo()
         $type = Ebizmarts_MailChimp_Model_Config::IS_PRODUCT;
         $productId = 910;
 
-        $mailchimpObserverMock = $this->getMockBuilder(Ebizmarts_MailChimp_Model_Observer::class)
-            ->setMethods(
-                array(
-                    'makeHelper', 'makeApiProduct', 'makeApiOrder', 'isBundleItem', 'isConfigurableItem',
-                    'getMailchimpEcommerceSyncDataModel'
-                )
-            )
-            ->getMock();
+        $mailchimpObserverMock = $this->getMailchimpObserverMock();
 
-        $observerMock = $this->getMockBuilder(Varien_Event_Observer::class)
-            ->setMethods(array('getEvent'))
-            ->getMock();
+        $observerMock = $this->getObserverMock();
 
-        $eventObserverMock = $this->getMockBuilder(Varien_Event::class)
-            ->disableOriginalConstructor()
-            ->setMethods(array('getCreditmemo'))
-            ->getMock();
+        $eventObserverMock = $this->getEventObserverMock();
 
         $creditMemoMock = $this->getMockBuilder(Mage_Sales_Model_Order_Creditmemo::class)
             ->disableOriginalConstructor()
             ->setMethods(array('getOrder', 'getAllItems'))
             ->getMock();
 
-        $orderMock = $this->getMockBuilder(Mage_Sales_Model_Order::class)
-            ->disableOriginalConstructor()
-            ->setMethods(array('getStoreId', 'getEntityId'))
-            ->getMock();
+        $orderMock = $this->getOrderMock();
 
-        $helperMock = $this->getMockBuilder(Ebizmarts_MailChimp_Helper_Data::class)
-            ->setMethods(array('isEcomSyncDataEnabled', 'getMCStoreId'))
-            ->getMock();
+        $helperMock = $this->getHelperMock();
 
         $productApiMock = $this->getMockBuilder(Ebizmarts_MailChimp_Model_Api_Products::class)
             ->setMethods(array('update'))
@@ -1837,14 +1441,9 @@ public function testNewCreditMemo()
             ->setMethods(array('update'))
             ->getMock();
 
-        $itemMock = $this->getMockBuilder(Mage_Sales_Model_Order_Item::class)
-            ->disableOriginalConstructor()
-            ->setMethods(array('getProductId'))
-            ->getMock();
+        $itemMock = $this->getOrderItemMock();
 
-        $dataProductMock = $this->getMockBuilder(Ebizmarts_MailChimp_Model_Ecommercesyncdata::class)
-            ->setMethods(array('getMailchimpSyncDeleted', 'getEcommerceSyncDataItem'))
-            ->getMock();
+        $dataProductMock = $this->getEcommerceModelMock();
 
         $mailchimpObserverMock->expects($this->once())
             ->method('makeHelper')
@@ -2038,37 +1637,20 @@ public function testCancelCreditMemo()
         $type = Ebizmarts_MailChimp_Model_Config::IS_PRODUCT;
         $productId = 910;
 
-        $mailchimpObserverMock = $this->getMockBuilder(Ebizmarts_MailChimp_Model_Observer::class)
-            ->setMethods(
-                array(
-                    'makeHelper', 'makeApiProduct', 'makeApiOrder', 'isBundleItem', 'isConfigurableItem',
-                    'getMailchimpEcommerceSyncDataModel'
-                )
-            )
-            ->getMock();
+        $mailchimpObserverMock = $this->getMailchimpObserverMock();
 
-        $observerMock = $this->getMockBuilder(Varien_Event_Observer::class)
-            ->setMethods(array('getEvent'))
-            ->getMock();
+        $observerMock = $this->getObserverMock();
 
-        $eventObserverMock = $this->getMockBuilder(Varien_Event::class)
-            ->disableOriginalConstructor()
-            ->setMethods(array('getCreditmemo'))
-            ->getMock();
+        $eventObserverMock = $this->getEventObserverMock();
 
         $creditMemoMock = $this->getMockBuilder(Mage_Sales_Model_Order_Creditmemo::class)
             ->disableOriginalConstructor()
             ->setMethods(array('getOrder', 'getAllItems'))
             ->getMock();
 
-        $orderMock = $this->getMockBuilder(Mage_Sales_Model_Order::class)
-            ->disableOriginalConstructor()
-            ->setMethods(array('getStoreId', 'getEntityId'))
-            ->getMock();
+        $orderMock = $this->getOrderMock();
 
-        $helperMock = $this->getMockBuilder(Ebizmarts_MailChimp_Helper_Data::class)
-            ->setMethods(array('isEcomSyncDataEnabled', 'getMCStoreId'))
-            ->getMock();
+        $helperMock = $this->getHelperMock();
 
         $productApiMock = $this->getMockBuilder(Ebizmarts_MailChimp_Model_Api_Products::class)
             ->setMethods(array('update'))
@@ -2083,9 +1665,7 @@ public function testCancelCreditMemo()
             ->setMethods(array('getProductId'))
             ->getMock();
 
-        $dataProductMock = $this->getMockBuilder(Ebizmarts_MailChimp_Model_Ecommercesyncdata::class)
-            ->setMethods(array('getMailchimpSyncDeleted', 'getEcommerceSyncDataItem'))
-            ->getMock();
+        $dataProductMock = $this->getEcommerceModelMock();
 
         $mailchimpObserverMock->expects($this->once())
             ->method('makeHelper')
@@ -2168,7 +1748,19 @@ protected function getMailchimpObserverMock()
     {
         return $this->getMockBuilder(Ebizmarts_MailChimp_Model_Observer::class)
             ->disableOriginalConstructor()
-            ->setMethods(array('getRequest', 'getSubscriberModel', 'makeHelper'))
+            ->setMethods(
+                array(
+                    'getRequest', 'getSubscriberModel', 'makeHelper', 'makeApiProduct',
+                    'removeCampaignData', 'getMailchimpEcommerceSyncDataModel', 'isBundleItem', 
+                    'isConfigurableItem', 'getRegistry', 'removeRegistry', 'getCoreResource',
+                    'addSuccessIfRequired', 'isMailchimpSave', 'createEmailCookie', 'makeApiSubscriber',
+                    'getStoreViewIdBySubscriber', 'isEmailConfirmationRequired', 'isMagentoSubscription',
+                    'isCustomerLoggedIn', 'getRequestActionName', 'getEmailFromPopUp', 'getEmailCookie',
+                    '_getCampaignCookie', '_getLandingCookie', 'getWarningMessageAdminHtmlSession',
+                    'getAdminSession', 'isListXorStoreInherited', 'getCatalogProductStatusModel',
+                    'makeApiOrder', 'makeApiCustomer', 'getCustomerModel'
+                )
+            )
             ->getMock();
     }
 
@@ -2176,7 +1768,12 @@ protected function getObserverMock()
     {
         return $this->getMockBuilder(Varien_Event_Observer::class)
             ->disableOriginalConstructor()
-            ->setMethods(array('getEvent'))
+            ->setMethods(
+                array(
+                    'getEvent', 'getOrderGridCollection', 'getBlock', 'getTransport', 
+                    'getObject'
+                    )
+                )
             ->getMock();
     }
 
@@ -2184,7 +1781,13 @@ protected function getEventObserverMock()
     {
         return $this->_eventObserverMock = $this->getMockBuilder(Varien_Event::class)
             ->disableOriginalConstructor()
-            ->setMethods(array('getCreditmemo'))
+            ->setMethods(
+                array(
+                        'getCreditmemo', 'getOrder', 'getSubscriber', 'getQuote', 'getItem', 
+                        'getHtml', 'setHtml', 'getProduct', 'getProductIds', 'getCustomer',
+                        'getCustomerAddress'
+                    )
+                )
             ->getMock();
     }
 
@@ -2200,7 +1803,13 @@ protected function getOrderMock()
     {
         return $this->_orderMock = $this->getMockBuilder(Mage_Sales_Model_Order::class)
             ->disableOriginalConstructor()
-            ->setMethods(array('getCustomerEmail'))
+            ->setMethods(
+                array(
+                    'getCustomerEmail', 'getStoreId', 'getCustomerFirstname', 
+                    'getCustomerLastname', 'getAllItems', 'getEntityId', 'setStoreId', 
+                    "setMailchimpCampaignId", "getMailchimpLandingPage", "setMailchimpLandingPage"
+                )
+            )
             ->getMock();
     }
 
@@ -2208,7 +1817,7 @@ protected function getRequestMock()
     {
         return $this->_requestMock = $this->getMockBuilder(Mage_Core_Controller_Request_Http::class)
             ->disableOriginalConstructor()
-            ->setMethods(array('getParam'))
+            ->setMethods(array('getParam', 'getPost', 'getParams'))
             ->getMock();
     }
 
@@ -2216,7 +1825,13 @@ protected function getSubscriberMock()
     {
         return $this->_subscriberMock = $this->getMockBuilder(Mage_Newsletter_Model_Subscriber::class)
             ->disableOriginalConstructor()
-            ->setMethods(array('loadByEmail'))
+            ->setMethods(
+                array(
+                    'loadByEmail', 'getCustomerId', 'setSubscriberFirstname', 'setSubscriberLastname', 
+                    'subscribe', 'getSubscriberSource', 'getIsStatusChanged', 'getStatus', 'setStatus', 
+                    'getStoreId', 'getId', 'loadByCustomer', 'setSubscriberEmail', 'save'
+                )
+            )
             ->getMock();
     }
 
@@ -2224,7 +1839,45 @@ protected function getHelperMock()
     {
         return $this->_helperMock = $this->getMockBuilder(Ebizmarts_MailChimp_Helper_Data::class)
             ->disableOriginalConstructor()
-            ->setMethods(array('unsubscribeMember'))
+            ->setMethods(
+                array(
+                    'unsubscribeMember', 'getMageApp', 'isEcomSyncDataEnabled', 'isSubscriptionEnabled', 
+                    'loadListSubscriber', 'saveEcommerceSyncData', 'getMCStoreId', 'getMonkeyInGrid', 
+                    'isEcomSyncDataEnabledInAnyScope','getAllMailChimpStoreIds', 'isEcommerceEnabled', 
+                    'isSubscriptionConfirmationEnabled', 'getStoreId', 'isUseMagentoEmailsEnabled',
+                    'saveInterestGroupData', 'isAbandonedCartEnabled', 'getInterestGroupsIfAvailable',
+                    'saveMailchimpConfig', 'addAdminWarning', 'getCurrentScope', 'getIfConfigExistsForScope', 
+                    'getGeneralList', 'getListIdByApiKeyAndMCStoreId'
+                )
+            )
+            ->getMock();
+    }
+
+    protected function getOrderItemMock()
+    {
+        return $this->getMockBuilder(Mage_Sales_Model_Order_Item::class)
+            ->disableOriginalConstructor()
+            ->setMethods(array('getProductType', 'getProductId', 'getStoreId'))
+            ->getMock();
+    }
+
+    protected function getMageAppMock()
+    {
+        return $this->getMockBuilder(Mage_Core_Model_App::class)
+            ->disableOriginalConstructor()
+            ->setMethods(array('getRequest', 'getStores'))
+            ->getMock();
+    }
+
+    protected function getEcommerceModelMock()
+    {
+        return $this->getMockBuilder(Ebizmarts_MailChimp_Model_Ecommercesyncdata::class)
+            ->setMethods(
+                array(
+                    'getMailchimpSyncDeleted', 'getEcommerceSyncDataItem', 'getMailchimpSyncModified',
+                    'delete'
+                )
+            )
             ->getMock();
     }
 }

From 29472aff82353ef092f317abd87d6a57e03a1c6c Mon Sep 17 00:00:00 2001
From: Javier Giorello <giorello@ebizmarts.com>
Date: Tue, 26 May 2020 09:17:58 -0300
Subject: [PATCH 7/9] Fixed phpcs

---
 .../tests/app/Ebizmarts/MailChimp/Model/ObserverTest.php      | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/dev/tests/mailchimp/tests/app/Ebizmarts/MailChimp/Model/ObserverTest.php b/dev/tests/mailchimp/tests/app/Ebizmarts/MailChimp/Model/ObserverTest.php
index 564ab8474..97b7d18b5 100755
--- a/dev/tests/mailchimp/tests/app/Ebizmarts/MailChimp/Model/ObserverTest.php
+++ b/dev/tests/mailchimp/tests/app/Ebizmarts/MailChimp/Model/ObserverTest.php
@@ -390,7 +390,7 @@ public function testNewOrder()
         $dataProductMock->expects($this->once())
             ->method('getMailchimpSyncDeleted')
             ->willReturn($isMarkedAsDeleted);
-        
+
         $dataProductMock->expects($this->once())
             ->method('getMailchimpSyncModified')
             ->willReturn($isMarkedAsDeleted);
@@ -482,7 +482,7 @@ public function testNewOrderNotModified()
         $dataProductMock->expects($this->once())
             ->method('getMailchimpSyncDeleted')
             ->willReturn($isMarkedAsDeleted);
-        
+
         $dataProductMock->expects($this->once())
             ->method('getMailchimpSyncModified')
             ->willReturn($isMarkedAsDeleted);

From 1713bf3614938b06ccb3d0ba715167b2c6e0b2c6 Mon Sep 17 00:00:00 2001
From: Javier Giorello <giorello@ebizmarts.com>
Date: Tue, 26 May 2020 09:37:35 -0300
Subject: [PATCH 8/9] Fixed phpcs

---
 .../MailChimp/Model/ObserverTest.php          | 22 +++++++++----------
 1 file changed, 11 insertions(+), 11 deletions(-)

diff --git a/dev/tests/mailchimp/tests/app/Ebizmarts/MailChimp/Model/ObserverTest.php b/dev/tests/mailchimp/tests/app/Ebizmarts/MailChimp/Model/ObserverTest.php
index 97b7d18b5..909ce32ce 100755
--- a/dev/tests/mailchimp/tests/app/Ebizmarts/MailChimp/Model/ObserverTest.php
+++ b/dev/tests/mailchimp/tests/app/Ebizmarts/MailChimp/Model/ObserverTest.php
@@ -1750,14 +1750,14 @@ protected function getMailchimpObserverMock()
             ->disableOriginalConstructor()
             ->setMethods(
                 array(
-                    'getRequest', 'getSubscriberModel', 'makeHelper', 'makeApiProduct',
+                    'getRequest', 'getSubscriberModel', 'makeHelper', 'makeApiProduct', 
                     'removeCampaignData', 'getMailchimpEcommerceSyncDataModel', 'isBundleItem', 
-                    'isConfigurableItem', 'getRegistry', 'removeRegistry', 'getCoreResource',
-                    'addSuccessIfRequired', 'isMailchimpSave', 'createEmailCookie', 'makeApiSubscriber',
-                    'getStoreViewIdBySubscriber', 'isEmailConfirmationRequired', 'isMagentoSubscription',
-                    'isCustomerLoggedIn', 'getRequestActionName', 'getEmailFromPopUp', 'getEmailCookie',
-                    '_getCampaignCookie', '_getLandingCookie', 'getWarningMessageAdminHtmlSession',
-                    'getAdminSession', 'isListXorStoreInherited', 'getCatalogProductStatusModel',
+                    'isConfigurableItem', 'getRegistry', 'removeRegistry', 'getCoreResource', 
+                    'addSuccessIfRequired', 'isMailchimpSave', 'createEmailCookie', 'makeApiSubscriber', 
+                    'getStoreViewIdBySubscriber', 'isEmailConfirmationRequired', 'isMagentoSubscription', 
+                    'isCustomerLoggedIn', 'getRequestActionName', 'getEmailFromPopUp', 'getEmailCookie', 
+                    '_getCampaignCookie', '_getLandingCookie', 'getWarningMessageAdminHtmlSession', 
+                    'getAdminSession', 'isListXorStoreInherited', 'getCatalogProductStatusModel', 
                     'makeApiOrder', 'makeApiCustomer', 'getCustomerModel'
                 )
             )
@@ -1783,8 +1783,8 @@ protected function getEventObserverMock()
             ->disableOriginalConstructor()
             ->setMethods(
                 array(
-                        'getCreditmemo', 'getOrder', 'getSubscriber', 'getQuote', 'getItem', 
-                        'getHtml', 'setHtml', 'getProduct', 'getProductIds', 'getCustomer',
+                        'getCreditmemo', 'getOrder', 'getSubscriber', 'getQuote', 'getItem',  
+                        'getHtml', 'setHtml', 'getProduct', 'getProductIds', 'getCustomer', 
                         'getCustomerAddress'
                     )
                 )
@@ -1844,8 +1844,8 @@ protected function getHelperMock()
                     'unsubscribeMember', 'getMageApp', 'isEcomSyncDataEnabled', 'isSubscriptionEnabled', 
                     'loadListSubscriber', 'saveEcommerceSyncData', 'getMCStoreId', 'getMonkeyInGrid', 
                     'isEcomSyncDataEnabledInAnyScope','getAllMailChimpStoreIds', 'isEcommerceEnabled', 
-                    'isSubscriptionConfirmationEnabled', 'getStoreId', 'isUseMagentoEmailsEnabled',
-                    'saveInterestGroupData', 'isAbandonedCartEnabled', 'getInterestGroupsIfAvailable',
+                    'isSubscriptionConfirmationEnabled', 'getStoreId', 'isUseMagentoEmailsEnabled', 
+                    'saveInterestGroupData', 'isAbandonedCartEnabled', 'getInterestGroupsIfAvailable', 
                     'saveMailchimpConfig', 'addAdminWarning', 'getCurrentScope', 'getIfConfigExistsForScope', 
                     'getGeneralList', 'getListIdByApiKeyAndMCStoreId'
                 )

From 361db83eeeb2340687fba5bdd694ec4969a818fb Mon Sep 17 00:00:00 2001
From: Javier Giorello <giorello@ebizmarts.com>
Date: Tue, 26 May 2020 10:04:11 -0300
Subject: [PATCH 9/9] Fix phpcs

---
 .../MailChimp/Model/ObserverTest.php          | 46 +++++++++----------
 1 file changed, 23 insertions(+), 23 deletions(-)

diff --git a/dev/tests/mailchimp/tests/app/Ebizmarts/MailChimp/Model/ObserverTest.php b/dev/tests/mailchimp/tests/app/Ebizmarts/MailChimp/Model/ObserverTest.php
index 909ce32ce..75fee4444 100755
--- a/dev/tests/mailchimp/tests/app/Ebizmarts/MailChimp/Model/ObserverTest.php
+++ b/dev/tests/mailchimp/tests/app/Ebizmarts/MailChimp/Model/ObserverTest.php
@@ -1750,14 +1750,14 @@ protected function getMailchimpObserverMock()
             ->disableOriginalConstructor()
             ->setMethods(
                 array(
-                    'getRequest', 'getSubscriberModel', 'makeHelper', 'makeApiProduct', 
-                    'removeCampaignData', 'getMailchimpEcommerceSyncDataModel', 'isBundleItem', 
-                    'isConfigurableItem', 'getRegistry', 'removeRegistry', 'getCoreResource', 
-                    'addSuccessIfRequired', 'isMailchimpSave', 'createEmailCookie', 'makeApiSubscriber', 
-                    'getStoreViewIdBySubscriber', 'isEmailConfirmationRequired', 'isMagentoSubscription', 
-                    'isCustomerLoggedIn', 'getRequestActionName', 'getEmailFromPopUp', 'getEmailCookie', 
-                    '_getCampaignCookie', '_getLandingCookie', 'getWarningMessageAdminHtmlSession', 
-                    'getAdminSession', 'isListXorStoreInherited', 'getCatalogProductStatusModel', 
+                    'getRequest', 'getSubscriberModel', 'makeHelper', 'makeApiProduct',
+                    'removeCampaignData', 'getMailchimpEcommerceSyncDataModel', 'isBundleItem',
+                    'isConfigurableItem', 'getRegistry', 'removeRegistry', 'getCoreResource',
+                    'addSuccessIfRequired', 'isMailchimpSave', 'createEmailCookie', 'makeApiSubscriber',
+                    'getStoreViewIdBySubscriber', 'isEmailConfirmationRequired', 'isMagentoSubscription',
+                    'isCustomerLoggedIn', 'getRequestActionName', 'getEmailFromPopUp', 'getEmailCookie',
+                    '_getCampaignCookie', '_getLandingCookie', 'getWarningMessageAdminHtmlSession',
+                    'getAdminSession', 'isListXorStoreInherited', 'getCatalogProductStatusModel',
                     'makeApiOrder', 'makeApiCustomer', 'getCustomerModel'
                 )
             )
@@ -1770,10 +1770,10 @@ protected function getObserverMock()
             ->disableOriginalConstructor()
             ->setMethods(
                 array(
-                    'getEvent', 'getOrderGridCollection', 'getBlock', 'getTransport', 
+                    'getEvent', 'getOrderGridCollection', 'getBlock', 'getTransport',
                     'getObject'
-                    )
                 )
+            )
             ->getMock();
     }
 
@@ -1783,11 +1783,11 @@ protected function getEventObserverMock()
             ->disableOriginalConstructor()
             ->setMethods(
                 array(
-                        'getCreditmemo', 'getOrder', 'getSubscriber', 'getQuote', 'getItem',  
-                        'getHtml', 'setHtml', 'getProduct', 'getProductIds', 'getCustomer', 
+                        'getCreditmemo', 'getOrder', 'getSubscriber', 'getQuote', 'getItem',
+                        'getHtml', 'setHtml', 'getProduct', 'getProductIds', 'getCustomer',
                         'getCustomerAddress'
-                    )
                 )
+            )
             ->getMock();
     }
 
@@ -1805,8 +1805,8 @@ protected function getOrderMock()
             ->disableOriginalConstructor()
             ->setMethods(
                 array(
-                    'getCustomerEmail', 'getStoreId', 'getCustomerFirstname', 
-                    'getCustomerLastname', 'getAllItems', 'getEntityId', 'setStoreId', 
+                    'getCustomerEmail', 'getStoreId', 'getCustomerFirstname',
+                    'getCustomerLastname', 'getAllItems', 'getEntityId', 'setStoreId',
                     "setMailchimpCampaignId", "getMailchimpLandingPage", "setMailchimpLandingPage"
                 )
             )
@@ -1827,8 +1827,8 @@ protected function getSubscriberMock()
             ->disableOriginalConstructor()
             ->setMethods(
                 array(
-                    'loadByEmail', 'getCustomerId', 'setSubscriberFirstname', 'setSubscriberLastname', 
-                    'subscribe', 'getSubscriberSource', 'getIsStatusChanged', 'getStatus', 'setStatus', 
+                    'loadByEmail', 'getCustomerId', 'setSubscriberFirstname', 'setSubscriberLastname',
+                    'subscribe', 'getSubscriberSource', 'getIsStatusChanged', 'getStatus', 'setStatus',
                     'getStoreId', 'getId', 'loadByCustomer', 'setSubscriberEmail', 'save'
                 )
             )
@@ -1841,12 +1841,12 @@ protected function getHelperMock()
             ->disableOriginalConstructor()
             ->setMethods(
                 array(
-                    'unsubscribeMember', 'getMageApp', 'isEcomSyncDataEnabled', 'isSubscriptionEnabled', 
-                    'loadListSubscriber', 'saveEcommerceSyncData', 'getMCStoreId', 'getMonkeyInGrid', 
-                    'isEcomSyncDataEnabledInAnyScope','getAllMailChimpStoreIds', 'isEcommerceEnabled', 
-                    'isSubscriptionConfirmationEnabled', 'getStoreId', 'isUseMagentoEmailsEnabled', 
-                    'saveInterestGroupData', 'isAbandonedCartEnabled', 'getInterestGroupsIfAvailable', 
-                    'saveMailchimpConfig', 'addAdminWarning', 'getCurrentScope', 'getIfConfigExistsForScope', 
+                    'unsubscribeMember', 'getMageApp', 'isEcomSyncDataEnabled', 'isSubscriptionEnabled',
+                    'loadListSubscriber', 'saveEcommerceSyncData', 'getMCStoreId', 'getMonkeyInGrid',
+                    'isEcomSyncDataEnabledInAnyScope','getAllMailChimpStoreIds', 'isEcommerceEnabled',
+                    'isSubscriptionConfirmationEnabled', 'getStoreId', 'isUseMagentoEmailsEnabled',
+                    'saveInterestGroupData', 'isAbandonedCartEnabled', 'getInterestGroupsIfAvailable',
+                    'saveMailchimpConfig', 'addAdminWarning', 'getCurrentScope', 'getIfConfigExistsForScope',
                     'getGeneralList', 'getListIdByApiKeyAndMCStoreId'
                 )
             )