get bid for carrier returns a response object

src/Bid/UseCases/GetBidForCarrier.php

@@ -2,7 +2,6 @@
 
 namespace FreightQuote\Bid\UseCases;
 
-use FreightQuote\Bid\Bid;
 use FreightQuote\Bid\BidRepository;
 use InvalidArgumentException;
 
@@ -15,19 +14,26 @@ class GetBidForCarrier
     /**
      * @throws InvalidArgumentException
      */
-    public function execute(GetBidForCarrierRequest $dto): ?Bid
+    public function execute(
-    {
+        GetBidForCarrierRequest $dto,
+    ): GetBidForCarrierResponse {
         $bid = $this->bidRepo->find($dto->id);
         if ($bid === null) {
             throw new InvalidArgumentException('Bid not found!');
         }
         if ($bid->isClosed() === true) {
-            return null;
+            return new GetBidForCarrierResponse(
+                isClosed: true,
+                bid: null,
+            );
         }
         if ($bid->getWasOpened() === false) {
             $bid->setWasOpened(true);
             $this->bidRepo->save($bid);
         }
-        return $bid;
+        return new GetBidForCarrierResponse(
+            isClosed: false,
+            bid: $bid,
+        );
     }
 }

src/Bid/UseCases/GetBidForCarrierResponse.php

@@ -0,0 +1,13 @@
+<?php
+
+namespace FreightQuote\Bid\UseCases;
+
+use FreightQuote\Bid\Bid;
+
+class GetBidForCarrierResponse
+{
+    public function __construct(
+        public bool $isClosed,
+        public ?Bid $bid,
+    ) {}
+}

tests/Unit/Bid/UseCases/GetBidForCarrierTest.php

@@ -5,6 +5,7 @@ namespace Tests\Unit\Bid\UseCases;
 use FreightQuote\Bid\Bid;
 use FreightQuote\Bid\UseCases\GetBidForCarrier;
 use FreightQuote\Bid\UseCases\GetBidForCarrierRequest;
+use FreightQuote\Bid\UseCases\GetBidForCarrierResponse;
 use PHPUnit\Framework\TestCase;
 use InvalidArgumentException;
 use Tests\Fakes\Bid\FakeBidRepository;
@@ -41,13 +42,13 @@ class GetBidForCarrierTest extends TestCase
             null,
             null,
         ));
-        $foundBid = $this->useCase->execute(
+        $response = $this->useCase->execute(
             new GetBidForCarrierRequest($bidId)
         );
-        $this->assertEquals(true, $foundBid->getWasOpened());
+        $this->assertEquals(true, $response->bid->getWasOpened());
     }
 
-    public function test_getting_closed_bid_returns_null(): void
+    public function test_getting_closed_bid_returns_response_with_null_bid(): void
     {
         $this->bidRepo->save(new Bid(
             0,
@@ -59,8 +60,14 @@ class GetBidForCarrierTest extends TestCase
             null,
             null,
         ));
-        $this->assertNull($this->useCase->execute(
+        $response = $this->useCase->execute(
             new GetBidForCarrierRequest(0),
-        ));
+        );
+        $this->assertInstanceOf(
+            GetBidForCarrierResponse::class,
+            $response
+        );
+        $this->assertNull($response->bid);
+        $this->assertTrue($response->isClosed);
     }
 }