Skip to content

Commit 77d9d2e

Browse files
committed
Updated to follow @schmengler's recommendation
1 parent 727439a commit 77d9d2e

File tree

2 files changed

+12
-19
lines changed

2 files changed

+12
-19
lines changed

setup/src/Magento/Setup/Module/Di/Code/Reader/ClassesScanner.php

Lines changed: 5 additions & 18 deletions
Original file line numberDiff line numberDiff line change
@@ -33,28 +33,16 @@ class ClassesScanner implements ClassesScannerInterface
3333
* @param string $generationDirectory
3434
*/
3535

36-
public function __construct(array $excludePatterns = [], $generationDirectory = null)
36+
public function __construct(array $excludePatterns = [], DirectoryList $directoryList = null)
3737
{
3838
$this->excludePatterns = $excludePatterns;
39-
$this->generationDirectory = $generationDirectory;
40-
}
41-
42-
/**
43-
* Retrieves the fully qualified path for var/generation.
44-
*
45-
* @return string
46-
*/
47-
48-
public function getGenerationDirectory()
49-
{
50-
if ($this->generationDirectory === null) {
39+
if (!$directoryList instanceof DirectoryList) {
5140
$directoryList = ObjectManager::getInstance()->get(DirectoryList::class);
52-
/* @var $directoryList DirectoryList */
53-
$this->generationDirectory = $directoryList->getPath(DirectoryList::GENERATION);
5441
}
55-
return $this->generationDirectory;
42+
$this->generationDirectory = $directoryList->getPath(DirectoryList::GENERATION);
5643
}
5744

45+
5846
/**
5947
* Adds exclude patterns
6048
*
@@ -75,8 +63,7 @@ public function addExcludePatterns(array $excludePatterns)
7563

7664
public function isGeneration($path)
7765
{
78-
$generation = $this->getGenerationDirectory();
79-
return strpos($path, $generation) === 0;
66+
return strpos($path, $this->generationDirectory) === 0;
8067
}
8168

8269
/**

setup/src/Magento/Setup/Test/Unit/Module/Di/Code/Reader/ClassesScannerTest.php

Lines changed: 7 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5,6 +5,8 @@
55
*/
66
namespace Magento\Setup\Test\Unit\Module\Di\Code\Reader;
77

8+
use Magento\Framework\App\Filesystem\DirectoryList;
9+
810
class ClassesScannerTest extends \PHPUnit_Framework_TestCase
911
{
1012
/**
@@ -23,7 +25,11 @@ class ClassesScannerTest extends \PHPUnit_Framework_TestCase
2325
protected function setUp()
2426
{
2527
$this->generation = realpath(__DIR__ . '/../../_files/var/generation');
26-
$this->model = new \Magento\Setup\Module\Di\Code\Reader\ClassesScanner([], $this->generation);
28+
$mock = $this->getMockBuilder(DirectoryList::class)->disableOriginalConstructor()->setMethods(
29+
['getPath']
30+
)->getMock();
31+
$mock->method('getPath')->willReturn($this->generation);
32+
$this->model = new \Magento\Setup\Module\Di\Code\Reader\ClassesScanner([], $mock);
2733
}
2834

2935
public function testGetList()

0 commit comments

Comments
 (0)