-
-
Notifications
You must be signed in to change notification settings - Fork 497
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
Fixes#1247 Add the test for SingUpDetail screen #1430
Fixes#1247 Add the test for SingUpDetail screen #1430
Conversation
ff8c7e0
to
c2681a3
Compare
@noman2002 @palisadoes @SiddheshKukade @TheHazeEffect Please review the PR mentors. |
@Ayush0Chaudhary Please fix the failing tests. You have probably not updated your local branch with the latest version of the code base. |
I merged the latest commit from develop to my branch @palisadoes @noman2002 |
fdd35f6
to
aa8bdd2
Compare
@palisadoes @noman2002 you can check it now!! |
test/widget_tests/after_auth_screens/app_settings/app_setting_page_test.dart
Outdated
Show resolved
Hide resolved
Codecov Report
📣 This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more @@ Coverage Diff @@
## develop #1430 +/- ##
===========================================
+ Coverage 68.38% 70.08% +1.70%
===========================================
Files 144 144
Lines 7164 7164
===========================================
+ Hits 4899 5021 +122
+ Misses 2265 2143 -122
📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
@noman2002 I added them back |
@noman2002 sir, what about this PR?? |
Yes we are good to go. |
What kind of change does this PR introduce?
Issue Number:
Fixes #1247
Did you add tests for your changes?
Yes, I added the unittest only
Summary
I added the test for sign up detail screen
Does this PR introduce a breaking change?
no
Have you read the contributing guide?
yes