test: cover case-sensitive application URL paths
This commit is contained in:
parent
bbff70c8d0
commit
cd95ac3d0d
2 changed files with 44 additions and 0 deletions
37
tests/Unit/ServicesControllerUrlDuplicateTest.php
Normal file
37
tests/Unit/ServicesControllerUrlDuplicateTest.php
Normal file
|
|
@ -0,0 +1,37 @@
|
|||
<?php
|
||||
|
||||
use App\Http\Controllers\Api\ServicesController;
|
||||
use App\Models\Service;
|
||||
|
||||
it('does not treat service URLs with different path casing as duplicates', function () {
|
||||
$controller = new ServicesController;
|
||||
$method = new ReflectionMethod($controller, 'applyServiceUrls');
|
||||
$service = new class extends Service
|
||||
{
|
||||
public function applications()
|
||||
{
|
||||
return new class
|
||||
{
|
||||
public function where(string $column, mixed $value): self
|
||||
{
|
||||
return $this;
|
||||
}
|
||||
|
||||
public function first(): null
|
||||
{
|
||||
return null;
|
||||
}
|
||||
};
|
||||
}
|
||||
};
|
||||
|
||||
$result = $method->invoke($controller, $service, [
|
||||
['name' => 'web', 'url' => 'https://example.com/Route'],
|
||||
['name' => 'api', 'url' => 'HTTPS://EXAMPLE.COM/route'],
|
||||
], '1');
|
||||
|
||||
expect($result['errors'] ?? [])->toBe([
|
||||
"Service container with 'web' not found.",
|
||||
"Service container with 'api' not found.",
|
||||
]);
|
||||
});
|
||||
|
|
@ -180,3 +180,10 @@
|
|||
|
||||
expect($environmentVariable->key)->toBe('APP_ENV');
|
||||
});
|
||||
|
||||
it('normalizes application domain scheme and host without lowercasing path query or fragment', function () {
|
||||
$domains = ' HTTPS://EXAMPLE.COM/MixedCase/Path?Token=ABC#Fragment, http://Sub.EXAMPLE.com/Api/V1 ';
|
||||
|
||||
expect(ValidationPatterns::normalizeApplicationDomains($domains))
|
||||
->toBe('https://example.com/MixedCase/Path?Token=ABC#Fragment,http://sub.example.com/Api/V1');
|
||||
});
|
||||
|
|
|
|||
Loading…
Reference in a new issue