PlannrCrm / laravel-fast-refresh-database

🚀 Refresh your test databases faster than you've ever seen before
MIT License
415 stars 18 forks source link

RefreshDatabase and FastRefreshDatabase is not workable with Laravel Dusk #6

Closed dr41d45 closed 1 year ago

dr41d45 commented 1 year ago

I would avoid the __DIR__ variable in the Pest file https://github.com/PlannrCrm/laravel-fast-refresh-database/blob/55052efd70ea24fc29ba530bf2dd9b68e97592cd/.github/README.md?plain=1#L57 due the RefreshDatabase trait is not compatible with Laravel Dusk, which uses only the DatabaseMigrations trait. Source: https://laravel.com/docs/9.x/dusk#migrations. My suggestion to document: uses(FastRefreshDatabase::class)->in('Feature'); instead of uses(FastRefreshDatabase::class)->in(__DIR__);

Sammyjo20 commented 1 year ago

That's a good shout, I thnk it might be good to note this in the documentation?

craigpotter commented 1 year ago

Closing this issue as #13 was merged.