Files
portal/app/tests/Feature/ClientSms/NoFundsStopTest.php
T
Дмитрий b6c837e28a chore(смс-клиент): приёмка Этапа 4 — формат по всей области модуля, а не по свежим файлам
Task 8, приёмка Этапа 4. Кода она не приносит по замыслу: все двенадцать строк
листа (4.1–4.8, 4.11–4.14) закрыты ДВУМЯ доказательствами каждая — тест И живой
прогон, — экранные строки отдельно посмотрены глазами в браузере. Итоговая
таблица Этапа 4 заполнена в приёмочном листе (лист в git не лежит).

В git уходит ровно одна правка — порядок импортов в шести файлах тестов (В-192).
Причина, по которой она вообще нашлась: три этапа подряд `pint` гонялся ТОЛЬКО
по свежим файлам, а по всей области модуля не гонялся ни разу. Заодно выяснено,
что жалобы `line_ending` чинить не надо — это виндовые переводы строк рабочей
копии, в git их нет вовсе (pint «исправил», git не увидел ни одного изменения).
Перегнаны тесты этих шести файлов: 41/41 зелено.

🔴 ДЕНЕЖНАЯ МИНА СВЕРХ ПОСТРОЕННОГО (В-190). Права спросил у самой базы матрицей
по всем таблицам модуля — и увидел, что у рабочей роли crm_app_user есть право
писать в таблицу заморозок, а на её счётчик номеров ad_wallet_holds_id_seq права
нет. Живой прогон парно: без права заморозка денег под боевой ролью ПАДАЕТ («нет
доступа к последовательности»), с правом идёт. Заморозка делается при КАЖДОМ
заказе рассылки, при заказе имени и при запуске рекламной кампании Яндекса.
Промах был мой: в В-142 я привёл стенд к эталону двумя точечными командами по
именам из плана вместо `ON ALL SEQUENCES`, как делает сам db/02_grants.sql.
Памятка выката переписана: вместо списка имён счётчиков — запрос, который САМ
находит все счётчики без права. На бою проверить (косвенно там всё в порядке —
рекламные кампании запускаются тем же кодом, но довод косвенный).

🔴 И ПЕРВЫЙ ЗАХОД ПРИЁМКИ ПРОШЁЛ «ЗЕЛЕНО», НЕ ДОКАЗАВ НИ ОДНОЙ ДЕНЕЖНОЙ СТРОКИ
(В-191): песочница гасит и возврат заморозки при срыве, и ночного работника
целиком. Датчик на будущее — не сдвинулась ни одна копейка, значит прогон не
доказал ничего, даже когда всё зелено.

Ещё три промаха своих же приборов: В-193 и В-194 (прогон не доходил до
состояния — сторож законно даёт попытке дожать её срок, а имя без отметки
согласования кнопка законно не включает: правда была в коде), В-195 (браузерный
замер читал таблицу рассылок вместо базы и «доказал» поломку, которой нет —
класс В-121, соврал прибор).

Живьём под боевой ролью crm_app_user: сторож зависших тремя заходами (пометка →
выдержка срока → срыв, заморозка 17.00 → 0.00); имя за долг вернулось и списало
ровно 100 ₽, отключённое владельцем осталось выключенным при 5 000 ₽; кнопка
включения отказала числами, без пометки клиента 404, с пометкой списала 600 ₽;
номера руками легли пятью видами записи; продолжение довело 5 из 5 за 42.50 ₽.
Под служебной ролью — чистка снимка тройкой (без права падение, без политики
srv_bypass «успешный ноль», с обоими удалено 3).

Прогоны: ClientSms 320/320 (13 пачек, все с первой попытки), приём лидов 17/17,
фронт 1700 + 3 пропущенных (одна чужая давняя ошибка, В-57), phpstan ровно 2
чужие давние, vue-tsc ровно 5 чужих давних в 5 файлах (git blame: от 25.07),
pint чисто.

Стенд сверен со снимком «до» поле за полем и совпал. Намеренное изменение одно:
рабочей роли выданы права на ВСЕ счётчики схемы, как в эталоне db/02_grants.sql.

🟡 Открытый вопрос владельцу — В-182: номера в журнале сообщений живут без срока,
обязательство «90 дней» закрывает только снимок получателей.

Ветка НЕ влита в main и НЕ выкачена. Порядок выката: миграции →
db/03_service_bypass_policies.sql → подсчёт политик srv_bypass (+8) → права →
контрольный запуск чистки снимка.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
2026-07-30 15:05:08 +03:00

222 lines
10 KiB
PHP
Raw Blame History

This file contains ambiguous Unicode characters
This file contains Unicode characters that might be confused with other characters. If you think that this is intentional, you can safely ignore this warning. Use the Escape button to reveal them.
<?php
declare(strict_types=1);
use App\Jobs\SendClientSmsCampaignJob;
use App\Models\AdWallet;
use App\Models\ClientSmsCampaign;
use App\Models\ClientSmsContact;
use App\Models\ClientSmsMessage;
use App\Models\Tenant;
use App\Services\Advertising\AdWalletService;
use App\Services\ClientSms\AllowedSmsOperators;
use App\Services\ClientSms\ClientSmsAudienceBuilder;
use App\Services\ClientSms\ClientSmsPricing;
use App\Services\ClientSms\ClientSmsRecipientSelector;
use App\Services\ClientSms\ClientSmsSnapshotReader;
use App\Services\ClientSms\ClientSmsSnapshotWriter;
use App\Services\Sms\OperatorNormalizer;
use App\Services\Sms\Providers\MtsSmsProvider;
use App\Services\Sms\SmsRouter;
use Carbon\CarbonImmutable;
use Illuminate\Foundation\Testing\RefreshDatabase;
use Illuminate\Support\Facades\DB;
use Illuminate\Support\Facades\Http;
/**
* Деньги кончились посреди рассылки (строка листа 2.4).
*
* Почему это вообще нужно: AdWalletService::charge при недоборе НЕ бросает
* исключение, а «прижимает» баланс к нулю. То есть без проверки рассылка молча
* доходит до конца, часть СМС уходит бесплатно, а клиент видит нулевой кошелёк
* без объяснения. Класс поломки — «молчаливый сбой» (20.07).
*
* 🪤 Ловушка В-41: вся смета ЗАМОРОЖЕНА при создании рассылки, поэтому
* «свободно = баланс − заморожено» ≈ 0 и наивная проверка остановила бы
* рассылку на первом же номере. Доступное считается как
* `баланс − заморожено + СВОЯ активная заморозка`.
*
* Номера только синтетические 7999… — в реальном режиме это боевая отправка.
*/
uses(RefreshDatabase::class);
// Этап 3: окно 10–20 по местному времени получателя. Время прогона фиксируем — иначе
// тест зависел бы от часа, в который его запустили (вечером все номера ждали бы утра).
// 09:00 UTC = полдень в Москве, окно открыто.
beforeEach(function () {
$this->travelTo(CarbonImmutable::parse('2026-08-03 09:00', 'UTC'));
});
/** Кампания в статусе queued со снимком цены (8.50 за СМС), как её создаёт контроллер. */
function noFundsCampaign(int $tenantId, int $totalSms): ClientSmsCampaign
{
return ClientSmsCampaign::create([
'tenant_id' => $tenantId,
'title' => 'Рассылка',
'body' => 'Здравствуйте!',
'sender_name' => 'liderra.ru',
'source' => ClientSmsCampaign::SOURCE_BASE,
'audience_days' => null,
'status' => ClientSmsCampaign::STATUS_QUEUED,
'segments' => 1,
'planned_count' => $totalSms,
'sent_count' => 0,
'total_sms' => $totalSms,
'price_rub_per_sms' => '8.50',
'estimated_cost_rub' => bcmul('8.50', (string) $totalSms, 2),
'actual_cost_rub' => null,
'created_by' => null,
]);
}
/**
* 🪤 В-46: канал выбирается ДВАЖДЫ — отборщиком при создании рассылки и читателем
* снимка при отправке. Читатель берёт роутер из контейнера, поэтому подменять надо
* роутер: иначе отправится ноль, а денежный тест останется зелёным, сравнивая
* ноль с нулём.
*/
function noFundsBindRoute(): void
{
app()->instance(SmsRouter::class, new SmsRouter([new MtsSmsProvider('x', ['mts'], ['*' => 0])]));
}
/** Снимок получателей — то, что в бою делает контроллер (строки листа 2.1–2.2). */
function noFundsSnapshot(ClientSmsCampaign $campaign): void
{
$selector = new ClientSmsRecipientSelector(app(SmsRouter::class), new OperatorNormalizer, new AllowedSmsOperators(new OperatorNormalizer));
$recipients = app(ClientSmsAudienceBuilder::class)->build($campaign);
app(ClientSmsSnapshotWriter::class)->write(
(int) $campaign->tenant_id,
(int) $campaign->id,
$selector->build($recipients, (int) $campaign->tenant_id),
);
}
/**
* Деньги «ушли на сторону» между созданием рассылки и отправкой — ровно так же,
* как если бы их потратил другой рекламный канал того же клиента.
*/
function noFundsDrainWallet(int $tenantId, string $leave): void
{
DB::table('ad_wallets')->where('tenant_id', $tenantId)->update(['balance_rub' => $leave]);
}
/** Готовая к отправке рассылка на 3 номера с замороженной сметой 25.50. */
function noFundsPrepare(): array
{
config(['services.sms.sandbox' => false]);
Http::fake(['api.mts.ru/*' => Http::response([
'code' => 0,
'data' => ['submitResults' => [['msid' => '79990000001', 'messageID' => 'MSG-OK-1', 'code' => 'OK']]],
], 200)]);
$tenant = Tenant::factory()->create();
$wallet = app(AdWalletService::class);
$wallet->topup($tenant->id, '1000.00', null, 'test');
foreach (['79995550001', '79995550002', '79995550003'] as $i => $phone) {
ClientSmsContact::create([
'tenant_id' => $tenant->id,
'phone' => $phone,
'name' => 'К'.$i,
'operator' => 'МТС',
'tz_offset_minutes' => 180,
// Этап 3: без известного региона номер не уходит вовсе (В-85).
'tz_offset_minutes' => 180,
]);
}
noFundsBindRoute();
$campaign = noFundsCampaign($tenant->id, 3);
noFundsSnapshot($campaign);
$wallet->freeze($tenant->id, 'sms', 'campaign', $campaign->id, '25.50'); // 8.50 × 3
// Осталось 12.00 — хватает ровно на одно СМС, на второе уже нет.
noFundsDrainWallet($tenant->id, '12.00');
return [$tenant, $campaign, $wallet];
}
function noFundsRunJob(int $campaignId, int $tenantId, AdWalletService $wallet): void
{
(new SendClientSmsCampaignJob($campaignId, $tenantId))->handle(
app(ClientSmsSnapshotReader::class),
app(ClientSmsPricing::class),
$wallet,
);
}
it('останавливается честно, когда деньги кончились посреди рассылки', function () {
[$tenant, $campaign, $wallet] = noFundsPrepare();
noFundsRunJob($campaign->id, $tenant->id, $wallet);
$campaign->refresh();
expect($campaign->status)->toBe(ClientSmsCampaign::STATUS_CANCELLED)
->and($campaign->stop_reason)->toBe(ClientSmsCampaign::STOP_NO_FUNDS)
->and($campaign->sent_count)->toBe(1) // ушло меньше, чем планировали
->and((string) $campaign->actual_cost_rub)->toBe('8.50');
// В журнале видно, на каком номере встали — а не «просто кончилось».
expect(ClientSmsMessage::where('campaign_id', $campaign->id)
->where('status', ClientSmsMessage::SKIP_NO_FUNDS)->count())->toBe(1);
});
it('не обнуляет баланс молча: списано ровно за отправленное', function () {
[$tenant, $campaign, $wallet] = noFundsPrepare();
noFundsRunJob($campaign->id, $tenant->id, $wallet);
$after = AdWallet::where('tenant_id', $tenant->id)->first();
// 12.00 − 8.50 за единственное ушедшее СМС. Не «0.00», к которому прижимает charge.
expect((string) $after->balance_rub)->toBe('3.50')
->and((string) $after->balance_rub)->toBe(
bcsub('12.00', (string) $campaign->fresh()->actual_cost_rub, 2)
)
->and((string) $after->frozen_rub)->toBe('0.00'); // заморозка снята, деньги не заперты
});
it('своя заморозка не мешает: денег хватает — уходят все, остановки нет', function () {
// Обратная сторона ловушки В-41: если бы «доступно» считалось как
// баланс − заморожено, эта рассылка встала бы на первом же номере при полном кошельке.
config(['services.sms.sandbox' => false]);
Http::fake(['api.mts.ru/*' => Http::response([
'code' => 0,
'data' => ['submitResults' => [['msid' => '79990000001', 'messageID' => 'MSG-OK-1', 'code' => 'OK']]],
], 200)]);
$tenant = Tenant::factory()->create();
$wallet = app(AdWalletService::class);
$wallet->topup($tenant->id, '25.50', null, 'test'); // ровно на смету, ни рублём больше
foreach (['79995550001', '79995550002', '79995550003'] as $i => $phone) {
ClientSmsContact::create([
'tenant_id' => $tenant->id,
'phone' => $phone,
'name' => 'К'.$i,
'operator' => 'МТС',
'tz_offset_minutes' => 180,
// Этап 3: без известного региона номер не уходит вовсе (В-85).
'tz_offset_minutes' => 180,
]);
}
noFundsBindRoute();
$campaign = noFundsCampaign($tenant->id, 3);
noFundsSnapshot($campaign);
$wallet->freeze($tenant->id, 'sms', 'campaign', $campaign->id, '25.50');
noFundsRunJob($campaign->id, $tenant->id, $wallet);
$campaign->refresh();
expect($campaign->status)->toBe(ClientSmsCampaign::STATUS_DONE)
->and($campaign->stop_reason)->toBeNull()
->and($campaign->sent_count)->toBe(3)
->and((string) $campaign->actual_cost_rub)->toBe('25.50');
expect(ClientSmsMessage::where('campaign_id', $campaign->id)
->where('status', ClientSmsMessage::SKIP_NO_FUNDS)->exists())->toBeFalse();
});