• Home
  • Features
  • Pricing
  • Docs
  • Announcements
  • Sign In

safe-global / safe-client-gateway / 20091945613

10 Dec 2025 08:19AM UTC coverage: 89.03% (-1.2%) from 90.212%
20091945613

push

github

web-flow
refactor(cache): change Zerion cache keys to be per address only (#2839)

* refactor(cache): change Zerion cache keys to be per address only

Remove chainId from Zerion cache keys for balances, collectibles, and positions.
This changes the caching strategy from per-chain-per-address to per-address only,
allowing data to be shared across chains for the same address.

Cache key changes:
- Before: {chainId}_zerion_balances_{safeAddress}
- After: zerion_balances_{safeAddress}

Similar changes applied to positions and collectibles cache keys.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>

Prettier

* refactor: remove unused chainId parameter from clear methods

Remove chainId parameter from clearBalances and clearCollectibles methods
in IBalancesApi interface and implementations. After the refactoring to
per-address cache keys, chainId is no longer needed by the API methods:

- ZerionBalancesApi uses address-only cache keys
- SafeBalancesApi uses its instance chainId variable

The repositories still accept chainId (needed to select the correct API)
but no longer pass it to the clear methods.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>

* Fix tests: Separate unit tests from integration/e2e tests

- Renamed tests requiring real infrastructure (.spec.ts -> .integration.spec.ts or .e2e-spec.ts)
- Updated package.json Jest config to exclude integration and e2e tests from default 'yarn test' run
- Tests requiring Redis, Postgres, or full app bootstrap are now properly categorized
- All 308 unit test suites now pass without requiring external services

Renamed files:
- Redis cache service test -> integration test
- Postgres database module tests (v1 & v2) -> integration tests
- Postgres database service test -> integration test
- Auth decorator test -> e2e test
- Spaces-related controller tests -> e2e tests
- ... (continued)

2835 of 3551 branches covered (79.84%)

Branch coverage included in aggregate %.

4 of 8 new or added lines in 5 files covered. (50.0%)

199 existing lines in 34 files now uncovered.

13616 of 14927 relevant lines covered (91.22%)

564.15 hits per line

Source File
Press 'n' to go to next uncovered line, 'b' for previous

94.44
/src/modules/users/users.module.ts
1
import { Module, forwardRef } from '@nestjs/common';
92✔
2
import { TypeOrmModule } from '@nestjs/typeorm';
92✔
3
import { PostgresDatabaseModuleV2 } from '@/datasources/db/v2/postgres-database.module';
92✔
4
import { User } from '@/modules/users/datasources/entities/users.entity.db';
92✔
5
import { Member } from '@/modules/users/datasources/entities/member.entity.db';
92✔
6
import { Wallet } from '@/modules/wallets/datasources/entities/wallets.entity.db';
92✔
7
import { IMembersRepository } from '@/modules/users/domain/members.repository.interface';
92✔
8
import { MembersRepository } from '@/modules/users/domain/members.repository';
92✔
9
import { IUsersRepository } from '@/modules/users/domain/users.repository.interface';
92✔
10
import { UsersRepository } from '@/modules/users/domain/users.repository';
92✔
11
import { UsersController } from '@/modules/users/routes/users.controller';
92✔
12
import { UsersService } from '@/modules/users/routes/users.service';
92✔
13
import { AuthModule } from '@/modules/auth/auth.module';
92✔
14
import { WalletsModule } from '@/modules/wallets/wallets.module';
92✔
15
import { SiweModule } from '@/modules/siwe/siwe.module';
92✔
16
import { SpacesModule } from '@/modules/spaces/spaces.module';
92✔
17

18
@Module({
19
  imports: [
20
    PostgresDatabaseModuleV2,
21
    TypeOrmModule.forFeature([User, Member, Wallet]),
22
    WalletsModule,
23
    AuthModule,
24
    SiweModule,
UNCOV
25
    forwardRef(() => SpacesModule),
×
26
  ],
27
  providers: [
28
    {
29
      provide: IUsersRepository,
30
      useClass: UsersRepository,
31
    },
32
    {
33
      provide: IMembersRepository,
34
      useClass: MembersRepository,
35
    },
36
    UsersService,
37
  ],
38
  controllers: [UsersController],
39
  exports: [IUsersRepository, IMembersRepository],
40
})
41
export class UsersModule {}
92✔
STATUS · Troubleshooting · Open an Issue · Sales · Support · CAREERS · ENTERPRISE · START FREE · SCHEDULE DEMO
ANNOUNCEMENTS · TWITTER · TOS & SLA · Supported CI Services · What's a CI service? · Automated Testing

© 2025 Coveralls, Inc