Skip to content

Commit

Permalink
Refactor assertMetrics function in cache_test.go
Browse files Browse the repository at this point in the history
  • Loading branch information
mantzas committed Jun 29, 2024
1 parent aaf3891 commit 7ec78bd
Showing 1 changed file with 0 additions and 13 deletions.
13 changes: 0 additions & 13 deletions component/http/cache/cache_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -1708,19 +1708,6 @@ func assertMetrics(t *testing.T, read *metricsdk.ManualReader) {
t.Error("unexpected metric")
}
}
// TODO: check the right metrics are gathered!!!
// assert.Equal(t, 1, len(collectedMetrics.ScopeMetrics))
// assert.Equal(t, 1, len(collectedMetrics.ScopeMetrics[0].Metrics))
// assert.Equal(t, "http.cache.status", collectedMetrics.ScopeMetrics[0].Metrics[0].Name)
// assert.Equal(t, "http.cache.expiration", collectedMetrics.ScopeMetrics[0].Metrics[0].Name)

// for k, v := range expected.values {
// // v.additions
// // v.errors
// // v.evictions
// // v.hits
// // v.misses
// }
}

type testingCacheEntity struct {
Expand Down

0 comments on commit 7ec78bd

Please sign in to comment.