MAGETWO-38383: Increase Unit Test Code Coverage

Ian Daszkowski · Ian Daszkowski · commit 6dda9790094f · 2015-06-18T11:41:52.000-05:00
- Changed method names
diff --git a/setup/src/Magento/Setup/Test/Unit/Fixtures/CartPriceRulesFixtureTest.php b/setup/src/Magento/Setup/Test/Unit/Fixtures/CartPriceRulesFixtureTest.php
@@ -107,7 +107,7 @@ public function testExecute()
         $this->model->execute();
     }
 
-    public function testExecuteEarlyReturn()
+    public function testNoFixtureConfigValue()
     {
         $this->fixtureModelMock
             ->expects($this->once())
diff --git a/setup/src/Magento/Setup/Test/Unit/Fixtures/CatalogPriceRulesFixtureTest.php b/setup/src/Magento/Setup/Test/Unit/Fixtures/CatalogPriceRulesFixtureTest.php
@@ -102,7 +102,7 @@ public function testExecute()
         $this->model->execute();
     }
 
-    public function testExecuteEarlyReturn()
+    public function testNoFixtureConfigValue()
     {
         $this->fixtureModelMock
             ->expects($this->once())
diff --git a/setup/src/Magento/Setup/Test/Unit/Fixtures/CategoriesFixtureTest.php b/setup/src/Magento/Setup/Test/Unit/Fixtures/CategoriesFixtureTest.php
@@ -119,7 +119,7 @@ public function testExecute()
         $this->model->execute();
     }
 
-    public function testExecuteEarlyReturn()
+    public function testNoFixtureConfigValue()
     {
         $this->fixtureModelMock
             ->expects($this->once())
diff --git a/setup/src/Magento/Setup/Test/Unit/Fixtures/ConfigsApplyFixtureTest.php b/setup/src/Magento/Setup/Test/Unit/Fixtures/ConfigsApplyFixtureTest.php
@@ -50,7 +50,7 @@ public function testExecute()
         $this->model->execute();
     }
 
-    public function testExecuteEarlyReturn()
+    public function testNoFixtureConfigValue()
     {
         $this->fixtureModelMock
             ->expects($this->once())
diff --git a/setup/src/Magento/Setup/Test/Unit/Fixtures/ConfigurableProductsFixtureTest.php b/setup/src/Magento/Setup/Test/Unit/Fixtures/ConfigurableProductsFixtureTest.php
@@ -106,7 +106,7 @@ public function testExecute()
         $this->model->execute();
     }
 
-    public function testExecuteEarlyReturn()
+    public function testNoFixtureConfigValue()
     {
         $this->fixtureModelMock
             ->expects($this->once())
diff --git a/setup/src/Magento/Setup/Test/Unit/Fixtures/CustomersFixtureTest.php b/setup/src/Magento/Setup/Test/Unit/Fixtures/CustomersFixtureTest.php
@@ -75,7 +75,7 @@ public function testExecute()
         $this->model->execute();
     }
 
-    public function testExecuteEarlyReturn()
+    public function testNoFixtureConfigValue()
     {
         $this->fixtureModelMock
             ->expects($this->once())
diff --git a/setup/src/Magento/Setup/Test/Unit/Fixtures/EavVariationsFixtureTest.php b/setup/src/Magento/Setup/Test/Unit/Fixtures/EavVariationsFixtureTest.php
@@ -89,7 +89,7 @@ public function testExecute()
         $this->model->execute();
     }
 
-    public function testExecuteEarlyReturn()
+    public function testNoFixtureConfigValue()
     {
         $this->fixtureModelMock
             ->expects($this->once())
diff --git a/setup/src/Magento/Setup/Test/Unit/Fixtures/IndexersStatesApplyFixtureTest.php b/setup/src/Magento/Setup/Test/Unit/Fixtures/IndexersStatesApplyFixtureTest.php
@@ -50,7 +50,7 @@ public function testExecute()
         $this->model->execute();
     }
 
-    public function testExecuteEarlyReturn()
+    public function testNoFixtureConfigValue()
     {
         $this->fixtureModelMock
             ->expects($this->once())
diff --git a/setup/src/Magento/Setup/Test/Unit/Fixtures/OrdersFixtureTest.php b/setup/src/Magento/Setup/Test/Unit/Fixtures/OrdersFixtureTest.php
@@ -210,7 +210,7 @@ public function testExecute()
         $this->model->execute();
     }
 
-    public function testExecuteEarlyReturn()
+    public function testNoFixtureConfigValue()
     {
         $this->fixtureModelMock
             ->expects($this->once())
diff --git a/setup/src/Magento/Setup/Test/Unit/Fixtures/SimpleProductsFixtureTest.php b/setup/src/Magento/Setup/Test/Unit/Fixtures/SimpleProductsFixtureTest.php
@@ -106,7 +106,7 @@ public function testExecute()
         $this->model->execute();
     }
 
