Yili - unit test : WeeklySummariesReport /FormattedReport#3252
Conversation
✅ Deploy Preview for highestgoodnetwork-dev ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
rnavyaprabha
left a comment
rnavyaprabha
left a comment
There was a problem hiding this comment.
I have checked the code changes and also saw all the test cases written, it looks good to me .Also all the test cases passed, here is the screenshot from my local system:
barnaboss305
left a comment
barnaboss305
left a comment
There was a problem hiding this comment.
All tests pass when I run the script.
vasavi-139
left a comment
vasavi-139
left a comment
There was a problem hiding this comment.
All tests passed when I ran the script, and here is the screenshot from my local system for reference.
Nayakar-Rohith
left a comment
Nayakar-Rohith
left a comment
There was a problem hiding this comment.
I have reviewed this PR and successfully ran the unit tests for WeeklySummariesReport/FormattedReport. All test cases passed without any issues, and the implementation looks good. Approving this PR.
|
I ran npm test FormattedReport.test.js, and all tests passed successfully with no errors. The test coverage looks good, and everything is functioning as expected. |
|
I have tested this PR and all the test cases passed successfully and the test coverage is good. |
vikasreddynomula
left a comment
vikasreddynomula
left a comment
There was a problem hiding this comment.
The code looks good and all the test cases are cleared approving this PR
nikhilrouth
left a comment
nikhilrouth
left a comment
There was a problem hiding this comment.
Reviewed and tested. All unit test cases pass as expected. Verified that the FormattedReport component correctly handles different scenarios. Everything works as expected so approving this PR 3252.
|
All tests passed successfully and the code is great
|
pdp5
left a comment
pdp5
left a comment
There was a problem hiding this comment.
I have thoroughly tested the defined steps and ran all the test cases, all of which passed successfully. Therefore, I am approving this PR.
KurtisIvey
left a comment
KurtisIvey
left a comment
There was a problem hiding this comment.
Test looks to cover the essentials we'd need for this component and everything functions as intended. You cover ensuring key elements appear, role based rendering, missing data, and link handling properly.
nikhilpittala16
left a comment
nikhilpittala16
left a comment
There was a problem hiding this comment.
I have tested PR and all the text cases have passed.
|
Thank you all, merging! |
14 participants
Description
Unit test for src/components/WeeklySummariesReport /FormattedReport
Main changes explained:
New file: FormattedReport.test.js : unit test cases
How to test:
Screenshots or videos of changes: