8000 [OptionsResolver] support array of instance validation by EVODelavega · Pull Request #17032 · symfony/symfony · GitHub
[go: up one dir, main page]
Skip to content

[OptionsResolver] support array of instance validation #17032

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
wants to merge 22 commits into from
Closed

[OptionsResolver] support array of instance validation #17032

Changes from 1 commit
Commits
Show all changes
22 commits
Select commit Hold shift + click to select a range
d4de104
Suggested fix for issue #15524
Sep 2, 2015
82f825c
Fix style issues (apply patch)
Sep 2, 2015
c42c1f4
Remove unused variable from method call
Sep 2, 2015
3609d44
Fix recursive call + add simple testcase
Sep 2, 2015
6b1c230
Fix coding style
Sep 2, 2015
0194c25
Add 2 successful testcases
Sep 2, 2015
ca21329
Fix coding style
Sep 2, 2015
6b639cf
Conform to coding standards (PSR-2 => elseif)
EVODelavega Sep 2, 2015
8ff6aac
Add early returns in response to review
EVODelavega Sep 2, 2015
4ab68e0
Add whitespace before return statement
EVODelavega Sep 2, 2015
2685d2e
Format complex types in exception message, too
EVODelavega Sep 8, 2015
1fc5ddb
Fix coding style
EVODelavega Sep 8, 2015
0368246
Fix typo
EVODelavega Sep 8, 2015
82d463f
Format the exception message clearer: in case complex typed arrays co…
EVODelavega Sep 8, 2015
5616371
Coding style -> new + ()
EVODelavega Sep 8, 2015
7c2613e
Add another test case, mainly to force another build
EVODelavega Sep 8, 2015
33b1e9c
Fix indentation
EVODelavega Sep 8, 2015
8000
4579f17
Add document feature in changelog
EVODelavega Sep 11, 2015
b189757
Use mb_substr instead of substr
EVODelavega Dec 16, 2015
84c8954
Update changelog -> merge master => target is now 3.1
EVODelavega Dec 16, 2015
c79c17b
Fix typo in docblock
EVODelavega Dec 18, 2015
c03ccdf
Fix dataprovider annotation -> rebase conflict
EVODelavega Dec 18, 2015
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Add 2 successful testcases
  • Loading branch information
@EVODelavega
Elias Van Ootegem authored and EVODelavega committed Dec 21, 2015
commit 0194c25a0e16715a9c9d70958d9cc346e2e45dd6
40 changes: 40 additions & 0 deletions src/Symfony/Component/OptionsResolver/Tests/OptionsResolverTest.php
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -565,6 +565,46 @@ public function testResolveSucceedsIfValidTypeMultiple()
$this->assertNotEmpty($this->resolver->resolve());
}

public function testResolveSucceedsIfTypedArray()
{
$this->resolver->setDefault('foo', null);
$this->resolver->setAllowedTypes('foo', array('null', '\DateTime[]'));

$data = array(
'foo' => array(
new \DateTime(),
new \DateTime(),
)
);
$result = $this->resolver->resolve($data);
$this->assertEquals($data, $result);
}

public function testResolveSucceedsNestedTypedArray()
{
$this->resolver->setDefault('foo', null);
$this->resolver->setAllowedTypes('foo', 'int[][]');

$expect = array(
'foo' => array(
range(1, 10),
),
);
$result = $this->resolver->resolve($expect);
$this->assertEquals($expect, $result);
}

/**
* @expectedException \Symfony\Component\OptionsResolver\Exception\InvalidOptionsException
*/
public function testResolveFailsIfNotInstanceOfClass()
{
$this->resolver->setDefault('foo', 'bar');
$this->resolver->setAllowedTypes('foo', '\stdClass');

$this->resolver->resolve();
}

public function testResolveSucceedsIfInstanceOfClass()
{
$this->resolver->setDefault('foo', new \stdClass());
Expand Down
0