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

Views: Create tests for profile_page.dart #1230

Closed
palisadoes opened this issue Feb 28, 2022 · 20 comments · Fixed by #2433
Closed

Views: Create tests for profile_page.dart #1230

palisadoes opened this issue Feb 28, 2022 · 20 comments · Fixed by #2433
Assignees
Labels
good first issue Good for newcomers test Testing application

Comments

@palisadoes
Copy link
Contributor

palisadoes commented Feb 28, 2022

The Talawa code base needs to be 100% reliable. This means we need to have 100% test code coverage.

Tests need to be written for file lib/views/after_auth_screens/profile/profile_page.dart

  • When complete, all methods, classes and/or functions in the file will need to be tested.
  • These tests must be placed in a single file with the name test/views/after_auth_screens/profile/profile_page_test.dart. You may need to create the appropriate directory structure to do this.

IMPORTANT:

Please refer to the parent issue on how to implement these tests correctly:

PR Acceptance Criteria

  • When complete this file must show 100% coverage when merged into the code base. This will be clearly visible when you submit your PR.
  • The current code coverage for the file can be found here. If the file isn't found in this directory, or there is a 404 error, then tests have not been created.
  • The PR will show a report for the code coverage for the file you have added. You can use that as a guide.
  • You can verify your own code coverage by creating an account at Codecov.io
  • No functionality must be removed to achieve this goal
@palisadoes palisadoes added the bug Something isn't working label Feb 28, 2022
@github-actions github-actions bot added parent The issue is generating more sub-issues test Testing application unapproved Unapproved, needs to be triaged labels Feb 28, 2022
@palisadoes palisadoes added points 01 good first issue Good for newcomers and removed bug Something isn't working parent The issue is generating more sub-issues labels Mar 1, 2022
@CyberWake
Copy link
Member

@Sagar2366 sir can you assign this to me if no one is working upon it.

@palisadoes palisadoes removed the unapproved Unapproved, needs to be triaged label Mar 11, 2022
@noman2002
Copy link
Member

Unassigned due to inactivity.

@noman2002 noman2002 added the unapproved Unapproved, needs to be triaged label Jan 31, 2023
@Ayush0Chaudhary
Copy link
Contributor

@palisadoes @noman2002 @TheHazeEffect @SiddheshKukade please assign me this issue!

@palisadoes
Copy link
Contributor Author

palisadoes commented Feb 5, 2023

@Ayush0Chaudhary You already have 2 assigned issues. Please focus on those before requesting more.

@aks-2101
Copy link

aks-2101 commented Feb 7, 2023

@palisadoes @noman2002 assign me this, i'll like to try this issue

@palisadoes palisadoes removed the unapproved Unapproved, needs to be triaged label Feb 7, 2023
@noman2002
Copy link
Member

@aks-2101 Any update ?

@palisadoes
Copy link
Contributor Author

Unassigning due to inactivity

@beraSayantan
Copy link

I want to work on this issue. Please assign!

@tarun824
Copy link

@palisadoes @Ayush0Chaudhary @noman2002 Can you assign this to me i can work on this

@Cioppolo14
Copy link
Contributor

@tarun824 Are you still working on this?

@palisadoes
Copy link
Contributor Author

Unassigning due to inactivity and no pull request

@vinfinity7
Copy link

Hi @Cioppolo14 , there seems to be test file written at the desired location test/views/after_auth_screens/profile/profile_page_test.dart , so kindly explain what needs to be done and i shall be willingly up for the task !

@Cioppolo14
Copy link
Contributor

@vinfinity7 For this issue, the test coverage for that file is 88%, and we need it to be at 100%. I will assign you to address this. Let me know if you change your mind!

Copy link

This issue did not get any activity in the past 10 days and will be closed in 180 days if no update occurs. Please check if the develop branch has fixed it and report again or close the issue.

@github-actions github-actions bot added the no-issue-activity No issue activity label Jan 19, 2024
@Cioppolo14
Copy link
Contributor

Unassigning due to inactivity

@github-actions github-actions bot removed the no-issue-activity No issue activity label Jan 20, 2024
@RunTerror
Copy link

@Cioppolo14 Please assign me this issue

Copy link

This issue did not get any activity in the past 10 days and will be closed in 180 days if no update occurs. Please check if the develop branch has fixed it and report again or close the issue.

@github-actions github-actions bot added the no-issue-activity No issue activity label Jan 31, 2024
@palisadoes
Copy link
Contributor Author

Unassigning, no activity

@github-actions github-actions bot removed the no-issue-activity No issue activity label Feb 1, 2024
Copy link

This issue did not get any activity in the past 10 days and will be closed in 180 days if no update occurs. Please check if the develop branch has fixed it and report again or close the issue.

@github-actions github-actions bot added the no-issue-activity No issue activity label Feb 14, 2024
@Azad99-9
Copy link
Contributor

@Cioppolo14 please assign me.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Good for newcomers test Testing application
Projects
None yet
Development

Successfully merging a pull request may close this issue.