GTBitsOfGood / angels-among-us

Rescue one until there are none
https://staging--angels-among-us.netlify.app
MIT License
4 stars 0 forks source link

Filter Posts by Covered #176

Closed uma-anand closed 1 year ago

uma-anand commented 1 year ago

Filter Posts by Covered

Issue Number(s): #167.

What does this PR change and why? We want to allow admins and content creators to be able to show 3 possible states of posts: only covered posts, only uncovered posts, and covered and uncovered posts. This is to enable users to quickly find a post to delete or uncover and to see the posts that volunteers are only able to view.

Checklist

Critical Changes

Testing

Enumerate steps to test the functionality of your ticket. This should include edge cases and testing of error handling, if applicable.

netlify[bot] commented 1 year ago

Deploy Preview for angels-among-us ready!

Name Link
Latest commit 1253ad3de8ce6333f8724d575a644e2ed1135cd2
Latest deploy log https://app.netlify.com/sites/angels-among-us/deploys/65318a80f448650007e7ef07
Deploy Preview https://deploy-preview-176--angels-among-us.netlify.app
Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

github-actions[bot] commented 1 year ago

Coverage report for web

St.:grey_question:
Category Percentage Covered / Total
πŸ”΄ Statements
28.31% (+0.96% πŸ”Ό)
464/1639
πŸ”΄ Branches
13.39% (+0.5% πŸ”Ό)
64/478
πŸ”΄ Functions
15.4% (+1.23% πŸ”Ό)
75/487
πŸ”΄ Lines
23.98% (+1.11% πŸ”Ό)
342/1426

Test suite run success

40 tests passing in 5 suites.

Report generated by πŸ§ͺjest coverage report action from 1253ad3de8ce6333f8724d575a644e2ed1135cd2

github-actions[bot] commented 1 year ago

Coverage report for web

St.:grey_question:
Category Percentage Covered / Total
πŸ”΄ Statements
25.81% (+0.04% πŸ”Ό)
396/1534
πŸ”΄ Branches
12.25% (-0.79% πŸ”»)
56/457
πŸ”΄ Functions
13.02% (+0.02% πŸ”Ό)
59/453
πŸ”΄ Lines
20.97% (-0.33% πŸ”»)
278/1326
Show files with reduced coverage πŸ”»
|
St.:grey_question:
| File | Statements | Branches | Functions | Lines | | :----------------------------------------------------------------------------------------------------------------------------------------------------: | :----------------- | :---------------------------------------------------------- | :------------------------------------------------------ | :------------------------------------------------------ | :---------------------------------------------------------- | | πŸ”΄ | db/actions/User.ts |
0% (-44.44% πŸ”»)
| 100% |
0% (-25% πŸ”»)
|
0% (-44.44% πŸ”»)
| | πŸ”΄ | db/models/User.ts |
0% (-100% πŸ”»)
|
0% (-100% πŸ”»)
|
0% (-100% πŸ”»)
|
0% (-100% πŸ”»)
|

Test suite run success

36 tests passing in 3 suites.

Report generated by πŸ§ͺjest coverage report action from 73ea973124a640444530603e73e00e8916e92044

uma-anand commented 1 year ago

I couldn't get the toMatchObject to work as mentioned so I've used another way of doing it but it now does make sure all the fields are the same and not just the ID.