From 39d027a39e9433d7aac60b595168049aa69a0ac2 Mon Sep 17 00:00:00 2001 From: Santiago Date: Tue, 25 Jul 2017 16:39:35 -0300 Subject: [PATCH] Test fixes. --- .../tests/app/Ebizmarts/MailChimp/Helper/DataTest.php | 3 +-- .../tests/app/Ebizmarts/MailChimp/Model/Api/BatchesTest.php | 5 +++-- 2 files changed, 4 insertions(+), 4 deletions(-) diff --git a/dev/tests/mailchimp/tests/app/Ebizmarts/MailChimp/Helper/DataTest.php b/dev/tests/mailchimp/tests/app/Ebizmarts/MailChimp/Helper/DataTest.php index 9ea6ceeed..49d0d511b 100644 --- a/dev/tests/mailchimp/tests/app/Ebizmarts/MailChimp/Helper/DataTest.php +++ b/dev/tests/mailchimp/tests/app/Ebizmarts/MailChimp/Helper/DataTest.php @@ -52,7 +52,7 @@ public function testDeleteStore() $scope = 'stores'; $helperMock = $this->getMockBuilder(Ebizmarts_MailChimp_Helper_Data::class) ->disableOriginalConstructor() - ->setMethods(array('getMCStoreId', 'getApiStores', 'getGeneralList', 'deleteCurrentWebhook', 'deleteLocalMCStoreData', 'logError')) + ->setMethods(array('getMCStoreId', 'getApiStores', 'getGeneralList', 'deleteCurrentWebhook', 'deleteLocalMCStoreData')) ->getMock(); $apiStoresMock = $this->getMockBuilder(Ebizmarts_MailChimp_Model_Api_Stores::class) ->disableOriginalConstructor() @@ -60,7 +60,6 @@ public function testDeleteStore() $helperMock->expects($this->once())->method('getMCStoreId')->with($scopeId, $scope)->willReturn('a18a1a8a1aa7aja1a'); $helperMock->expects($this->once())->method('getApiStores')->willReturn($apiStoresMock); - $helperMock->expects($this->once())->method('logError')->with('Error message', $scopeId, $scope); $helperMock->expects($this->once())->method('getGeneralList')->with($scopeId, $scope)->willReturn('listId'); $helperMock->expects($this->once())->method('deleteCurrentWebhook')->with($scopeId, $scope, 'listId'); $helperMock->expects($this->once())->method('deleteLocalMCStoreData')->with($scopeId, $scope); diff --git a/dev/tests/mailchimp/tests/app/Ebizmarts/MailChimp/Model/Api/BatchesTest.php b/dev/tests/mailchimp/tests/app/Ebizmarts/MailChimp/Model/Api/BatchesTest.php index 4cbfcbcfb..b2ba14efe 100644 --- a/dev/tests/mailchimp/tests/app/Ebizmarts/MailChimp/Model/Api/BatchesTest.php +++ b/dev/tests/mailchimp/tests/app/Ebizmarts/MailChimp/Model/Api/BatchesTest.php @@ -28,7 +28,7 @@ public function testSendEcommerceBatch() $helperMock = $this->getMockBuilder(Ebizmarts_MailChimp_Helper_Data::class) ->disableOriginalConstructor() - ->setMethods(array('getMCStoreId', 'isMailChimpEnabled', 'isEcomSyncDataEnabled', 'getApi', 'getIsReseted')) + ->setMethods(array('getMCStoreId', 'isMailChimpEnabled', 'isEcomSyncDataEnabled', 'getApi', 'getIsReseted', 'getMCIsSyncing')) ->getMock(); $apiCustomersMock = $this->getMockBuilder(Ebizmarts_MailChimp_Model_Api_Customers::class) @@ -56,7 +56,7 @@ public function testSendEcommerceBatch() $apiMock = $this->getMockBuilder(Ebizmarts_MailChimp::class) ->disableOriginalConstructor() - // ->setMethods(array('')) +// ->setMethods(array('edit')) ->getMock(); $apiBatchesMock->expects($this->once())->method('getHelper')->willReturn($helperMock); @@ -69,6 +69,7 @@ public function testSendEcommerceBatch() $helperMock->expects($this->once())->method('getMCStoreId')->with(1)->willReturn('b81c3085c51fa593e1d6b0cf59884f3e'); $helperMock->expects($this->once())->method('isMailChimpEnabled')->with(1)->willReturn(1); $helperMock->expects($this->once())->method('isEcomSyncDataEnabled')->with(1)->willReturn(1); + $helperMock->expects($this->once())->method('getMCIsSyncing')->with(1)->willReturn(0); $apiCustomersMock->expects($this->once())->method('createBatchJson')->with('b81c3085c51fa593e1d6b0cf59884f3e', 1); $apiProductsMock->expects($this->once())->method('createBatchJson')->with('b81c3085c51fa593e1d6b0cf59884f3e', 1);