ScalaConsultants/mesmer

Fix tests flakiness due to delta aggregation temporality

Closed this issue · 1 comments

There are some Ignored tests with a comment: // TODO: Fix the test so that it works with DELTA aggregation
relied on cumulative nature of instruments of InMemoryMetricExporter but now it was changed internally in the AgentForTesting to be DELTA: https://github.com/open-telemetry/opentelemetry-java-instrumentation/pull/5923/files/9ea0823de46a4ac6597a3cd588f78fa3af573bb0#r856619839

To be fixed or replaced with something better

UPDATE: In general, all integration tests that wait for a metric value can be flaky - revise them and fix.

There is also // TODO: Make this test work again but with DELTA aggregation temporality comment