使用 findOrFail 时获取“尝试获取非对象的属性'小计'”

Question

我在服务中有以下代码：

public function add(array $data) : Order

    {

        // retrieve item data

        $item       = MenuItem::findOrFail($data[OrderItem::ORDER_ITEM_ITEM_ID]);

        $quantity   = $data[OrderItem::ORDER_ITEM_QUANTITY];

        $order_no   = $data[Order::ORDER_NO] ?? null;

        $session_id = $data[Order::ORDER_SESSION_ID];

        $order = $order_no ? Order::findOrFail($order_no) : $this->createOrder([Order::ORDER_SESSION_ID => $session_id]);

        $order_item = $this->order_item->createOrderItem($order->order_no, $item, $quantity);

        // update order total

        $order->subtotal += $order_item->subtotal;

        $order->total    += $order_item->subtotal;

        $order->update();

        return $order;

    }

运行测试时，我得到：

获取“尝试获取非对象的属性'小计'”

检索到的对象不为空，否则它将失败。但是，小计和总计都不属于对象。如果我打印对象，我会得到除这两个字段之外的其他字段。它们都在我的数组中，我在迁移中定义了一个。$fillable->default(0);

这是怎么回事？

Answer 1

根据我们在评论中的对话，您在函数中遇到的问题之一，并因此导致测试中的问题，是一个依赖项，其中每次都无法保证结果。也就是说，对函数中另一个服务的调用不保证提供特定的输出。add把你的测试想象成这样。我从零个项目和零小计的订单开始我想向该订单添加商品我想以一个包含一个项目的订单和一个十美元的小计结束在此测试中，您不关心订单项的创建方式，您只关心它是否具有特定值，以便将其添加到订单中时，最终结果是您期望的。在这种情况下，您希望成为一个模拟对象，然后您可以模拟调用以返回已知订单对象。向您展示一个完整的解决方案有点困难，因为它可能涉及一些架构更改，但您的测试将如下所示：$this->order_itemcreateOrderItempublic function it_adds_an_item_to_an_order(){  $orderItem = Mock(OrderItem::class); // Use whatever mocking lib is available  $orderitem->shouldReceive('createOrderItem').andReturn(new OrderItem([$subtotal=>10.00]));  $orderService = new OrderService($orderItem);  $order = $orderService->add($data);  $this->assertNotNull($order);  $this->assertEquals(10.00, $order->subtotal);}