add LoggingMetricExporterAutoConfiguration to spring factories

[zeitlinger]

it was forgotten before

[jeanbisutti]

It may be useful to add automatic tests as it's done for LoggingSpanExporter

[zeitlinger]

It may be useful to add automatic tests as it's done for LoggingSpanExporter

adding external tests is quite an effort: I've done that for various spring configurations and versions in a distro - and I'm happy to contribute that - but it's quite big.

[jeanbisutti]

It may be useful to add automatic tests as it's done for LoggingSpanExporter

adding external tests is quite an effort: I've done that for various spring configurations and versions in a distro - and I'm happy to contribute that - but it's quite big.

There are several tests verifying the presence or absence of the LoggingSpanExporter bean, for different configurations. See this test for example. You could do the same thing for the LoggingMetricExporter bean.

[jeanbisutti]

It would be great to document this feature on https://opentelemetry.io/ after the next release

[zeitlinger]

It may be useful to add automatic tests as it's done for LoggingSpanExporter

adding external tests is quite an effort: I've done that for various spring configurations and versions in a distro - and I'm happy to contribute that - but it's quite big.

There are several tests verifying the presence or absence of the LoggingSpanExporter bean, for different configurations. See this test for example. You could do the same thing for the LoggingMetricExporter bean.

This PR is changing the spring factories, which are not used in the test you mentioned IIUC.

[jeanbisutti]

My point was about the tests on the autoconfiguration creating the LoggingMetricExporter bean.

The test class for this autoconfiguration in fact already exists (LoggingMetricExporterAutoConfigurationTest).