-
Notifications
You must be signed in to change notification settings - Fork 7
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
Corrections and enhancements to the Healthcheck endpoints #2313
Conversation
Codecov Report
@@ Coverage Diff @@
## develop #2313 +/- ##
===========================================
- Coverage 89.78% 89.77% -0.01%
===========================================
Files 324 324
Lines 5569 5567 -2
===========================================
- Hits 5000 4998 -2
Misses 569 569
Continue to review full report at Codecov.
|
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.
Looks good, apart from the lists of services that can be cached.
@@ -1,231 +1,298 @@ | |||
const proxyquire = require('proxyquire') | |||
|
|||
const _successDependencies = [ |
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.
Optional: using _ notation for private is not something I have seen across the rest of the codebase
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.
Based on lack of time I don't want to block 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.
Looks good, added optional comment regarding prefixes for private variables.
0fcc4ef
to
687d87a
Compare
The purpose of the microservice check is solely to provide a way for CloudFoundry components to be able to tell whether or not the router etc has mounted properly. If CF requires checks of other services it will undertake those directly.
687d87a
to
c454784
Compare
Description of change
This PR addresses a few things:
this
and thebeforeEach()
antipattern from unit tests