Skip to content

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

Open
Sekhar-Kumar-Dash wants to merge 20 commits into
keploy:mainfrom
Sekhar-Kumar-Dash:master2
Open

adding the unittest for wrapped node fetch function and its github action#82
Sekhar-Kumar-Dash wants to merge 20 commits into
keploy:mainfrom
Sekhar-Kumar-Dash:master2

Conversation

@Sekhar-Kumar-Dash

Copy link
Copy Markdown
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

re-Tick commented Mar 29, 2023

Copy link
Copy Markdown
Contributor

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

re-Tick commented Mar 30, 2023

Copy link
Copy Markdown
Contributor

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

Sekhar-Kumar-Dash commented Mar 31, 2023

Copy link
Copy Markdown
Contributor Author

@re-Tick please review the PR

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

re-Tick commented Apr 3, 2023

Copy link
Copy Markdown
Contributor

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

keploy Bot commented Apr 15, 2025

Copy link
Copy Markdown

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
Copy Markdown

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