stock-screener.service.ts: убран самописный тип TbankAccess и @Inject.
stock-screener.service.spec.ts: jest.mock для tbank.api (паттерн соседних spec).
stock.controller.ts: ручной allowed-Set сохранён с комментарием — без него e2e получает 200 вместо 400: глобальный whitelist pipe срезает неизвестные ключи раньше param-pipe с forbidNonWhitelisted. Вывод CQ-023 аудита не подтвердился (см. комментарий в Issue).
Verification
npm run verify — exit 0, 41 suites / 308 tests (исходная база); после переноса на develop с #154 — tsc и unit src/modules/investments (139 tests) зелёные; CI проверяет полный verify.
node test/sqlite.cjs stocks.e2e-spec.ts — 48/48.
Breaking Changes
Нет.
Заменяет #153 (commitlint `body-max-line-length`; без force-push — коммит перенесён на актуальный `develop` с переносом строк в body).
## Summary
Удалён строковый DI-алиас `'TBANK_ACCESS'`; `StockScreenerService` инжектит `TbankAccessService` из `tbank.api` напрямую, как соседние сервисы.
## Linked Issue
Closes #151
## Changes
- `instruments.module.ts`: убран `{ provide: 'TBANK_ACCESS', useExisting: TbankAccessService }`.
- `stock-screener.service.ts`: убран самописный тип `TbankAccess` и `@Inject`.
- `stock-screener.service.spec.ts`: `jest.mock` для `tbank.api` (паттерн соседних spec).
- `stock.controller.ts`: ручной `allowed`-Set **сохранён** с комментарием — без него e2e получает 200 вместо 400: глобальный `whitelist` pipe срезает неизвестные ключи раньше param-pipe с `forbidNonWhitelisted`. Вывод CQ-023 аудита не подтвердился (см. комментарий в Issue).
## Verification
- `npm run verify` — exit 0, 41 suites / 308 tests (исходная база); после переноса на develop с #154 — `tsc` и unit `src/modules/investments` (139 tests) зелёные; CI проверяет полный verify.
- `node test/sqlite.cjs stocks.e2e-spec.ts` — 48/48.
## Breaking Changes
Нет.
Drop the 'TBANK_ACCESS' string alias. The raw-body whitelist in
searchScreener stays: the global whitelist pipe strips unknown keys
before the param pipe can reject them.
Blocking a user prevents them from interacting with repositories, such as opening or commenting on pull requests or issues. Learn more about blocking a user.
Заменяет #153 (commitlint
body-max-line-length; без force-push — коммит перенесён на актуальныйdevelopс переносом строк в body).Summary
Удалён строковый DI-алиас
'TBANK_ACCESS';StockScreenerServiceинжектитTbankAccessServiceизtbank.apiнапрямую, как соседние сервисы.Linked Issue
Closes #151
Changes
instruments.module.ts: убран{ provide: 'TBANK_ACCESS', useExisting: TbankAccessService }.stock-screener.service.ts: убран самописный типTbankAccessи@Inject.stock-screener.service.spec.ts:jest.mockдляtbank.api(паттерн соседних spec).stock.controller.ts: ручнойallowed-Set сохранён с комментарием — без него e2e получает 200 вместо 400: глобальныйwhitelistpipe срезает неизвестные ключи раньше param-pipe сforbidNonWhitelisted. Вывод CQ-023 аудита не подтвердился (см. комментарий в Issue).Verification
npm run verify— exit 0, 41 suites / 308 tests (исходная база); после переноса на develop с #154 —tscи unitsrc/modules/investments(139 tests) зелёные; CI проверяет полный verify.node test/sqlite.cjs stocks.e2e-spec.ts— 48/48.Breaking Changes
Нет.