ENGCOM-3336: Added $websiteString property to test class #142

magento-engcom-team · magento-engcom-team · commit 2f639c0c1549 · 2018-10-30T08:54:26.000-05:00
- Merge Pull Request #142 from maxalmonte14/import-export-improvements:improvement/supress_undefined_websiteString_property_error - Merged commits: 1. 06a47b5 2. e32808b
diff --git a/app/code/Magento/AdvancedPricingImportExport/Test/Unit/Model/Import/AdvancedPricing/Validator/WebsiteTest.php b/app/code/Magento/AdvancedPricingImportExport/Test/Unit/Model/Import/AdvancedPricing/Validator/WebsiteTest.php
@@ -103,13 +103,13 @@ public function testGetAllWebsitesValue()
         $this->webSiteModel->expects($this->once())->method('getBaseCurrency')->willReturn($currency);
 
         $expectedResult = AdvancedPricing::VALUE_ALL_WEBSITES . ' [' . $currencyCode . ']';
-        $this->websiteString = $this->getMockBuilder(
+        $websiteString = $this->getMockBuilder(
             \Magento\AdvancedPricingImportExport\Model\Import\AdvancedPricing\Validator\Website::class
         )
             ->setMethods(['_clearMessages', '_addMessages'])
             ->setConstructorArgs([$this->storeResolver, $this->webSiteModel])
             ->getMock();
-        $result = $this->websiteString->getAllWebsitesValue();
+        $result = $websiteString->getAllWebsitesValue();
 
         $this->assertEquals($expectedResult, $result);
     }

Original file line number	Diff line number	Diff line change
`@@ -103,13 +103,13 @@ public function testGetAllWebsitesValue()`
`103`	`103`	`$this->webSiteModel->expects($this->once())->method('getBaseCurrency')->willReturn($currency);`
`104`	`104`
`105`	`105`	`$expectedResult = AdvancedPricing::VALUE_ALL_WEBSITES . ' [' . $currencyCode . ']';`
`106`		`- $this->websiteString = $this->getMockBuilder(`
	`106`	`+ $websiteString = $this->getMockBuilder(`
`107`	`107`	`\Magento\AdvancedPricingImportExport\Model\Import\AdvancedPricing\Validator\Website::class`
`108`	`108`	`)`
`109`	`109`	`->setMethods(['_clearMessages', '_addMessages'])`
`110`	`110`	`->setConstructorArgs([$this->storeResolver, $this->webSiteModel])`
`111`	`111`	`->getMock();`
`112`		`- $result = $this->websiteString->getAllWebsitesValue();`
	`112`	`+ $result = $websiteString->getAllWebsitesValue();`
`113`	`113`
`114`	`114`	`$this->assertEquals($expectedResult, $result);`
`115`	`115`	`}`