safe-global / safe-client-gateway / 27215725977

90%

Build Type

push

github

Committed by web-flow

Commit Message

feat: UUID as spaceId instead of numeric value (#3117)

* feat: use UUID for public Space ID (initial migration)

* fix: adjust controllers for address book and members

* test: update tests

* refactor: return uuid directly from SpacesRepository.update

* refactor: findByUuid as a separate function for better code readability

* test: adjust e2e tests

* feat: accept legacy numeric Space ID on read endpoints only for FE fallback

* fix: UUIDs for the surveys controller

* test: fix spaces controller tests

* fix: use UUID for invitations

* fix: harden UUID resolution

* refactor:  consolidate Space id resolution into SpaceIdPipe / LegacySpaceIdPipe

* fix: replace number with string in tests for spaceId

* fix: formatting

* fix: integration tests

* fix: integration tests for members; chore: linting

* chore: add license headers

* refactor: cut redundant queries from GET /spaces/:id

* fix: unify 400 error msg body

* test: improve unit and integration test coverage

* refactor: cleanup

* fix: tests

* feat: accept legacy numeric space id on mutating routes for FE fallback

* refactor: harden typing with UUID instead of a string

* refactor: rename `Space` to `Workspace` in err msgs

* fix: integration tests

* fix: validate UUID in findIdByIdOrUuid; drop dead int check; document SpaceIdPipe wiring

* chore: update swagger

* refactor: address comments (UUID regex as a const, new tests; code adjustments for better readability

* feat: update members, address book, surveys etc to use space.uuid

* chore: update swagger

* refactor: separate func for fake UUIDs, extract new consts, add explanation comments

* chore: merge conflicts

---------

Co-authored-by: dsh <11198975+LucieFaire@users.noreply.github.com>

Coverage Stats

4501 of 5371 branches covered (83.8%)

Branch coverage included in aggregate %.

75 of 83 new or added lines in 32 files covered. (90.36%)

2 existing lines in 2 files now uncovered.

16786 of 18288 relevant lines covered (91.79%)

575.36 hits per line