Merge pull request #979 from pborreli/patch-1
[InstallerTest] Fixed return status was always 0pull/975/merge
commit
35561767fa
|
@ -197,7 +197,7 @@ class InstallerTest extends TestCase
|
||||||
$application->get('install')->setCode(function ($input, $output) use ($installer) {
|
$application->get('install')->setCode(function ($input, $output) use ($installer) {
|
||||||
$installer->setDevMode($input->getOption('dev'));
|
$installer->setDevMode($input->getOption('dev'));
|
||||||
|
|
||||||
return $installer->run();
|
return $installer->run() ? 0 : 1;
|
||||||
});
|
});
|
||||||
|
|
||||||
$application->get('update')->setCode(function ($input, $output) use ($installer) {
|
$application->get('update')->setCode(function ($input, $output) use ($installer) {
|
||||||
|
@ -206,7 +206,7 @@ class InstallerTest extends TestCase
|
||||||
->setUpdate(true)
|
->setUpdate(true)
|
||||||
->setUpdateWhitelist($input->getArgument('packages'));
|
->setUpdateWhitelist($input->getArgument('packages'));
|
||||||
|
|
||||||
return $installer->run();
|
return $installer->run() ? 0 : 1;
|
||||||
});
|
});
|
||||||
|
|
||||||
if (!preg_match('{^(install|update)\b}', $run)) {
|
if (!preg_match('{^(install|update)\b}', $run)) {
|
||||||
|
|
Loading…
Reference in New Issue