Skip to content

Test cases and assertions for Login service #21

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 5 commits into
base: master
Choose a base branch
from
Open

Test cases and assertions for Login service #21

wants to merge 5 commits into from

Conversation

@rdj68
Copy link
Contributor

@rdj68 rdj68 commented Oct 22, 2023 •
edited
Loading

This commit contains:

  1. Added return type error for sendOtp function and handled case where error will be sent to frontend if input fails validations
  2. Added Test cases and assertions for the login and verify function of loginService
  3. Added Mock code for otp_client and the two channels phone_client and Email_client

@rdj68 rdj68 requested a review from This-Is-Prince October 22, 2023 06:35
@rdj68 rdj68 changed the title Tests for Login s= Test cases and assertions for Login service Oct 22, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@SaiNageswarS SaiNageswarS SaiNageswarS approved these changes

@This-Is-Prince This-Is-Prince Awaiting requested review from This-Is-Prince

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@rdj68 @SaiNageswarS