Skip to content

Unit Tests for Wallet Adapters Issue #350#503

Merged
Ceejaytech25 merged 2 commits into
ceejaylaboratory:mainfrom
Ndanusa:issue-350-wallet-adapters-tests
May 31, 2026
Merged

Unit Tests for Wallet Adapters Issue #350#503
Ceejaytech25 merged 2 commits into
ceejaylaboratory:mainfrom
Ndanusa:issue-350-wallet-adapters-tests

Conversation

@Ndanusa
Copy link
Copy Markdown
Contributor

@Ndanusa Ndanusa commented May 31, 2026

Description

This PR implements the task to add generic Wallet Adapter types and specifically implement and unit test the Freighter wallet adapter for the AnchorPoint frontend.

What was done

  • Wallet Adapters interface: Added generic types for wallet interactions (types.ts).
  • Freighter Adapter: Implemented FreighterAdapter class for interacting with @stellar/freighter-api.
  • Unit Tests: Created comprehensive unit tests for FreighterAdapter to ensure maximum coverage and correct handling of connection states, errors, and cancellations (FreighterAdapter.test.ts).
  • Dev Dependencies: Installed and configured vitest to enable execution of unit tests within the dashboard Vite project.
  • Configuration updates: Updated package.json to include "test": "vitest run" and added vitest.config.ts.

Acceptance Criteria Handled

  • Implementation completed according to project standards.
  • Code is properly documented and commented.
  • Unit/Integration tests provided.
  • Passing CI pipeline readiness (tests, lint, build prepared).
  • Maintained institutional dark mode aesthetic alignment in any potential future frontend usage by decoupling logic properly.

Testing Steps

Run the following from the /dashboard directory to execute unit tests:

npm run test

Related Issues

Closes #350
Closes #352

@drips-wave
Copy link
Copy Markdown

drips-wave Bot commented May 31, 2026

@Ndanusa Great news! 🎉 Based on an automated assessment of this PR, the linked Wave issue(s) no longer count against your application limits.

You can now already apply to more issues while waiting for a review of this PR. Keep up the great work! 🚀

Learn more about application limits

@Ceejaytech25
Copy link
Copy Markdown
Contributor

LGTM!

@Ceejaytech25 Ceejaytech25 merged commit a6d606f into ceejaylaboratory:main May 31, 2026
1 check failed
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.

[Frontend] Implement French translation for SEP-24 flow [Frontend] Unit Tests for Wallet Adapters

2 participants