-
Notifications
You must be signed in to change notification settings - Fork 27
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
Tested the manual Taking up of challenges with relevant snapshots #50
Conversation
@SHITIZ-AGGARWAL when will my PR be reviewed and merged? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@Sakalya100 Refer this for how tests should be added. You have to make a new folder for snapshots. Also you have only added workflow. Refer this for seeing the formatting of tests and you have to include tests for every possible scenario.
@SHITIZ-AGGARWAL Okay I will make the necessary changes |
@Sakalya100 Update |
@SHITIZ-AGGARWAL will be done by tomorrow |
@Sakalya100 any update on this |
Closing due to inactivity |
Description
This PR is regarding feature request #11 in which test has been done successfully to check manual taking of various functionalities of challenges and relevant snapshots,gifs have been added
Related Issue
Fixes #11
Type of Contribution:
Checklist:
develop
branch.Screenshots (if applicable):