-1

我有这些迁移:

class CreateOrdersTable extends Migration
{
    public function up()
    {
        Schema::create('orders', function (Blueprint $table) {
            $table->bigIncrements('id');
            $table->unsignedBigInteger('user_id');
            $table->timestamps();

            $table->foreign('user_id')->references('id')->on('users');
        });
    }
}
class CreatePaymentsTable extends Migration
{
    public function up()
    {
        Schema::create('payments', function (Blueprint $table) {
            $table->bigIncrements('id');
            $table->unsignedBigInteger('order_id');
            $table->timestamps();

            $table->foreign('order_id')->references('id')->on('orders');
        });
    }
}

这些工厂:

$factory->define(Payment::class, function (Faker $faker) {
    return [
        'order_id' => factory(Order::class)->create(),
    ];
});
$factory->define(Order::class, function (Faker $faker) {
    return [
        'user_id' => factory(User::class)->create(),
    ];
});

现在在我的测试中我有这个:

/** @test */
public function it_should_count_1_order()
{
     $order = factory(Order::class)->create();

     $payment = factory(Payment::class)->create([
         'order_id' => $order->id,
     ]);

     $this->assertEquals(1, Order::count())
}

Order 表计数给了我2。为什么?应该是1因为我告诉支付工厂order_id用给定的订单覆盖。我错过了什么吗?

4

2 回答 2

1

由于您Payment factory正在创建一个Order,因此您不需要在Order之前创建一个并将其传递给Payment factory.

/** @test */
public function it_should_count_1_order()
{
     $payment = factory(Payment::class)->create();

     $this->assertEquals(1, Order::count())
}
于 2020-01-09T18:35:50.597 回答
0

您应该尝试像这样定义您的工厂:

$factory->define(Order::class, function (Faker $faker) {
    return [
        'user_id' => factory(User::class),
});

如果它没有帮助,请确保您在测试DatabaseTransactionsDatabseMigrations测试中使用并在第一次运行测试之前清除数据库。否则每次运行测试时,您都会在数据库中获得越来越多的记录。

就个人而言,我认为使用这样的断言:

$this->assertEquals(1, Order::count())

在测试中不是一个好主意。当您创建更高级的测试时,您可能会决定创建一些其他附加数据,并且您不应该关心之前创建的模型。我个人希望这样的测试:

$initialOrdersCount = Order::count();

// here you run some actions you want to test

$this->assertSame($initialOrdersCount  + 1, Order::count());

这样,如果应用程序中发生任何其他事情或将被添加到测试中,我不在乎我是否在数据库 2 或 3 模型中。对我来说,重要的是数量或订单应该增加一。

于 2020-01-09T19:54:35.233 回答