-
Notifications
You must be signed in to change notification settings - Fork 26
Conversation
5742ee6
to
037393b
Compare
* @dataProvider unexpectedConfigValueDataProvider | ||
*/ | ||
public function testUnexpectedValueConfig($configs) | ||
{ | ||
$parameterBag = new ParameterBag(array('kernel.debug' => true)); | ||
$container = new ContainerBuilder($parameterBag); | ||
|
||
$this->if($extension = new TestedClass()) | ||
->exception(function() use($extension, $configs, $container) { | ||
$this->if ($extension = new TestedClass()) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
if ()
=> it's weird to have a space between a methodName and its bracket
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
bracket ? {
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
[ ] => Brackets
( ) => Round brackets or parentheses
{ } => Curly brackets or braces
〈 〉=> Angle brackets or chevrons
bracket
works for everything
@@ -236,16 +278,20 @@ public function testInvalidConfig() | |||
$parameterBag = new ParameterBag(array('kernel.debug' => true)); | |||
$container = new ContainerBuilder($parameterBag); | |||
|
|||
$this->if($extension = new TestedClass()) | |||
->exception(function() use($extension, $configs, $container) { | |||
$this->if ($extension = new TestedClass()) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
same
037393b
to
f4c1b66
Compare
@@ -213,13 +252,16 @@ public function testUnexpectedValueConfig($configs) | |||
$container = new ContainerBuilder($parameterBag); | |||
|
|||
$this->if($extension = new TestedClass()) | |||
->exception(function() use($extension, $configs, $container) { | |||
->exception(function () use ($extension, $configs, $container) { | |||
$extension->load($configs, $container); | |||
}) | |||
->isInstanceOf('\Symfony\Component\Config\Definition\Exception\InvalidConfigurationException'); | |||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Petite ligne en trop ?
f4c1b66
to
c95374c
Compare
Separation between client configuration and connection configuration (with keyspaces defined).
We do not have to define a keyspace for a new client.