-
-
Notifications
You must be signed in to change notification settings - Fork 408
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
feat: mock data compress-decompress #209
Conversation
CLA Assistant Lite bot All contributors have signed the CLA ✍️ ✅ |
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.
Thank you and congratulations 🎉 for opening your very first pull request in keploy
I have read the CLA Document and I hereby sign the CLA |
@Sarthak160 @re-Tick please review |
Hii @Shivansh-yadav13, I have tried the changes locally for recording testcases in samples-go/gin-mongo app. The server is crashing with error-stack:
|
19256cb
to
5e22eaf
Compare
Hi @Shivansh-yadav13 . Are you still working on this part ?? I was trying to run my application on test mode after recording few tests and It was showing below stacktrace -
It's unable to fetch testcases , I tried to debug and found that it is unable to read data during unzipping |
6356102
to
c6f1546
Compare
Hey @Shivansh-yadav13 I have fixed this for test mode you can refer that to make changes in your branch |
@Sarthak160 should I close this PR? |
Or you can just add changes in your branch , I made a separate branch coz I didn't have access to your repo 🙂 |
Signed-off-by: Shivansh-yadav13 <yadavshivansh@gmail.com>
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. Thanks for the contribution @Shivansh-yadav13
Related Issue
Compressed the data on Encode wiriting mocks into yaml and decompressed on Decode
Type of change
Checklist:
Screenshots (if any)