[HttpKernel] fixed file uploads in functional tests when no file was selected
Allow user to submit a form with no file selected.
This commit is contained in:
parent
48980b0a1c
commit
6f56d878c9
|
@ -143,7 +143,9 @@ EOF;
|
||||||
{
|
{
|
||||||
$httpRequest = Request::create($request->getUri(), $request->getMethod(), $request->getParameters(), $request->getCookies(), $request->getFiles(), $request->getServer(), $request->getContent());
|
$httpRequest = Request::create($request->getUri(), $request->getMethod(), $request->getParameters(), $request->getCookies(), $request->getFiles(), $request->getServer(), $request->getContent());
|
||||||
|
|
||||||
$httpRequest->files->replace($this->filterFiles($httpRequest->files->all()));
|
foreach ($this->filterFiles($httpRequest->files->all()) as $key => $value) {
|
||||||
|
$httpRequest->files->set($key, $value);
|
||||||
|
}
|
||||||
|
|
||||||
return $httpRequest;
|
return $httpRequest;
|
||||||
}
|
}
|
||||||
|
@ -189,8 +191,6 @@ EOF;
|
||||||
true
|
true
|
||||||
);
|
);
|
||||||
}
|
}
|
||||||
} else {
|
|
||||||
$filtered[$key] = $value;
|
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|
|
@ -143,6 +143,21 @@ class ClientTest extends \PHPUnit_Framework_TestCase
|
||||||
unlink($target);
|
unlink($target);
|
||||||
}
|
}
|
||||||
|
|
||||||
|
public function testUploadedFileWhenNoFileSelected()
|
||||||
|
{
|
||||||
|
$kernel = new TestHttpKernel();
|
||||||
|
$client = new Client($kernel);
|
||||||
|
|
||||||
|
$file = array('tmp_name' => '', 'name' => '', 'type' => '', 'size' => 0, 'error' => UPLOAD_ERR_NO_FILE);
|
||||||
|
|
||||||
|
$client->request('POST', '/', array(), array('foo' => $file));
|
||||||
|
|
||||||
|
$files = $client->getRequest()->files->all();
|
||||||
|
|
||||||
|
$this->assertCount(1, $files);
|
||||||
|
$this->assertNull($files['foo']);
|
||||||
|
}
|
||||||
|
|
||||||
public function testUploadedFileWhenSizeExceedsUploadMaxFileSize()
|
public function testUploadedFileWhenSizeExceedsUploadMaxFileSize()
|
||||||
{
|
{
|
||||||
$source = tempnam(sys_get_temp_dir(), 'source');
|
$source = tempnam(sys_get_temp_dir(), 'source');
|
||||||
|
|
Reference in New Issue