safe-global / safe-client-gateway / 28593316003

87%

Build Type

push

github

Committed by web-flow

Commit Message

feat: make Zerion wallet portfolio cache TTL configurable (#3197)

* feat: make Zerion wallet portfolio cache TTL configurable

The /v2/safes overview caches the Zerion wallet portfolio response via ZerionWalletPortfolioApi, which previously used a hardcoded 10s TTL. Expose it as ZERION_WALLET_PORTFOLIO_CACHE_TTL_SECONDS (config: balances.providers.zerion.walletPortfolioCacheTtlSeconds, default 10) so it can be tuned on prod without a redeploy to relieve Zerion rate-limit pressure.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

* test: set walletPortfolioCacheTtlSeconds in wallet-portfolio spec fake config

After merging main (which added zerion-wallet-portfolio-api.service.spec.ts), the spec's FakeConfigurationService didn't set the new walletPortfolioCacheTtlSeconds key this PR adds, so the constructor's getOrThrow threw and all 6 tests failed. Set the key in the fake config.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

* refactor: rename config to walletPortfolioTtlSeconds for consistency

Aligns the config key and env var with the other TTL settings
(ZERION_WALLET_PORTFOLIO_TTL_SECONDS); the default stays at 10s.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>

---------

Co-authored-by: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

Coverage Stats

3515 of 4304 branches covered (81.67%)

Branch coverage included in aggregate %.

1 of 1 new or added line in 1 file covered. (100.0%)

1 existing line in 1 file now uncovered.

9294 of 10497 relevant lines covered (88.54%)

520.66 hits per line