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.

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

[Tests]: Write Test for beacon project. #46

Closed
8 tasks
devansh12b2 opened this issue Dec 29, 2021 · 4 comments · Fixed by #114 · May be fixed by #115
Closed
8 tasks

[Tests]: Write Test for beacon project. #46

devansh12b2 opened this issue Dec 29, 2021 · 4 comments · Fixed by #114 · May be fixed by #115

Comments

@devansh12b2
Copy link
Contributor

devansh12b2 commented Dec 29, 2021 •
edited by nb9960
Loading

The following architecture needs to be followed while writing tests:

  • model_tests

  • service_tests

  • widget_tests

  • viewmodel_tests

  • Login Screen

  • Home page

  • beacon_card.dart

  • create_join_dialog.dart

  • dialog_boxes.dart

  • hike_button.dart

  • hike_screen_widget.dart

  • shape_painter.dart

For the Hike screen, MVVM architecture isn't followed so I am creating an issue for refactoring it, once it's done then tests can be added for that screen too.

Feel free to add/suggest anything that you think is missing.
@devansh12b2
Copy link
Contributor Author

I will like to work on 1 of these.

@Aadeesh11
Copy link
Contributor

Aadeesh11 commented Dec 29, 2021 •
edited
Loading

I would also like to write some tests.

@ItsAdityaKSingh
Copy link
Collaborator

I too would like to work on writing these tests!

@ItsAdityaKSingh ItsAdityaKSingh mentioned this issue Dec 29, 2021
Closed
6 tasks
@nb9960 nb9960 changed the title Write Test for beacon project. [Tests]: Write Test for beacon project. Dec 29, 2021
@nb9960
Copy link
Collaborator

nb9960 commented Dec 29, 2021

Edited the above list, feel free to comment here for the specific widget/screen/model/query/mutation you want to work on (you can add to the list too), will assign you by mentioning there. But remember only one particular task at a time :)

@devansh12b2 @Aadeesh11 @ItsAdityaKSingh @Literal-Eval

@Aadeesh11 Aadeesh11 mentioned this issue Feb 4, 2022
Merged
This was referenced Feb 4, 2022
Open
Closed
@nb9960 nb9960 linked a pull request Feb 12, 2022 that will close this issue
Test: Model test (Beacon and User model). #114
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
4 participants
@nb9960 @Aadeesh11 @ItsAdityaKSingh @devansh12b2