Rather than duplicate the whole method, could we convert the existing method
into a utility and have two tests which use that? Something like:
```
public void testSimpleStats() {
verifyStats(m -> (double) m.metricValue());
}
public void testDeprecatedMetricValueMethod() {
verifyStats(KafkaMetric::value);
}
private void verifyStats(Function<KafkaMetric, Double> metricValueFunc) {
// Existing testSimpleStats method modified to apply the function
}
```
When we get rid of the deprecated method, we could remove the function and use
the method directly. What do you think?
[ Full content available at: https://github.com/apache/kafka/pull/5626 ]
This message was relayed via gitbox.apache.org for [email protected]
