Skip to content

Commit 0a37095

Browse files
eddielauOlga Kopylova
authored andcommitted
MAGETWO-33157: Conflict restriction checking is missing version checking in enable/disable module CLI
- added unit test
1 parent f42716b commit 0a37095

File tree

1 file changed

+12
-3
lines changed

1 file changed

+12
-3
lines changed

dev/tests/unit/testsuite/Magento/Framework/Module/PackageInfoTest.php

Lines changed: 12 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -32,10 +32,10 @@ public function setUp()
3232

3333
$composerData = [
3434
'A' => '{"name":"a", "require":{"b":"0.1"}, "conflict":{"c":"0.1"}, "version":"0.1"}',
35-
'B' => '{"name":"b", "require":{"d":"0.1"}, "version":"0.1"}',
35+
'B' => '{"name":"b", "require":{"d":"0.3"}, "version":"0.2"}',
3636
'C' => '{"name":"c", "require":{"e":"0.1"}, "version":"0.1"}',
37-
'D' => '{"name":"d", "conflict":{"c":"0.1"}, "version":"0.1"}',
38-
'E' => '{"name":"e", "version":"0.1"}',
37+
'D' => '{"name":"d", "conflict":{"c":"0.1"}, "version":"0.3"}',
38+
'E' => '{"name":"e", "version":"0.4"}',
3939
];
4040
$fileIteratorMock = $this->getMock('Magento\Framework\Config\FileIterator', [], [], '', false);
4141
$fileIteratorMock->expects($this->once())
@@ -92,4 +92,13 @@ public function testGetConflictReturnPackageName()
9292
$this->assertEquals(['c' => '0.1'], $this->packageInfo->getConflict('D', false));
9393
$this->assertEquals([], $this->packageInfo->getConflict('E', false));
9494
}
95+
96+
public function testGetVersion()
97+
{
98+
$this->assertEquals('0.1', $this->packageInfo->getVersion('A'));
99+
$this->assertEquals('0.2', $this->packageInfo->getVersion('B'));
100+
$this->assertEquals('0.1', $this->packageInfo->getVersion('C'));
101+
$this->assertEquals('0.3', $this->packageInfo->getVersion('D'));
102+
$this->assertEquals('0.4', $this->packageInfo->getVersion('E'));
103+
}
95104
}

0 commit comments

Comments
 (0)