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
add: support of custom mock name #1835
Conversation
@shivamsouravjha please resolve merge conflicts |
48ac9ef
to
680f216
Compare
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.
address the comments
pkg/service/replay/service.go
Outdated
@@ -63,3 +63,8 @@ type Telemetry interface { | |||
type RequestEmulator interface { | |||
SimulateRequest(ctx context.Context, appID uint64, tc *models.TestCase, testSetID string) (*models.HTTPResp, error) | |||
} | |||
type TestResult interface { | |||
TestRunStatus(ctx context.Context, status bool, testSetID string) | |||
MockName() string |
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.
function name cannot be a noun , how can I know what MockName, MockFile does and what are they are doing inside TestResult.
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.
Try to see if you use the same TestUtilInstance and add functions to it and use them.
92575fe
to
b530cf5
Compare
Signed-off-by: shivamsouravjha <shivamsouravjha@gmail.com>
84f5f7a
to
071363a
Compare
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.
Please address the comments
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.
ship it!
Related Issue
Closes: #[issue number that will be closed through this PR]
Describe the changes you've made
A clear and concise description of what you have done to successfully close your assigned issue. Any new files? or anything you feel to let us know!
Type of change
Please let us know if any test cases are added
Please describe the tests(if any). Provide instructions how its affecting the coverage.
Describe if there is any unusual behaviour of your code(Write
NA
if there isn't)A clear and concise description of it.
Checklist:
Screenshots (if any)