🔥 Do not return key field if there is no primary key (#85)

* Do not return key field if there is no primary key

* Add tests for pivots without primary key

Author: erikgaal

src/Eloquent/ModelSchema.php

@@ -209,6 +209,10 @@ protected function getKeyField(): array
     {
         $key = $this->instance->getKeyName();
 
+        if (! $key) {
+            return [];
+        }
+
         return [$key => $this->registry->field($this->registry->ID())->fillable(false)->unique()];
     }
 

tests/Feature/AttachPivotMutationTest.php

@@ -49,6 +49,11 @@ public function it_lets_you_attach_pivot_ids_with_pivot_data()
                     { id: "'.$role->id.'", customPivot: { comment: "foobar" } }
                 ]) {
                     id
+                    customRoles {
+                        customPivot {
+                            comment
+                        }
+                    }
                 }
             }
         ';

tests/Stubs/Models/UserRole.php

@@ -6,6 +6,8 @@
 
 class UserRole extends Pivot
 {
+    protected $primaryKey = null;
+
     public $fillable = [
         'comment',
     ];

tests/migrations/0000_00_00_000005_create_role_user_table.php

@@ -11,7 +11,6 @@ class CreateRoleUserTable extends Migration
     public function up()
     {
         Schema::create('role_user', function ($table) {
-            $table->increments('id');
             $table->unsignedInteger('user_id');
             $table->unsignedInteger('role_id');
             $table->text('comment')->nullable();