Skip to content

Commit 7ee7473

Browse files
committed
fix delete visits-monitoring route delete visit monitoring and redirect test
1 parent 430db98 commit 7ee7473

File tree

2 files changed

+5
-6
lines changed

2 files changed

+5
-6
lines changed

src/Controllers/VisitMonitoringController.php

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -13,9 +13,9 @@ public function index()
1313
return view('LaravelUserMonitoring::visits-monitoring.index', compact('visits'));
1414
}
1515

16-
public function destroy(VisitMonitoring $visitMonitoring)
16+
public function destroy(int $id)
1717
{
18-
$visitMonitoring->delete();
18+
VisitMonitoring::query()->findOrFail($id)->delete();
1919

2020
// TODO: Add alert
2121
return to_route('user-monitoring.visits-monitoring');

tests/Feature/VisitMonitoringControllerTest.php

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -16,13 +16,12 @@
1616
});
1717

1818
test('delete visits-monitoring route delete visit monitoring and redirect', function () {
19-
get(route('user-monitoring.actions-monitoring'));
19+
get(route('user-monitoring.visits-monitoring'));
2020

21-
$visitMonitoring = VisitMonitoring::query()->first();
22-
$response = delete(route('user-monitoring.visits-monitoring-delete', $visitMonitoring->id));
21+
$response = delete(route('user-monitoring.visits-monitoring-delete', 1));
2322
$response->assertRedirect(route('user-monitoring.visits-monitoring'));
2423

2524
// DB Assertions
2625
assertDatabaseCount(config('user-monitoring.visit_monitoring.table'), 1);
27-
assertDatabaseMissing(config('user-monitoring.visit_monitoring.table'), ['page' => url('/')]);
26+
assertDatabaseMissing(config('user-monitoring.visit_monitoring.table'), ['page' => route('user-monitoring.visits-monitoring')]);
2827
});

0 commit comments

Comments
 (0)