-
Notifications
You must be signed in to change notification settings - Fork 282
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
test(cmd-api-server): jestify grpc-proto-gen-ts-client-healthcheck test #3588
base: main
Are you sure you want to change the base?
Conversation
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.
Thanks, left one small comment
// await t.doesNotReject(responsePromise, "No error in healthcheck OK"); | ||
// const res = await responsePromise; | ||
|
||
// const resHc = res?.toObject(); | ||
|
||
// t.ok(resHc, `healthcheck response truthy OK`); | ||
// t.ok(resHc?.createdAt, `resHc.createdAt truthy OK`); | ||
// t.ok(resHc?.memoryUsage, `resHc.memoryUsage truthy OK`); | ||
// t.ok(resHc?.memoryUsage?.rss, `resHc.memoryUsage.rss truthy OK`); | ||
// t.ok(resHc?.success, `resHc.success truthy OK`); | ||
// t.end(); |
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.
Can this commented code be removed?
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.
@outSH Oops, thank you for catching that!
Edit: Removed the comments just now!
1. Also fixing a minor bug in config/config-service-example-config-validity.test.ts where the path of the temporary files generated by the test were being saved in the wrong directory instead of $PROJECT_ROOT/.tmp/... Signed-off-by: Peter Somogyvari <peter.somogyvari@accenture.com>
7ad630a
to
3411b82
Compare
where the path of the temporary files generated by the test were being saved
in the wrong directory instead of $PROJECT_ROOT/.tmp/...
Signed-off-by: Peter Somogyvari peter.somogyvari@accenture.com
Pull Request Requirements
upstream/main
branch and squashed into single commit to help maintainers review it more efficient and to avoid spaghetti git commit graphs that obfuscate which commit did exactly what change, when and, why.-s
flag when usinggit commit
command. You may refer to this link for more information.Character Limit
A Must Read for Beginners
For rebasing and squashing, here's a must read guide for beginners.