[FrameworkBundle] fix AbstractController::handleForm()

Author: nicolas-grekas

Controller/AbstractController.php

@@ -314,7 +314,7 @@ public function handleForm(FormInterface $form, Request $request, callable $onSu
         } else {
             $response = $render($form, $data, $request);
 
-            if ($submitted && 200 === $response->getStatusCode()) {
+            if ($response instanceof Response && $submitted && 200 === $response->getStatusCode()) {
                 $response->setStatusCode(Response::HTTP_UNPROCESSABLE_ENTITY);
             }
         }

Tests/Controller/AbstractControllerTest.php

@@ -494,12 +494,12 @@ public function testHandleFormTypeError()
     {
         $form = $this->createMock(FormInterface::class);
         $form->expects($this->once())->method('isSubmitted')->willReturn(true);
-        $form->expects($this->once())->method('isValid')->willReturn(true);
+        $form->expects($this->once())->method('isValid')->willReturn(false);
 
         $controller = $this->createController();
 
         $this->expectException(\TypeError::class);
-        $this->expectExceptionMessage('The "$onSuccess" callable passed to "Symfony\Bundle\FrameworkBundle\Tests\Controller\TestAbstractController::handleForm()" must return a Response, "string" returned.');
+        $this->expectExceptionMessage('The "$render" callable passed to "Symfony\Bundle\FrameworkBundle\Tests\Controller\TestAbstractController::handleForm()" must return a Response, "string" returned.');
 
         $response = $controller->handleForm(
             $form,