Skip to content

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 <sekharkumardash229@gmail.com>
Signed-off-by: sekhar kumar dash <sekharkumardash229@gmail.com>
Signed-off-by: sekhar kumar dash <sekharkumardash229@gmail.com>
Signed-off-by: sekhar kumar dash <sekharkumardash229@gmail.com>
@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

Signed-off-by: sekhar kumar dash <sekharkumardash229@gmail.com>
Signed-off-by: sekhar kumar dash <sekharkumardash229@gmail.com>
Signed-off-by: sekhar kumar dash <sekharkumardash229@gmail.com>
Signed-off-by: sekhar kumar dash <sekharkumardash229@gmail.com>
Signed-off-by: sekhar kumar dash <sekharkumardash229@gmail.com>
Signed-off-by: sekhar kumar dash <sekharkumardash229@gmail.com>
@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 <sekharkumardash229@gmail.com>
Signed-off-by: sekhar kumar dash <sekharkumardash229@gmail.com>
Signed-off-by: sekhar kumar dash <sekharkumardash229@gmail.com>
Signed-off-by: sekhar kumar dash <sekharkumardash229@gmail.com>
Signed-off-by: sekhar kumar dash <sekharkumardash229@gmail.com>
Signed-off-by: sekhar kumar dash <sekharkumardash229@gmail.com>
Signed-off-by: sekhar kumar dash <sekharkumardash229@gmail.com>
@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
@keploy
Copy link

keploy bot commented Apr 15, 2025

Nice Pr 😄. We currently support Unit Test Generation only for Go projects.
Stay tuned – support for more languages is coming soon! 🚀

@keploy-staging
Copy link

Nice Pr 😄. We currently support Unit Test Generation only for Go projects.
Stay tuned – support for more languages is coming soon! 🚀

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