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

adding the unittest for wrapped node fetch function and its github action #82

Open
wants to merge 20 commits into
base: main
Choose a base branch
from

Conversation

Sekhar-Kumar-Dash
Copy link
Contributor

Related Issue

Info about Issue or bug
Closes: keploy/keploy#354

added a unittest for wrappednodefetch function and added github actions.
note:the unittest is inside the folder so the commit message for this is set as test:adding a folder

Signed-off-by: sekhar kumar dash <[email protected]>
Signed-off-by: sekhar kumar dash <[email protected]>
Signed-off-by: sekhar kumar dash <[email protected]>
Signed-off-by: sekhar kumar dash <[email protected]>
@re-Tick
Copy link
Contributor

re-Tick commented Mar 29, 2023

Hi @Sekhar-Kumar-Dash, the problem in commit messages is that a space is missing after ":".
Example: test: adding a folder
You can use this for commit messages

@re-Tick
Copy link
Contributor

re-Tick commented Mar 30, 2023

The unit test filename should be of the format ".test.ts"

Signed-off-by: sekhar kumar dash <[email protected]>
Signed-off-by: sekhar kumar dash <[email protected]>
Signed-off-by: sekhar kumar dash <[email protected]>
Signed-off-by: sekhar kumar dash <[email protected]>
Signed-off-by: sekhar kumar dash <[email protected]>
Signed-off-by: sekhar kumar dash <[email protected]>
Signed-off-by: sekhar kumar dash <[email protected]>
@Sekhar-Kumar-Dash
Copy link
Contributor Author

Sekhar-Kumar-Dash commented Mar 31, 2023

@re-Tick please review the PR

@nehagup nehagup requested a review from re-Tick April 3, 2023 07:34
@re-Tick
Copy link
Contributor

re-Tick commented Apr 3, 2023

LGTM. Please add the link in the GSOC readme

@re-Tick re-Tick added Don't Merge Not to be merged until gsoc results Accepted PR is reviewed and Accepted labels Apr 3, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Accepted PR is reviewed and Accepted Don't Merge Not to be merged until gsoc results
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[test]: add unit tests for octokit module
2 participants