Verified Commit e5db9e47 authored by aguestuser's avatar aguestuser

[340] fix bug in `testApp.metricsResource`

* it omitted a `gauges` field, causing integration tests that called
  `metrics.setGauge` to incorrectly (and confusingly!) fail
parent a79b8454
......@@ -43,6 +43,7 @@ const socketPoolResource = async () => {
const metricsResource = () => {
const counterStub = { labels: () => ({ inc: () => null }) }
const histogramStub = { labels: () => ({ observe: () => null }) }
const gaugueStub = { labels: () => ({ set: () => null }) }
return {
run: () => ({
register: {
......@@ -57,6 +58,9 @@ const metricsResource = () => {
histograms: {
MESSAGE_ROUNDTRIP: histogramStub,
},
gauges: {
CHANNEL_HEALTH: gaugueStub,
},
}),
}
}
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment