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

Improve Code Coverage in src/screens/ForgotPassword/ForgotPassword.tsx #3025

Closed
hustlernik opened this issue Dec 29, 2024 · 6 comments
Closed
Assignees
Labels
bug Something isn't working good first issue Good for newcomers test Testing application ui/ux issue related and being worked with the figma file of the Admin UI unapproved

Comments

@hustlernik
Copy link
Contributor

Goal

The goal of this issue is to improve the code coverage for the screen: src/screens/ForgotPassword/ForgotPassword.tsx

Tasks

  1. Review the file to identify sections of code that are being ignored by Codecov or are not covered by tests.
  2. Create or update test cases to ensure 100% coverage for the file.
  3. Remove any /* istanbul ignore */ or equivalent statements that bypass code coverage reporting, unless absolutely necessary.

Resources

  • Refer to the foundational documentation on writing test cases in the repository.
  • Check the Codecov report for details on the uncovered lines.

Acceptance Criteria

  • All sections of the file are covered by tests.
  • Code coverage for the file reaches 100%.
  • PR created with necessary updates, passing all checks and reviews.
@hustlernik hustlernik added the bug Something isn't working label Dec 29, 2024
@github-actions github-actions bot added documentation Improvements or additions to documentation test Testing application ui/ux issue related and being worked with the figma file of the Admin UI unapproved good first issue Good for newcomers labels Dec 29, 2024
@PratapRathi
Copy link
Contributor

@palisadoes please assign this issue to me.

@palisadoes
Copy link
Contributor

@palisadoes please assign this issue to me.

Our policy is to assign no more than two issues to each contributor across all repositories. This way everyone gets a chance to participate in the projects. We sometimes give exceptions for more urgent cases and sometimes we lose track, but the policy stands. You have reached your limit, please wait until your existing issues are closed before requesting more issues. You could unassign yourself from one of the other issues too.

@PratapRathi
Copy link
Contributor

Thank you for clarifying! I’ll focus on my current issues and only request more once they’re resolved.

@palisadoes palisadoes removed the documentation Improvements or additions to documentation label Dec 29, 2024
@PratapRathi
Copy link
Contributor

PratapRathi commented Dec 31, 2024

@palisadoes please assign this issue to me.

@saurabh-kr12
Copy link

@palisadoes can i work on this issue?

@PratapRathi
Copy link
Contributor

@palisadoes please assign this issue to me, PR for this issue is ready.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working good first issue Good for newcomers test Testing application ui/ux issue related and being worked with the figma file of the Admin UI unapproved
Projects
Archived in project
Development

No branches or pull requests

4 participants