Skip to content

Commit 1eed064

Browse files
committed
Use getKey() method instead of hardcoded id property in tests
Replace hardcoded ->id access with proper Eloquent ->getKey() method for better compatibility with models that use non-standard primary keys
1 parent 44a3b73 commit 1eed064

File tree

1 file changed

+5
-5
lines changed

1 file changed

+5
-5
lines changed

tests/Feature/DragDropFunctionalityTest.php

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,7 @@
1717
expect($task->status)->toBe('todo');
1818

1919
Livewire::test(TestBoard::class)
20-
->call('updateRecordsOrderAndColumn', 'in_progress', [$task->id])
20+
->call('updateRecordsOrderAndColumn', 'in_progress', [$task->getKey()])
2121
->assertReturned(true);
2222

2323
$task->refresh();
@@ -29,7 +29,7 @@
2929
$task2 = Task::where('title', 'Task 2')->first();
3030

3131
Livewire::test(TestBoard::class)
32-
->call('updateRecordsOrderAndColumn', 'todo', [$task2->id, $task1->id]);
32+
->call('updateRecordsOrderAndColumn', 'todo', [$task2->getKey(), $task1->getKey()]);
3333

3434
$task1->refresh();
3535
$task2->refresh();
@@ -90,9 +90,9 @@
9090

9191
$component = Livewire::test(TestBoard::class);
9292

93-
$component->call('getBoardRecord', $task->id);
93+
$component->call('getBoardRecord', $task->getKey());
9494

95-
$result = $component->instance()->getBoardRecord($task->id);
95+
$result = $component->instance()->getBoardRecord($task->getKey());
9696
expect($result)->toBeInstanceOf(Task::class);
97-
expect($result->id)->toBe($task->id);
97+
expect($result->getKey())->toBe($task->getKey());
9898
});

0 commit comments

Comments
 (0)