Skip to content

Commit

Permalink
Disable integration test
Browse files Browse the repository at this point in the history
Signed-off-by: Paschalis Tsilias <[email protected]>
  • Loading branch information
tpaschalis committed Nov 6, 2023
1 parent b0715a7 commit 329ff93
Showing 1 changed file with 11 additions and 6 deletions.
17 changes: 11 additions & 6 deletions integration-tests/tests/otlp-metrics/otlp_metrics_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -13,11 +13,16 @@ func TestOtlpMetrics(t *testing.T) {
var metricResponse common.MetricResponse
assert.EventuallyWithT(t, func(c *assert.CollectT) {
err := common.FetchDataFromURL(query, &metricResponse)
assert.NoError(c, err)
if assert.NotEmpty(c, metricResponse.Data.Result) {
assert.Equal(c, metricResponse.Data.Result[0].Metric.Name, "span_metrics_duration_bucket")
assert.Equal(c, metricResponse.Data.Result[0].Metric.TestName, "otlp_metrics")
assert.NotEmpty(c, metricResponse.Data.Result[0].Value.Value)
}
//assert.NoError(c, err)
//if assert.NotEmpty(c, metricResponse.Data.Result) {
// assert.NoError(c, err)
// assert.Equal(c, metricResponse.Data.Result[0].Metric.Name, "span_metrics_duration_bucket")
// assert.Equal(c, metricResponse.Data.Result[0].Metric.TestName, "otlp_metrics")
// assert.NotEmpty(c, metricResponse.Data.Result[0].Value.Value)
//}
// Disabling the above checks as part of the workaround in #5684.
// TODO(@ptodev, @tpaschalis) Fix this once the workaround in #5684 is removed.
assert.Error(c, err)
assert.Empty(c, metricResponse.Data.Result)
}, common.DefaultTimeout, common.DefaultRetryInterval, "Data did not satisfy the conditions within the time limit")
}

0 comments on commit 329ff93

Please sign in to comment.