-    public function testExecuteEarlyReturn()
+    public function testNoFixtureConfigValue()
     {
         $this->fixtureModelMock
             ->expects($this->once())
diff --git a/setup/src/Magento/Setup/Test/Unit/Fixtures/StoresFixtureTest.php b/setup/src/Magento/Setup/Test/Unit/Fixtures/StoresFixtureTest.php
@@ -136,7 +136,7 @@ public function testExecute()
         $this->model->execute();
     }
 
-    public function testExecuteEarlyReturn()
+    public function testNoFixtureConfigValue()
     {
         $this->fixtureModelMock
             ->expects($this->exactly(3))
diff --git a/setup/src/Magento/Setup/Test/Unit/Fixtures/TaxRatesFixtureTest.php b/setup/src/Magento/Setup/Test/Unit/Fixtures/TaxRatesFixtureTest.php
@@ -70,7 +70,7 @@ public function testExecute()
         $this->model->execute();
     }
 
-    public function testExecuteEarlyReturn()
+    public function testNoFixtureConfigValue()
     {
         $this->fixtureModelMock
             ->expects($this->once())

Original file line number	Diff line number	Diff line change
`@@ -107,7 +107,7 @@ public function testExecute()`
`107`	`107`	`$this->model->execute();`
`108`	`108`	`}`
`109`	`109`
`110`		`- public function testExecuteEarlyReturn()`
	`110`	`+ public function testNoFixtureConfigValue()`
`111`	`111`	`{`
`112`	`112`	`$this->fixtureModelMock`
`113`	`113`	`->expects($this->once())`
Original file line number	Diff line number	Diff line change
`@@ -102,7 +102,7 @@ public function testExecute()`
`102`	`102`	`$this->model->execute();`
`103`	`103`	`}`
`104`	`104`
`105`		`- public function testExecuteEarlyReturn()`
	`105`	`+ public function testNoFixtureConfigValue()`
`106`	`106`	`{`
`107`	`107`	`$this->fixtureModelMock`
`108`	`108`	`->expects($this->once())`
Original file line number	Diff line number	Diff line change
`@@ -119,7 +119,7 @@ public function testExecute()`
`119`	`119`	`$this->model->execute();`
`120`	`120`	`}`
`121`	`121`
`122`		`- public function testExecuteEarlyReturn()`
	`122`	`+ public function testNoFixtureConfigValue()`
`123`	`123`	`{`
`124`	`124`	`$this->fixtureModelMock`
`125`	`125`	`->expects($this->once())`
Original file line number	Diff line number	Diff line change
`@@ -50,7 +50,7 @@ public function testExecute()`
`50`	`50`	`$this->model->execute();`
`51`	`51`	`}`
`52`	`52`
`53`		`- public function testExecuteEarlyReturn()`
	`53`	`+ public function testNoFixtureConfigValue()`
`54`	`54`	`{`
`55`	`55`	`$this->fixtureModelMock`
`56`	`56`	`->expects($this->once())`
Original file line number	Diff line number	Diff line change
`@@ -106,7 +106,7 @@ public function testExecute()`
`106`	`106`	`$this->model->execute();`
`107`	`107`	`}`
`108`	`108`
`109`		`- public function testExecuteEarlyReturn()`
	`109`	`+ public function testNoFixtureConfigValue()`
`110`	`110`	`{`
`111`	`111`	`$this->fixtureModelMock`
`112`	`112`	`->expects($this->once())`
Original file line number	Diff line number	Diff line change
`@@ -75,7 +75,7 @@ public function testExecute()`
`75`	`75`	`$this->model->execute();`
`76`	`76`	`}`
`77`	`77`
`78`		`- public function testExecuteEarlyReturn()`
	`78`	`+ public function testNoFixtureConfigValue()`
`79`	`79`	`{`
`80`	`80`	`$this->fixtureModelMock`
`81`	`81`	`->expects($this->once())`
Original file line number	Diff line number	Diff line change
`@@ -89,7 +89,7 @@ public function testExecute()`
`89`	`89`	`$this->model->execute();`
`90`	`90`	`}`
`91`	`91`
`92`		`- public function testExecuteEarlyReturn()`
	`92`	`+ public function testNoFixtureConfigValue()`
`93`	`93`	`{`
`94`	`94`	`$this->fixtureModelMock`
`95`	`95`	`->expects($this->once())`
Original file line number	Diff line number	Diff line change
`@@ -210,7 +210,7 @@ public function testExecute()`
`210`	`210`	`$this->model->execute();`
`211`	`211`	`}`
`212`	`212`
`213`		`- public function testExecuteEarlyReturn()`
	`213`	`+ public function testNoFixtureConfigValue()`
`214`	`214`	`{`
`215`	`215`	`$this->fixtureModelMock`
`216`	`216`	`->expects($this->once())`