-
Couldn't load subscription status.
- Fork 26
gh-44 remove/replace headers while recording tc #47
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
base: main
Are you sure you want to change the base?
Conversation
|
Hi @shreyanshshah27, please make changes for mock library to remove and replace fields |
Signed-off-by: Shreyansh Shah <shreyansh_shah@yahoo.com>
b939e50 to
f71f27a
Compare
Signed-off-by: Shreyansh Shah <shreyansh_shah@yahoo.com>
|
Hey @re-Tick Please review the PR, I've made the required changes in mock library to support this feature. |
mock/mock.ts
Outdated
| testId: conf.Name, | ||
| mocks: [], | ||
| fileExport: true, | ||
| Remove: conf.Remove, |
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.
This Remove and Replace should be added in record mode. Since we filtering during recording.
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.
You are correct. I've fixed this.
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.
LGTM.
Signed-off-by: Shreyansh Shah <shreyansh_shah@yahoo.com>
|
Nice Pr 😄. We currently support Unit Test Generation only for Go projects. |
1 similar comment
|
Nice Pr 😄. We currently support Unit Test Generation only for Go projects. |
closes keploy/keploy#395