Skip to content

Commit b56c97f

Browse files
committed
fix: improve tests
1 parent 066a57b commit b56c97f

File tree

2 files changed

+3
-10
lines changed

2 files changed

+3
-10
lines changed

router-tests/prometheus_event_metrics_test.go

Lines changed: 2 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -277,10 +277,6 @@ func TestFlakyEventMetrics(t *testing.T) {
277277
})
278278
require.JSONEq(t, `{"data":{"updateAvailability":{"id":3}}}`, resOne.Body)
279279

280-
// Trigger the second subscription via NATS
281-
err = xEnv.NatsConnectionDefault.Publish(xEnv.GetPubSubName("employeeUpdated.3"), []byte(`{"id":3,"__typename": "Employee"}`))
282-
require.NoError(t, err)
283-
284280
err = xEnv.NatsConnectionDefault.Flush()
285281
require.NoError(t, err)
286282

@@ -304,7 +300,7 @@ func TestFlakyEventMetrics(t *testing.T) {
304300
destination := findMetricLabelByName(metrics, "messaging_destination_name")
305301
require.True(t, strings.HasSuffix(destination.GetValue(), "employeeUpdated.3"))
306302

307-
require.Equal(t, float64(2), metrics[0].Counter.GetValue())
303+
require.Equal(t, float64(1), metrics[0].Counter.GetValue())
308304
})
309305

310306
require.NoError(t, client.Close())
@@ -349,6 +345,7 @@ func TestFlakyEventMetrics(t *testing.T) {
349345
operation := findMetricLabelByName(metrics, "messaging_operation_name")
350346
require.Equal(t, "publish", operation.GetValue())
351347
require.Nil(t, findMetricLabelByName(metrics, "error_type"))
348+
352349
system := findMetricLabelByName(metrics, "messaging_system")
353350
require.Equal(t, "redis", system.GetValue())
354351

router-tests/telemetry/event_metrics_test.go

Lines changed: 1 addition & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -301,10 +301,6 @@ func TestFlakyEventMetrics(t *testing.T) {
301301
})
302302
require.JSONEq(t, `{"data":{"updateAvailability":{"id":3}}}`, resOne.Body)
303303

304-
// Trigger the second subscription via NATS
305-
err = xEnv.NatsConnectionDefault.Publish(xEnv.GetPubSubName("employeeUpdated.3"), []byte(`{"id":3,"__typename": "Employee"}`))
306-
require.NoError(t, err)
307-
308304
err = xEnv.NatsConnectionDefault.Flush()
309305
require.NoError(t, err)
310306

@@ -337,7 +333,7 @@ func TestFlakyEventMetrics(t *testing.T) {
337333
_, hasErr := attrs.Value(otelattrs.MessagingErrorType)
338334
require.False(t, hasErr)
339335

340-
require.Equal(t, int64(2), sum.DataPoints[0].Value)
336+
require.Equal(t, int64(1), sum.DataPoints[0].Value)
341337
})
342338

343339
require.NoError(t, client.Close())

0 commit comments

Comments
 (0)