safe-global / safe-client-gateway / 21516004463

88%

push

github

feat: new V3 Owners endpoint (#2886)

* feat: add SafeSchemaV2 and paginated getSafesByOwnerV2 datasource (#2882)

* feat: add SafeSchemaV2 and paginated getSafesByOwnerV2 datasource

* docs: add a clarification comment for getSafesByOwnerV2

* feat: getSafesByOwnerV2 with pagination support (SafeRepository) (#2883)

* feat: add SafeSchemaV2 and paginated getSafesByOwnerV2 datasource

* docs: add a clarification comment for getSafesByOwnerV2

* feat: add getSafesByOwnerV2 implementation with pagination support in SafeRepository

* chore: format

* fix: move logging after the loop (if next page is still non-null) to check if more pages remain after the limit

* feat: add getAllSafesByOwnerV2 to fetch all Safes with the new endpoint

* refactor: optimize getAllSafesByOwner functions; add max pages to the config; minor code improvement

* test: add tests for owners service

* refactor: add a new type `SafesByChainId`

* fix: no shadowing an outer `next` variable

* tests: new tests for safe.repository; move tests from service layer to the repo layer

* refactor: accumulate safe addresses instead of an entire safe object

* refactor: simplify the new getSafes functions

* chore: linting

* chore: formatting

* chore: formatting

* feat: v3 Owners Controller (#2884)

* feat: add SafeSchemaV2 and paginated getSafesByOwnerV2 datasource

* docs: add a clarification comment for getSafesByOwnerV2

* feat: add getSafesByOwnerV2 implementation with pagination support in SafeRepository

* chore: format

* fix: move logging after the loop (if next page is still non-null) to check if more pages remain after the limit

* feat: add getAllSafesByOwnerV2 to fetch all Safes with the new endpoint

* feat: add owners controller v3 using transaction service v2 endpoint

* fix: nullable values in SafeSchemaV2

* test: add tests for v3 owners controller

* test: add tests for the new schema

* fix: endpoint returns { chainId: [] } instead of { chainId: null } if no safes foun... (continued)

2872 of 3627 branches covered (79.18%)

Branch coverage included in aggregate %.

13334 of 14593 relevant lines covered (91.37%)

563.84 hits per line