Laravel 5.2 |测试 UploadedFile 在发布后错过了 $test 值。漏洞？

Question

更新 2016/04/26 11:30 GMT+2 解决方法

从 Laravel 5.2.15 开始，去掉了 $test 参数，但是没有明确的原因，因为 Symfony 的 UploadedFile 仍然有 $test 参数。

一种解决方法是临时使用 Laravel 5.2.14。

2016/04/26 11:00 GMT+2 更新

Laravel 自己的 UploadedFile 没有通过 $test 参数。查看这些资源：

• https://github.com/laravel/framework/issues/12620
• https://github.com/laravel/framework/commit/5062a9b42632e55ee90b7397141c0b12622447e1

我知道，还有一个问题：How to test file upload in Laravel 5.2，但标记的答案对我不起作用。

测试用例

我创建了一个 Symfony 的 UploadedFile 类的实例，并将 $test 设置为 true。我将文件发布到file/upload。

class FileControllerTest extends TestCase
{
    use \Illuminate\Foundation\Testing\DatabaseTransactions;

    private $file;

    public function setUp()
    {
        parent::setUp();

        $this->file = new Symfony\Component\HttpFoundation\File\UploadedFile(
            public_path() . '/examples/example.jpg',
            'example.jpg',
            'image/jpeg',
            filesize(public_path() . '/examples/example.jpg'),
            null,
            true // for $test
        );
    }

    /** @test */
    public function it_uploads_a_valid_file()
    {
        var_dump($this->file); // $test = true
        $this->call('POST', 'file/upload', [], [], ['file' => $this->file],
            ['accept' => 'application/json']);

        $this->assertResponseOk();
    }
}

控制器

namespace App\Http\Controllers;

class FileController extends Controller
{
    public function upload(Request $request)
    {
        var_dump($request->file('file')); // $test = false

        return [];
    }
}

问题

• 要发布的文件有 true 的参数 $test
• 发布的文件到达upload()

• $request->file('file') 包含正确的参数，但是

  $test 是 false

似乎参数 $test 没有被 post 调用过去。这是一个错误吗？

Answer 1

说明

这真的很有趣。您在创建这篇文章时已经注意到了很多（如果有人有问题，请仔细阅读）。

在这个commit 中，正如你已经提到的，$testing 参数被删除，类的代码被简化，删除反射以获取Symfony\Component\HttpFoundation\File\UploadedFile 的testing 属性值。

现在棘手的事情是，根据您正在测试的内容，您可能不会注意到更改并且一切都可能正常工作，但在某些情况下它不会并且您不会真正知道原因。

例如，一切都可能正常工作 - 文件将毫无问题地上传，但如果您添加到您的请求类中，例如 mimes 规则，如下所示：

'logo' => ['mimes:jpeg,png'],

它会失败告诉你文件有无效的mime（这是因为在内部它也会验证文件是否真的上传，如果测试实际上它与真正的上传不一样）。

解决方案是再次查看提交中真正更改的内容以及方法的外观。在this file 上传文件的实例返回如下：

 return $file instanceof static ? $file : new static(
            $file->getRealPath(), $file->getClientOriginalName(), $file->getClientMimeType(),
            $file->getClientSize(), $file->getError()
        );

因此，如果文件是此类的实例，它将不加修改地返回此实例，否则它将立即创建对象而不将 $testing 参数传递给类构造函数。

解决方案

所以为了解决这个问题，在测试文件上传时你不应该使用

\Symfony\Component\HttpFoundation\File\UploadedFile

上课了。您现在应该使用

\Illuminate\Http\UploadedFile

在测试文件上传时不要出现任何奇怪的问题（当然你仍然应该将true作为$testing参数传递给这个对象构造函数，但现在它会在以后使用没有问题）