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

Adds rust tests for the registration flow #715

Merged
merged 2 commits into from
Jul 1, 2022

Conversation

frederikrothenberger
Copy link
Member

@frederikrothenberger frederikrothenberger commented Jun 28, 2022

The registration flow consistes of two canister calls: create_challenge and register.
As part of testing these canister calls the inteface of api::create_challenge has been changed to return a Result<Challenge, CallError> to make error behaviour testable.
Related to registration, this PR includes tests related to the intialization of the user range (ported from haskell).
The test for the user range widening has been moved to the http test module (together with other metrics) since it is unrelated to user registration.

The registration flow consistes of two canister calls: create_challenge and register.
As part of testing these canister calls the inteface of api::create_challenge has been changed to return a Result<Challenge, CallError> to make error behaviour testable.
Related to registration, this PR includes tests related to the intialization of the user range (ported from haskell).
The test for the user range widening has been moved to the http test module (together with other metrics) since it is unrelated to user registration.
Copy link
Member

@peterpeterparker peterpeterparker left a comment

Choose a reason for hiding this comment

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

Rubber stamp

@frederikrothenberger frederikrothenberger merged commit 4739426 into main Jul 1, 2022
@frederikrothenberger frederikrothenberger deleted the frederik/rust-register-tests branch July 1, 2022 14:54
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