archtechx/tenancy
1
0
Fork 0
mirror of https://github.com/archtechx/tenancy.git synced 2025-12-12 11:14:04 +00:00
Code Activity

Cloning: clarify case where neither paths nor domains differ

Browse source 
In such a case, the cloned route will actually *override* the original
route, rather than being unused as the original docblock claimed.

Also adds a static make() function for convenience.
This commit is contained in:
Samuel Štancl 2025-11-08 18:39:28 +01:00
parent 69bf768424
commit 97c5afd2cf
No known key found for this signature in database
GPG key ID: BA146259A1E16C57
2 changed files with 21 additions and 1 deletions
Download patch file Download diff file Expand all files Collapse all files

15
tests/CloneActionTest.php
View file

@ -422,3 +422,18 @@ test('existing context flags are removed during cloning', function () {
expect(tenancy()->getRouteMiddleware(RouteFacade::getRoutes()->getByName('tenant.foo')))
->not()->toContain('universal');
});
test('cloning a route without a prefix or differing domains overrides the original route', function () {
RouteFacade::get('/foo', fn () => true)->name('foo')->middleware(['clone']);
expect(collect(RouteFacade::getRoutes()->get())->map->getName())->toContain('foo');
$cloneAction = CloneRoutesAsTenant::make();
$cloneAction->cloneRoute('foo')
->addTenantParameter(false)
->tenantParameterBeforePrefix(false)
->handle();
expect(collect(RouteFacade::getRoutes()->get())->map->getName())->toContain('tenant.foo');
expect(collect(RouteFacade::getRoutes()->get())->map->getName())->not()->toContain('foo');
});