minor #13716 [Console] make an assertion more precise in testGetOptionDefaults (nanocom)
This PR was merged into the 2.3 branch.
Discussion
----------
[Console] make an assertion more precise in testGetOptionDefaults
| Q | A
| ------------- | ---
| Bug fix? | [no]
| New feature? | no]
| BC breaks? | [no]
| Deprecations? | [no]
| Tests pass? | [yes]
| License | MIT
Minor fix, the default values expected in the test were not exactly the same as what is actually returned.
Commits
-------
06a42e1
[Console] fix assertion in testGetOptionDefaults
This commit is contained in:
commit
1cc1d8d12e
@ -341,7 +341,7 @@ class InputDefinitionTest extends \PHPUnit_Framework_TestCase
|
|||||||
new InputOption('foo7', null, InputOption::VALUE_OPTIONAL | InputOption::VALUE_IS_ARRAY, '', array(1, 2)),
|
new InputOption('foo7', null, InputOption::VALUE_OPTIONAL | InputOption::VALUE_IS_ARRAY, '', array(1, 2)),
|
||||||
));
|
));
|
||||||
$defaults = array(
|
$defaults = array(
|
||||||
'foo1' => null,
|
'foo1' => false,
|
||||||
'foo2' => null,
|
'foo2' => null,
|
||||||
'foo3' => 'default',
|
'foo3' => 'default',
|
||||||
'foo4' => null,
|
'foo4' => null,
|
||||||
@ -349,7 +349,7 @@ class InputDefinitionTest extends \PHPUnit_Framework_TestCase
|
|||||||
'foo6' => array(),
|
'foo6' => array(),
|
||||||
'foo7' => array(1, 2),
|
'foo7' => array(1, 2),
|
||||||
);
|
);
|
||||||
$this->assertEquals($defaults, $definition->getOptionDefaults(), '->getOptionDefaults() returns the default values for all options');
|
$this->assertSame($defaults, $definition->getOptionDefaults(), '->getOptionDefaults() returns the default values for all options');
|
||||||
}
|
}
|
||||||
|
|
||||||
public function testGetSynopsis()
|
public function testGetSynopsis()
|
||||||
|
Reference in New Issue
Block a user