You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
minor symfony#16446 OptionsResolver test coverage (eventhorizonpl)
This PR was submitted for the 2.8 branch but it was merged into the 2.7 branch instead (closessymfony#16446).
Discussion
----------
OptionsResolver test coverage
Hi,
This PR adds 100% test code coverage to OptionsResolver component.
Best regards,
Michal
| Q | A
| ------------- | ---
| Bug fix? | no
| New feature? | no
| BC breaks? | no
| Deprecations? | no
| Tests pass? | yes
| License | MIT
Commits
-------
185950c OptionsResolver test coverage
array(true, 'string', 'The option "option" with value true is expected to be of type "string", but is of type "boolean".'),
516
+
array(false, 'string', 'The option "option" with value false is expected to be of type "string", but is of type "boolean".'),
517
+
array(fopen(__FILE__, 'r'), 'string', 'The option "option" with value resource is expected to be of type "string", but is of type "resource".'),
518
+
array(array(), 'string', 'The option "option" with value array is expected to be of type "string", but is of type "array".'),
519
+
array(newOptionsResolver(), 'string', 'The option "option" with value Symfony\Component\OptionsResolver\OptionsResolver is expected to be of type "string", but is of type "Symfony\Component\OptionsResolver\OptionsResolver".'),
520
+
array(42, 'string', 'The option "option" with value 42 is expected to be of type "string", but is of type "integer".'),
521
+
array(null, 'string', 'The option "option" with value null is expected to be of type "string", but is of type "NULL".'),
522
+
array('bar', '\stdClass', 'The option "option" with value "bar" is expected to be of type "\stdClass", but is of type "string".'),
523
+
);
523
524
}
524
525
525
526
publicfunctiontestResolveSucceedsIfValidType()
@@ -550,17 +551,6 @@ public function testResolveSucceedsIfValidTypeMultiple()
0 commit comments