Дмитрий
a711d474d8
Пополнение баланса больше не оставляет проекты заблокированными навечно. Новый ProjectBlockReleaseService: после зачисления (единая точка BillingTopupService::topup — и ручное пополнение, и онлайн через PaymentWebhookController) проверяет, хватает ли баланса на суммарный дневной лимит ВСЕХ активных проектов тенанта, включая заблокированные. Хватает → снимает preflight_blocked_at со всех + диспатчит SyncSupplierProjectJob; не хватает → не трогает никого и возвращает дефицит (политика всё-или-ничего, решение владельца). Зеркалит BalancePreflightService и фильтр sweep. TDD: 4 теста (release при покрытии, удержание при нехватке, всё-или-ничего на двух проектах, no-op без заблокированных). Регрессия billing 114/114. larastan/deptrac исключены точечно — пред-существующая краснота. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
97 lines
4.0 KiB
PHP
97 lines
4.0 KiB
PHP
<?php
|
|
|
|
declare(strict_types=1);
|
|
|
|
use App\Jobs\SyncSupplierProjectJob;
|
|
use App\Models\PricingTier;
|
|
use App\Models\Project;
|
|
use App\Models\Tenant;
|
|
use App\Services\Billing\BillingTopupService;
|
|
use Illuminate\Foundation\Testing\DatabaseTransactions;
|
|
use Illuminate\Support\Facades\DB;
|
|
use Illuminate\Support\Facades\Queue;
|
|
use Tests\Concerns\SharesSupplierPdo;
|
|
|
|
uses(DatabaseTransactions::class);
|
|
uses(SharesSupplierPdo::class);
|
|
|
|
beforeEach(function () {
|
|
Queue::fake();
|
|
DB::statement("SELECT set_config('app.current_tenant_id', '0', true)");
|
|
PricingTier::query()->create([
|
|
'tier_no' => 1,
|
|
'leads_in_tier' => 100,
|
|
'price_per_lead_kopecks' => 5000, // 50₽/лид — capacity = balance/50
|
|
'is_active' => true,
|
|
'effective_from' => now(),
|
|
]);
|
|
});
|
|
|
|
// E (балансовый блок): авто-снятие preflight_blocked_at при пополнении.
|
|
// Политика «всё-или-ничего»: хватает на суммарный дневной лимит ВСЕХ активных
|
|
// проектов (вкл. заблокированные) → снять блок со всех + синк; не хватает → никого.
|
|
|
|
it('releases blocked project when topup covers the full order', function () {
|
|
$tenant = Tenant::factory()->withRequisites()->create(['balance_rub' => '0.00']);
|
|
$project = Project::factory()->for($tenant)->create([
|
|
'is_active' => true,
|
|
'daily_limit_target' => 30,
|
|
'preflight_blocked_at' => now(),
|
|
]);
|
|
|
|
// 2000₽ / 50 = 40 capacity >= 30 → снять блок + синк.
|
|
app(BillingTopupService::class)->topup($tenant->id, '2000.00', null);
|
|
|
|
expect($project->fresh()->preflight_blocked_at)->toBeNull();
|
|
Queue::assertPushed(SyncSupplierProjectJob::class);
|
|
});
|
|
|
|
it('keeps blocked when topup still insufficient', function () {
|
|
$tenant = Tenant::factory()->withRequisites()->create(['balance_rub' => '0.00']);
|
|
$project = Project::factory()->for($tenant)->create([
|
|
'is_active' => true,
|
|
'daily_limit_target' => 30,
|
|
'preflight_blocked_at' => now(),
|
|
]);
|
|
|
|
// 500₽ / 50 = 10 capacity < 30 → остаётся заблокированным, синка нет.
|
|
app(BillingTopupService::class)->topup($tenant->id, '500.00', null);
|
|
|
|
expect($project->fresh()->preflight_blocked_at)->not->toBeNull();
|
|
Queue::assertNotPushed(SyncSupplierProjectJob::class);
|
|
});
|
|
|
|
it('releases all-or-nothing across multiple blocked projects', function () {
|
|
$tenant = Tenant::factory()->withRequisites()->create(['balance_rub' => '0.00']);
|
|
$p1 = Project::factory()->for($tenant)->create([
|
|
'is_active' => true, 'daily_limit_target' => 20, 'preflight_blocked_at' => now(),
|
|
]);
|
|
$p2 = Project::factory()->for($tenant)->create([
|
|
'is_active' => true, 'daily_limit_target' => 20, 'preflight_blocked_at' => now(),
|
|
]);
|
|
|
|
// required = 20+20 = 40. 1500₽/50 = 30 capacity < 40 → НЕ снимать никого.
|
|
app(BillingTopupService::class)->topup($tenant->id, '1500.00', null);
|
|
expect($p1->fresh()->preflight_blocked_at)->not->toBeNull();
|
|
expect($p2->fresh()->preflight_blocked_at)->not->toBeNull();
|
|
|
|
// дополнили до 2000 → capacity 40 >= 40 → снять блок с ОБОИХ.
|
|
app(BillingTopupService::class)->topup($tenant->id, '500.00', null);
|
|
expect($p1->fresh()->preflight_blocked_at)->toBeNull();
|
|
expect($p2->fresh()->preflight_blocked_at)->toBeNull();
|
|
});
|
|
|
|
it('does nothing when tenant has no blocked projects', function () {
|
|
$tenant = Tenant::factory()->withRequisites()->create(['balance_rub' => '0.00']);
|
|
$project = Project::factory()->for($tenant)->create([
|
|
'is_active' => true,
|
|
'daily_limit_target' => 10,
|
|
'preflight_blocked_at' => null,
|
|
]);
|
|
|
|
app(BillingTopupService::class)->topup($tenant->id, '2000.00', null);
|
|
|
|
expect($project->fresh()->preflight_blocked_at)->toBeNull();
|
|
Queue::assertNotPushed(SyncSupplierProjectJob::class);
|
|
});
|