Skip to content

Commit 6e20892

Browse files
committed
minor #15955 [Form] Minor fixes (sebpacz)
This PR was merged into the 4.4 branch. Discussion ---------- [Form] Minor fixes I found two comments that point to the wrong variable. <!-- If your pull request fixes a BUG, use the oldest maintained branch that contains the bug (see https://symfony.com/releases for the list of maintained branches). If your pull request documents a NEW FEATURE, use the same Symfony branch where the feature was introduced (and `5.x` for features of unreleased versions). --> Commits ------- 9cb23a5 [Form] Minor fixes
2 parents 56b2c6b + 9cb23a5 commit 6e20892

File tree

1 file changed

+3
-3
lines changed

1 file changed

+3
-3
lines changed

form/unit_testing.rst

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -55,7 +55,7 @@ The simplest ``TypeTestCase`` implementation looks like the following::
5555
];
5656

5757
$model = new TestObject();
58-
// $formData will retrieve data from the form submission; pass it as the second argument
58+
// $model will retrieve data from the form submission; pass it as the second argument
5959
$form = $this->factory->create(TestedType::class, $model);
6060

6161
$expected = new TestObject();
@@ -67,7 +67,7 @@ The simplest ``TypeTestCase`` implementation looks like the following::
6767
// This check ensures there are no transformation failures
6868
$this->assertTrue($form->isSynchronized());
6969

70-
// check that $formData was modified as expected when the form was submitted
70+
// check that $model was modified as expected when the form was submitted
7171
$this->assertEquals($expected, $model);
7272
}
7373

@@ -88,7 +88,7 @@ The simplest ``TypeTestCase`` implementation looks like the following::
8888
So, what does it test? Here comes a detailed explanation.
8989

9090
First you verify if the ``FormType`` compiles. This includes basic class
91-
inheritance, the ``buildForm()`` function and options resolution. This should
91+
inheritance, the ``buildForm()`` method and options resolution. This should
9292
be the first test you write::
9393

9494
$form = $this->factory->create(TestedType::class, $formData);

0 commit comments

Comments
 (0)