Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat(at_auth): Added registrar service class #771

Open
wants to merge 2 commits into
base: trunk
Choose a base branch
from

Conversation

Zambrella
Copy link

- What I did

  • Created the RegistrarServiceBase interface to define the contract for the service.
  • Implemented the RegistrarServiceImpl to handle API communications with the registrar.
  • Added ValidatePersonResponse to parse responses for validation requests.
  • Introduced RegistrarException to standardise error handling for failed API calls.
  • Implemented HTTP request retry logic with configurable retries and delay.
  • Added comprehensive unit tests using mocktail to validate API interactions.

- How to verify it
Run the test suite:

cd packages/at_auth
dart test test/registrar_test.dart

- Description for the changelog
Added registrar service to handle API communications with registrar

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.

1 participant