Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

Commit02b3828

Browse files
committed
test(coderd/prometheusmetrics/insights): improveTestCollectInsights
1 parent0087e9a commit02b3828

File tree

1 file changed

+13
-5
lines changed

1 file changed

+13
-5
lines changed

‎coderd/prometheusmetrics/insights/metricscollector_test.go

Lines changed: 13 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -9,6 +9,7 @@ import (
99
"testing"
1010
"time"
1111

12+
"github.com/google/go-cmp/cmp"
1213
"github.com/google/uuid"
1314
"github.com/prometheus/client_golang/prometheus"
1415
io_prometheus_client"github.com/prometheus/client_model/go"
@@ -110,7 +111,7 @@ func TestCollectInsights(t *testing.T) {
110111
UserID:user.UserID,
111112
WorkspaceID:workspace.ID,
112113
AgentID:resources[0].Agents[0].ID,
113-
AccessMethod:"terminal",
114+
AccessMethod:"path",
114115
SlugOrPort:"golden-slug",
115116
SessionID:uuid.New(),
116117
SessionStartedAt:time.Now().Add(-3*time.Minute),
@@ -163,13 +164,16 @@ func TestCollectInsights(t *testing.T) {
163164
require.NoError(t,err)
164165

165166
collected:=map[string]int{}
166-
assert.Eventuallyf(t,func()bool {
167+
ok:=assert.Eventuallyf(t,func()bool {
167168
// When
168169
metrics,err:=registry.Gather()
169-
require.NoError(t,err)
170+
if!assert.NoError(t,err) {
171+
returnfalse
172+
}
170173

171174
// Then
172175
for_,metric:=rangemetrics {
176+
t.Logf("metric: %s: %#v",metric.GetName(),metric)
173177
switchmetric.GetName() {
174178
case"coderd_insights_applications_usage_seconds","coderd_insights_templates_active_users","coderd_insights_parameters":
175179
for_,m:=rangemetric.Metric {
@@ -180,12 +184,16 @@ func TestCollectInsights(t *testing.T) {
180184
collected[key]=int(m.Gauge.GetValue())
181185
}
182186
default:
183-
require.FailNowf(t,"unexpected metric collected","metric: %s",metric.GetName())
187+
assert.Failf(t,"unexpected metric collected","metric: %s",metric.GetName())
184188
}
185189
}
186190

187191
returninsightsMetricsAreEqual(golden,collected)
188-
},testutil.WaitMedium,testutil.IntervalFast,"template insights are inconsistent with golden files, got: %v",collected)
192+
},testutil.WaitMedium,testutil.IntervalFast,"template insights are inconsistent with golden files")
193+
if!ok {
194+
diff:=cmp.Diff(golden,collected)
195+
assert.Empty(t,diff,"template insights are inconsistent with golden files (-golden +collected)")
196+
}
189197
}
190198

191199
funcmetricLabelAsString(m*io_prometheus_client.Metric)string {

0 commit comments

Comments
 (0)

[8]ページ先頭

©2009-2025 Movatter.jp