Merge pull request #85 from sugarcrm/issue-84

Fixes #84: Fix PHPUnit tests
sugarcrm-developers · Jul 13, 2018 · 06ad7c8 · 06ad7c8
2 parents 2362a8f + e40a790
commit 06ad7c8
Showing 1 changed file with 5 additions and 4 deletions.
diff --git a/...om/tests/School/unit-php/modules/Campaigns/clients/base/api/OnlineApplicationsAPITest.php b/...om/tests/School/unit-php/modules/Campaigns/clients/base/api/OnlineApplicationsAPITest.php
@@ -2,10 +2,12 @@
 
 use Sugarcrm\SugarcrmTestsUnit\TestMockHelper;
 
+require_once 'custom/modules/Campaigns/clients/base/api/OnlineApplicationsAPI.php';
+
 /**
  * @coversDefaultClass \OnlineApplicationsAPI
  */
-class OnlineApplicationsAPI extends \PHPUnit\Framework\TestCase
+class OnlineApplicationsAPITest extends \PHPUnit\Framework\TestCase
 {
 
     /**
@@ -28,8 +30,7 @@ protected function setUp()
 
         $this->campaignBean = $this->createMock('SugarBean');
 
-        $this->api = TestMockHelper::createPartialMock($this,
-            '\\Sugarcrm\\Sugarcrm\\custom\\application_campaign\\ApplicationCampaignManager',
+        $this->api = TestMockHelper::createPartialMock($this, 'OnlineApplicationsAPI',
             ['getSugarQuery', 'getNewCampaignBean']);
         $this->api->method('getSugarQuery')->willReturn($this->sugarQuery);
         $this->api->method('getNewCampaignBean')->willReturn($this->campaignBean);
@@ -51,7 +52,7 @@ public function testGetOnlineApplicationsCampaignIdWithResults()
         $this->sugarQuery->method('execute')->willReturn($this->queryResults);
 
         $this->sugarQuery->expects($this->once())->method('execute');
-        $this->assertEquals($this->campaignId, $this->api->getOnlineApplicationsCampaignId());
+        $this->assertEquals(array('id' => $this->campaignId), $this->api->getOnlineApplicationsCampaignId());
     }
 
     /**