test: setup CI and framework for unit tests #129
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
lsetiawan
reviewed
Oct 17, 2024
| @@ -0,0 +1,32 @@ | |||
| name: Frontend CI | |||
|
|
|||
| on: | |||
There was a problem hiding this comment.
Could you make this workflow to run only for changes within src/support_sphere
There was a problem hiding this comment.
Also, probably should run on push to main as well
lsetiawan
reviewed
Oct 17, 2024
| authService = AuthService(); | ||
| }); | ||
|
|
||
| group('AuthService', () { |
There was a problem hiding this comment.
What is group vs setUp? Could you provide a little bit of description for these two? Thanks!
parvatijay2901
pushed a commit
to parvatijay2901/post-disaster-comms
that referenced
this pull request
Oct 22, 2024
* test: setup CI and framework for unit tests * style: update dummy test * test: authService signUp codes tests * test: add build_runner for mocking test classes in CI testing * ci: run workflow only for frontend changes * docs: add description for test group and setup
parvatijay2901
pushed a commit
to parvatijay2901/post-disaster-comms
that referenced
this pull request
Oct 22, 2024
* test: setup CI and framework for unit tests * style: update dummy test * test: authService signUp codes tests * test: add build_runner for mocking test classes in CI testing * ci: run workflow only for frontend changes * docs: add description for test group and setup
lsetiawan
pushed a commit
that referenced
this pull request
Oct 23, 2024
* Update the function to be more DRY * test: setup CI and framework for unit tests (#129) * test: setup CI and framework for unit tests * style: update dummy test * test: authService signUp codes tests * test: add build_runner for mocking test classes in CI testing * ci: run workflow only for frontend changes * docs: add description for test group and setup * Pass the class instead of cubit --------- Co-authored-by: Parvati Jayakumar <parvati@Parvatis-MacBook-Pro.local> Co-authored-by: anujsinha3 <anuj3@uw.edu>
lsetiawan
added a commit
that referenced
this pull request
Oct 29, 2024
* Updated SignUp Button to remain grey till all the entries are valid * Updated Login Button to remain grey till all the entries are valid * test: setup CI and framework for unit tests (#129) * test: setup CI and framework for unit tests * style: update dummy test * test: authService signUp codes tests * test: add build_runner for mocking test classes in CI testing * ci: run workflow only for frontend changes * docs: add description for test group and setup * Move isAllFieldsFilled logic to LoginCubit * Move isAllFieldsFilled logic to SignupCubit * Update the button to activate only when it is valid and have all the fields filled in * refactor: Make hardcoded string for routes and app roles to constants (#148) * feat: Add data analysis framework (#153) * Fix isValid bug * Split field and status listeners in Signup/Login forms * Build both password/confirm password again when either of them change * Show Validation Error while filling out the fields * Update validateValue function/function call to make it more dry * Have Signup/Login button enable feature in the state rather than as a function * Have isAllFields as a state variable and update the way Login/Signup button enable state is called * Have isSignupButtonEnabled and isLoginButtonEnabled in cubit to avoid the flickering issue --------- Co-authored-by: Parvati Jayakumar <parvati@Parvatis-MacBook-Pro.local> Co-authored-by: anujsinha3 <anuj3@uw.edu> Co-authored-by: Don Setiawan <landungs@uw.edu>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
#92