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

Unittests for theme_view_model.dart #1120

Closed
palisadoes opened this issue Jan 16, 2022 · 3 comments · Fixed by #1202
Closed

Unittests for theme_view_model.dart #1120

palisadoes opened this issue Jan 16, 2022 · 3 comments · Fixed by #1202
Assignees
@umang-sinha
Labels
good first issue Good for newcomers test Testing application

Comments

@palisadoes
Copy link
Contributor

palisadoes commented Jan 16, 2022 •
edited

We will need unittests done for all methods, classes and functions found in this file.

Any widgets referenced in this file must also have unittests done

theme_view_model.dart

This file can be found in the lib/view_model directory

Parent Issue:

PR Acceptance Criteria
@palisadoes palisadoes added test Testing application points 02 wip Work in Progress labels Jan 16, 2022
@github-actions github-actions bot added parent The issue is generating more sub-issues unapproved Unapproved, needs to be triaged labels Jan 16, 2022
@palisadoes palisadoes mentioned this issue Jan 16, 2022
Closed
@palisadoes palisadoes added good first issue Good for newcomers and removed wip Work in Progress parent The issue is generating more sub-issues labels Jan 16, 2022
@umang-sinha
Copy link

hey I am working on this. Please assign it to me

@palisadoes palisadoes removed the unapproved Unapproved, needs to be triaged label Jan 23, 2022
@palisadoes
Copy link
Contributor Author

Please see the updated criteria in the comments

@ahluwaliatikant
Copy link
Contributor

ahluwaliatikant commented Feb 11, 2022 •
edited

Hey @umang-sinha, are you still working on this? @palisadoes, Can I take this up?

@ahluwaliatikant ahluwaliatikant mentioned this issue Feb 11, 2022
Merged
Ayush0Chaudhary pushed a commit to Ayush0Chaudhary/talawa that referenced this issue Mar 29, 2023
@BhuvaneshPatil
First user default super admin (PalisadoesFoundation#1120)
7e6ef89 
* adds function to create super admin from env email

* adds test for super admin creation failing scene

* changes in INSTALLATION.md

* adds env variable to workflow

* adds toc and await

* minor changes

* mocks the env variable

* refactoring for sign up

* changes undefined to USER

* fixes typo

* updates installation for admin email
@palisadoes palisadoes mentioned this issue Nov 26, 2023
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@umang-sinha umang-sinha

Labels
good first issue Good for newcomers test Testing application
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Added Tests for theme_view_model ahluwaliatikant/talawa
3 participants
@palisadoes @ahluwaliatikant @umang-sinha