-
-
Notifications
You must be signed in to change notification settings - Fork 504
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 unit/ widget testing of the App. #326
Comments
I can help with writing widget tests. |
Hey @piyushpradhan ! Let's wait for the issue to get approved. If it gets approved, as I will be also working on this please update with widget are you working on in order to avoid any duplicate work. |
Yeah, sure. We'll do that. |
Agreed, let's add some overflow tests as well. Assigning @arteevraina |
@jordanliu Should I add overflow tests for all the screen in a single PR or handle create different PRs for different screens? |
@arteevraina app testing is much needed issue to be fixed. |
Looks good ! I am currently writing widget tests for |
can i help @arteevraina @Sagar2366 ? |
Hi @shikharsaurav! You can contribute unit/widget tests but make sure to update here to avoid any duplicacy of work. I am waiting for #462 to merge after that I will add some more tests for |
Hi @arteevraina , I am trying to write tests for newsArticle.dart and newsfeed.dart. |
Hi @utkarshshendge ! It would be better if you handle one widget in a single PR. So that other participants can also get involved :) I see that test for newsArticle has already been written. Please cross check before sending a patch in order to avoid any duplicate work. In the meantime, I will write tests for |
In order to write tests for |
Hey @arteevraina could you please add link to the test folder in this comment #326 (comment) ? |
Im adding widget tests for |
I'm adding tests for |
I will be working on the widget test for |
* added role based authorization directive * fixes
Improving unit/widget testing of the app. Currently, the tests have been only been written for validators used in
Register
andLogin
Screens. I will help to improve the testing of the app and which would help to avoid any breaking changes in the future.Folder: https://github.com/PalisadoesFoundation/talawa/tree/master/test
Other Information.
If mentors/ participants have anything to add to this issue. Please provide your valuable suggestions. I would keep that in mind while sending the patches. If you want to add any widget/unit test please update here before working.
The text was updated successfully, but these errors were encountered: