-
Notifications
You must be signed in to change notification settings - Fork 357
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
Improve test command coverage for networking use cases #382
Comments
I would like to work on this @noaabarki |
@Kanika637, it's yours 💪🏻 |
@Kanika637 Are you still working on this issue, or can we reassign it?😊 |
Hey @adifay, actually I could not get hold of the issue. You can reassign it, sorry for the inconvenience. |
Hey @noaabarki, can you send me the slack invitation, I need to troubleshoot some issues I have while installing datree, and also to discuss this GitHub issue, I want to take it up. |
Hi @KiranSatyaRaj, you got it! This is the invite URL for Datree Slack channel: https://join.slack.com/t/datreecommunity/shared_invite/zt-1kaxe67uc-o0uO3EuK3kxjV3PSeqe~ZA. |
Hey @noaabarki, I didn't work with Git much and had less idea on how to deal with it, all these days I was learning Git and blogging it along the way, I'm back now and will start working on this issue, so far I got a hold of this if there's anything I'm stuck with I'll get back to you, thanks Noaa |
Hey @noaabarki Can I work on this if no one is working? What I understand from the issue is that- We have to add tests based on- Test case 2nd- Test case 3th- having a small doubt in which directory I need to create the test file? Like We need to have a test file in which the test scene will be there |
Describe the solution you'd like
Add tests for networking failures flows in test command
datree test <path>
Basic use cases to cover:
Failed to create evaluation
CreateEvaluation
Evaluate
Failed to request evaluation
CreateEvaluation
Evaluate
with returnedevaluationID
Requirements
Golang basic level.
The text was updated successfully, but these errors were encountered: