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

Write a tutorial on Flutter Testing #1481

Closed
literalEval opened this issue Feb 11, 2023 · 24 comments
Closed

Write a tutorial on Flutter Testing #1481

literalEval opened this issue Feb 11, 2023 · 24 comments
Labels
bug Something isn't working documentation Update documentation feature request test Testing application

Comments

@literalEval
Copy link
Member

Recently I have seen a couple of test PRs which are

  • Not covering all the cases of the file
  • Importing unnecessary files
  • Not generating/using mocks properly
  • .....

Describe the solution you'd like
I would like to write some sort of documentation on testing with Flutter so that

  • Beginners will get a decent resource to start with
  • New-comers will get to know about the existing mocks, stubbs that we already have and can you them

Describe alternatives you've considered
An alternative could be to document/comment the mocks and stubbs themselves, but I think creating a webpage/documentation page would be a better idea.

Approach to be followed (optional)

  • What are tests and why do we need them ?
  • How to cover all the corner cases ?
  • How to start with Flutter testing ?
  • What are mocks and to generate them ?
  • How to stub methods and classes ?
  • What to do if you find a bug while writing test for that file ?
  • .....
@github-actions github-actions bot added bug Something isn't working documentation Update documentation test Testing application unapproved Unapproved, needs to be triaged labels Feb 11, 2023
@palisadoes
Copy link
Contributor

Thanks. This is really needed.

  1. This needs to be added to the CONTRIBUTING.md file.
  2. However, please focus on that linting issue. That's a game changer, and it would be really sweet if we could make it into a GitHub Action as another Palisadoes repo!

@literalEval
Copy link
Member Author

Sure. I will work on the linting issue along with other small issues. Since it is a huge task in itself and will require some time and work, I will maybe use fun issues like this one for taking a break from it for some time :)

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

noman2002 commented Feb 12, 2023

@literalEval Sorry I can't assign this. You are way too above the limit. 5 issue is assigned in your name.

@literalEval
Copy link
Member Author

I have already created PRs for 2 of them :) Have complete half of one of the issues and started the documentation one. Also, this is a fun thing to do (writing an article), so I think I can do it.

@prathamesh-mutkure
Copy link

Hey @noman2002, can I work on this?

@literalEval
Copy link
Member Author

@prathamesh-mutkure I am already working on this. Have already wrote half of the article.

@palisadoes
Copy link
Contributor

It's not assigned to anyone. Both of you have two open issues.

Either wait until your PRs are merged otherwise we'll assign it to someone else.

Everyone needs a chance

@literalEval
Copy link
Member Author

@palisadoes @noman2002 please assign this to me.

@palisadoes
Copy link
Contributor

Ravi, you have two issues. Please return when they are completed

@noman2002
Copy link
Member

@prathamesh-mutkure Please go ahead.
@literalEval You have already worked on various PRs. Let him work on this one.

@prathamesh-mutkure prathamesh-mutkure removed their assignment Feb 18, 2023
@github-actions
Copy link

github-actions bot commented Mar 5, 2023

This issue did not get any activity in the past 14 days and will be closed in 365 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 Mar 5, 2023
@ALIRAZA17
Copy link

@palisadoes Can I work on this one?

@literalEval
Copy link
Member Author

@ALIRAZA17 Welcome to Palisadoes !! please make sure you write a thorough testing guide.

@github-actions
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 Oct 26, 2023
@palisadoes
Copy link
Contributor

Unassigning due to inactivity

@github-actions github-actions bot removed the no-issue-activity No issue activity label Oct 28, 2023
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 Nov 23, 2023
@palisadoes
Copy link
Contributor

@literalEval Is there a reference on the flutter site that we can use instead of having to write this ourselves?
If there is one, let us know.

@github-actions github-actions bot removed the no-issue-activity No issue activity label Nov 24, 2023
Copy link

github-actions bot commented Dec 5, 2023

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 Dec 5, 2023
@literalEval
Copy link
Member Author

literalEval commented Dec 19, 2023

@palisadoes this guide at official Flutter site might do.
Though I am free now, so might write a full documentation tailored for our codebase.

@github-actions github-actions bot removed the no-issue-activity No issue activity label Dec 20, 2023
@Cioppolo14
Copy link

@literalEval That link will work. Can you add it to the CONTRIBUTING.md file & Talawa docs and we can close this issue out?

@literalEval
Copy link
Member Author

Sure. Will do tomorrow morning. Thanks.

@Cioppolo14
Copy link

@literalEval Can we close this?

@palisadoes
Copy link
Contributor

Closing. We'll add the link to the CONTRIBUTING.md file

@literalEval
Copy link
Member Author

Thanks.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working documentation Update documentation feature request test Testing application
Projects
None yet
Development

Successfully merging a pull request may close this issue.

6 participants