[PLA-1629] Fixes MarketplaceSale (#51)

database/migrations/2024_06_23_083100_nullable_listing_on_marketplace_sales_table.php

@@ -0,0 +1,27 @@
+<?php
+
+use Illuminate\Database\Migrations\Migration;
+use Illuminate\Database\Schema\Blueprint;
+use Illuminate\Support\Facades\Schema;
+
+return new class () extends Migration {
+    /**
+     * Run the migrations.
+     */
+    public function up(): void
+    {
+        Schema::table('marketplace_sales', function (Blueprint $table) {
+            $table->string('listing_chain_id')->nullable()->change();
+        });
+    }
+
+    /**
+     * Reverse the migrations.
+     */
+    public function down(): void
+    {
+        Schema::table('marketplace_sales', function (Blueprint $table) {
+            $table->string('listing_chain_id')->index()->change();
+        });
+    }
+};

src/GraphQL/Types/MarketplaceSaleType.php

@@ -44,7 +44,7 @@ public function fields(): array
                 'is_relation' => true,
             ],
             'listing' => [
-                'type' => GraphQL::type('MarketplaceListing!'),
+                'type' => GraphQL::type('MarketplaceListing'),
                 'description' => __('enjin-platform-marketplace::type.marketplace_listing.description'),
                 'is_relation' => true,
             ],

src/MarketplaceServiceProvider.php

@@ -21,6 +21,9 @@ public function configurePackage(Package $package): void
             ->hasMigration('create_marketplace_bids_table')
             ->hasMigration('create_marketplace_sales_table')
             ->hasMigration('create_marketplace_states_table')
+            ->hasMigration('add_listing_id_to_marketplace_sales_table')
+            ->hasMigration('drop_market_place_listing_id_in_marketplace_sales_table')
+            ->hasMigration('nullable_listing_on_marketplace_sales_table')
             ->hasTranslations();
     }
 

tests/Feature/GraphQL/Queries/GetSaleTest.php

@@ -4,6 +4,7 @@
 
 use Enjin\Platform\Marketplace\Tests\Feature\GraphQL\TestCaseGraphQL;
 use Enjin\Platform\Support\Hex;
+use Illuminate\Support\Facades\Schema;
 
 class GetSaleTest extends TestCaseGraphQL
 {
@@ -19,6 +20,23 @@ public function test_it_can_get_sale(): void
             $this->method,
             ['id' => $listing->sales->first()->id]
         );
+
+        $this->assertNotEmpty($response);
+    }
+
+    public function test_it_can_get_a_sale_with_no_listing(): void
+    {
+        Schema::disableForeignKeyConstraints();
+        $listing = $this->createListing();
+        $saleId = $listing->sales->first()->id;
+        $listing->delete();
+        Schema::enableForeignKeyConstraints();
+
+        $response = $this->graphql(
+            $this->method,
+            ['id' => $saleId]
+        );
+
         $this->assertNotEmpty($response);
     }