Skip to content

feat: address book integration tests#117

Merged
mraszyk merged 3 commits intomainfrom
mraszyk/address-book-integration-tests
Jan 26, 2024
Merged

feat: address book integration tests#117
mraszyk merged 3 commits intomainfrom
mraszyk/address-book-integration-tests

Conversation

@mraszyk
Copy link
Copy Markdown
Contributor

@mraszyk mraszyk commented Jan 26, 2024

This MR adds integration tests for address book operations:

  • creating new address book entries
  • updating existing address book entries
  • removing existing address book entries
  • retrieving single address book entries by their IDs
  • listing existing address book entries

It also adds default proposal policies for the admin users to perform arbitrary address book operations.

@mraszyk mraszyk requested a review from keplervital January 26, 2024 09:34
Copy link
Copy Markdown
Member

@keplervital keplervital left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@mraszyk mraszyk merged commit a4c079a into main Jan 26, 2024
@mraszyk mraszyk deleted the mraszyk/address-book-integration-tests branch January 26, 2024 13:59
aterga added a commit that referenced this pull request Apr 3, 2026
Bump direct dependencies and add override for transitive dep:
- playwright 1.49.1 → 1.55.1 in docs (high, #117/#118)
- validator 13.12.0 → 13.15.35 in apps/wallet (high, #152/#153; also fixes medium #122/#123)
- axios <1.8.2 → ^1.8.2 via pnpm override (high, #70)

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants