MAGETWO-80778: Delete CallExit function for After plugin logic execution 2.2-develop [BackPort] #11200

miguelbalparda · miguelbalparda · commit 9699f079d396 · 2017-10-03T12:59:59.000-05:00
- Merge Pull Request #11200 from osrecio/magento2:PR#DeleteExitFileFactory-2.2 - Merged commits: 1. e050922
diff --git a/lib/internal/Magento/Framework/App/Response/Http/FileFactory.php b/lib/internal/Magento/Framework/App/Response/Http/FileFactory.php
@@ -99,19 +99,7 @@ public function create(
             if (!empty($content['rm'])) {
                 $dir->delete($file);
             }
-            $this->callExit();
         }
         return $this->_response;
     }
-
-    /**
-     * Call exit
-     *
-     * @return void
-     * @SuppressWarnings(PHPMD.ExitExpression)
-     */
-    protected function callExit()
-    {
-        exit(0);
-    }
 }
diff --git a/lib/internal/Magento/Framework/App/Test/Unit/Response/Http/FileFactoryTest.php b/lib/internal/Magento/Framework/App/Test/Unit/Response/Http/FileFactoryTest.php
@@ -235,7 +235,7 @@ private function getModel()
     private function getModelMock()
     {
         $modelMock = $this->getMockBuilder(\Magento\Framework\App\Response\Http\FileFactory::class)
-            ->setMethods(['callExit'])
+            ->setMethods(null)
             ->setConstructorArgs(
                 [
                     'response' => $this->responseMock,

Original file line number	Diff line number	Diff line change
`@@ -99,19 +99,7 @@ public function create(`
`99`	`99`	`if (!empty($content['rm'])) {`
`100`	`100`	`$dir->delete($file);`
`101`	`101`	`}`
`102`		`- $this->callExit();`
`103`	`102`	`}`
`104`	`103`	`return $this->_response;`
`105`	`104`	`}`
`106`		`-`
`107`		`- /**`
`108`		`- * Call exit`
`109`		`- *`
`110`		`- * @return void`
`111`		`- * @SuppressWarnings(PHPMD.ExitExpression)`
`112`		`- */`
`113`		`- protected function callExit()`
`114`		`- {`
`115`		`- exit(0);`
`116`		`- }`
`117`	`105`	`}`
Original file line number	Diff line number	Diff line change
`@@ -235,7 +235,7 @@ private function getModel()`
`235`	`235`	`private function getModelMock()`
`236`	`236`	`{`
`237`	`237`	`$modelMock = $this->getMockBuilder(\Magento\Framework\App\Response\Http\FileFactory::class)`
`238`		`- ->setMethods(['callExit'])`
	`238`	`+ ->setMethods(null)`
`239`	`239`	`->setConstructorArgs(`
`240`	`240`	`[`
`241`	`241`	`'response' => $this->responseMock,`