Skip to content

Commit

Permalink
minor #17913 [2.8] [Form] minor fix some regression in tests with pla…
Browse files Browse the repository at this point in the history
…ceholder in AbstractLayout (HeahDude)

This PR was merged into the 2.8 branch.

Discussion
----------

[2.8] [Form] minor fix some regression in tests with placeholder in AbstractLayout

| Q             | A
| ------------- | ---
| Bug fix?      | no
| New feature?  | no
| BC breaks?    | no
| Deprecations? | no
| Tests pass?   | yes
| Fixed tickets | n/a
| License       | MIT
| Doc PR        | -

Commits
-------

6977583 [2.8] [Form] minor fix some tests with placeholder in AbstractLayout
  • Loading branch information
fabpot committed Feb 26, 2016
2 parents 20d4c58 + 6977583 commit dcd4d2e
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 2 deletions.
Expand Up @@ -847,7 +847,8 @@ public function testSingleChoiceExpandedWithPlaceholderWithoutTranslation()
'choices_as_values' => true,
'multiple' => false,
'expanded' => true,
'translation_domain' => false,
'required' => false,
'choice_translation_domain' => false,
'placeholder' => 'Placeholder&Not&Translated',
));

Expand Down
4 changes: 3 additions & 1 deletion src/Symfony/Component/Form/Tests/AbstractLayoutTest.php
Expand Up @@ -1016,6 +1016,7 @@ public function testSingleChoiceExpandedWithoutTranslation()
'multiple' => false,
'expanded' => true,
'choice_translation_domain' => false,
'placeholder' => 'Placeholder&Not&Translated',
));

$this->assertWidgetMatchesXpath($form->createView(), array(),
Expand Down Expand Up @@ -1092,7 +1093,8 @@ public function testSingleChoiceExpandedWithPlaceholderWithoutTranslation()
'choices_as_values' => true,
'multiple' => false,
'expanded' => true,
'translation_domain' => false,
'required' => false,
'choice_translation_domain' => false,
'placeholder' => 'Placeholder&Not&Translated',
));

Expand Down

0 comments on commit dcd4d2e

Please sign in to comment.