Дмитрий
2ec70b338f
Accessibility (Pa11y live) / a11y (push) Has been cancelled
Закрыто 36 из 55 пре-существующих падений backend-набора (55 to 19), всё тест-сторона, код продукта не тронут. Группы: - incident-показ/РКН: добавлен SharesSupplierPdo + синхрон уровня транзакции в трейте (вложенный transaction на общем PDO теперь делает SAVEPOINT, не повторный BEGIN). - auto-pause и lead-delivery: тесты создают project_routing_snapshots, от которого зависит выбор кандидатов в LeadRouter (slepok-инвариант). - изоляция 16 протекающих тестов: добавлен DatabaseTransactions (где нужно плюс SharesSupplierPdo) — перестали оставлять committed-строки, отравлявшие глобально сканирующие тесты (snapshot, verify-audit, size-N). - partition time-bombs: ensureRange месячных партиций для тестов на дату 2026-05. - устаревшие ассерты: SchemaDelta метрики v8.35 to v8.52, ProjectsStore телефон 8 to 7 нормализуется, incidents-watch фильтр активного admin, register captcha_token, impersonation активный юзер тенанта, activity_log.deal_id, ProjectUpdateDedup пауза. Остаток 19 (отдельно): verify-audit-chains и size-N (протекатели audit-строк), webhook B-префикс (решение владельца), пара env/каскадных. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
51 lines
1.8 KiB
PHP
51 lines
1.8 KiB
PHP
<?php
|
|
|
|
declare(strict_types=1);
|
|
|
|
use App\Models\Project;
|
|
use App\Models\Tenant;
|
|
use App\Services\Project\ProjectService;
|
|
use Illuminate\Http\Exceptions\HttpResponseException;
|
|
use Illuminate\Support\Facades\Queue;
|
|
|
|
uses(\Illuminate\Foundation\Testing\DatabaseTransactions::class);
|
|
|
|
beforeEach(function () {
|
|
Queue::fake();
|
|
$this->tenant = Tenant::factory()->create(['balance_leads' => 100]);
|
|
});
|
|
|
|
function makeCall(array $over = []): array
|
|
{
|
|
return array_merge([
|
|
'name' => 'Проект A', 'signal_type' => 'call', 'signal_identifier' => '79991110000',
|
|
'daily_limit_target' => 5, 'regions' => [], 'delivery_days_mask' => 31,
|
|
], $over);
|
|
}
|
|
|
|
it('blocks duplicate source within tenant with human message', function () {
|
|
app(ProjectService::class)->create($this->tenant, makeCall());
|
|
expect(fn () => app(ProjectService::class)
|
|
->create($this->tenant, makeCall(['name' => 'Проект B'])))
|
|
->toThrow(HttpResponseException::class);
|
|
});
|
|
|
|
it('allows same source for a different tenant (sharing)', function () {
|
|
$other = Tenant::factory()->create(['balance_leads' => 100]);
|
|
app(ProjectService::class)->create($this->tenant, makeCall());
|
|
$p = app(ProjectService::class)->create($other, makeCall(['name' => 'Проект B']));
|
|
expect($p)->toBeInstanceOf(Project::class);
|
|
});
|
|
|
|
it('blocks duplicate name within tenant with human message (not SQL)', function () {
|
|
app(ProjectService::class)->create($this->tenant, makeCall());
|
|
try {
|
|
app(ProjectService::class)
|
|
->create($this->tenant, makeCall(['name' => 'Проект A', 'signal_identifier' => '79992220000']));
|
|
$this->fail('expected HttpResponseException');
|
|
} catch (HttpResponseException $e) {
|
|
$body = $e->getResponse()->getData(true);
|
|
expect($body['errors']['name'][0] ?? '')->not->toContain('SQLSTATE');
|
|
}
|
|
});
|