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.

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

added tests for PinnedPostPage #1280

Conversation

guptapriyanshu7
Copy link
Contributor

What kind of change does this PR introduce?

Added test for PinnedPostPage widget

Issue Number:

Fixes #1235

Did you add tests for your changes?

Yes

Have you read the contributing guide?

Yes

@codecov-commenter
Copy link

Codecov Report

Merging #1280 (5ef0054) into develop (615b5b6) will increase coverage by 0.51%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##           develop    #1280      +/-   ##
===========================================
+ Coverage    60.08%   60.59%   +0.51%     
===========================================
  Files          125      125              
  Lines         6363     6360       -3     
===========================================
+ Hits          3823     3854      +31     
+ Misses        2540     2506      -34     
Impacted Files Coverage Δ
lib/views/pre_auth_screens/set_url.dart 94.49% <0.00%> (-0.06%) ⬇️
lib/main.dart 1.56% <0.00%> (ø)
lib/view_model/lang_view_model.dart 100.00% <0.00%> (ø)
...model/pre_auth_view_models/set_url_view_model.dart 100.00% <0.00%> (ø)
.../pre_auth_screens/waiting_to_join_private_org.dart 100.00% <0.00%> (+1.66%) ⬆️
lib/widgets/custom_list_tile.dart 78.66% <0.00%> (+2.66%) ⬆️
lib/views/pre_auth_screens/change_password.dart 100.00% <0.00%> (+3.22%) ⬆️
lib/widgets/post_widget.dart 88.09% <0.00%> (+3.57%) ⬆️
lib/views/pre_auth_screens/recover.dart 100.00% <0.00%> (+4.16%) ⬆️
lib/models/post/post_model.dart 61.36% <0.00%> (+6.81%) ⬆️
... and 2 more

📣 Codecov can now indicate which changes are the most critical in Pull Requests. Learn more

@palisadoes
Copy link
Contributor

Do you know why pinned_post_page.dart not showing up in the coverage report?

@palisadoes
Copy link
Contributor

@Sagar2366 @DeltaHarbinger @TheHazeEffect please review.

@palisadoes palisadoes merged commit a4dc583 into PalisadoesFoundation:develop Mar 24, 2022
@Sagar2366 Sagar2366 added the test Testing application label Mar 24, 2022
@guptapriyanshu7 guptapriyanshu7 deleted the pinned_post_page_test#1235 branch June 2, 2022 14:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
test Testing application
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Views: Create tests for pinned_post_page.dart
4 participants