newrelic / newrelic-telemetry-sdk-java

Java library for sending telemetry data to New Relic
Apache License 2.0
41 stars 37 forks source link

Freshen integration tests #293

Closed tmancill closed 2 years ago

tmancill commented 2 years ago

Hi! I ran into a few failures attempting to run the integration tests on an Apple M1 - e.g. org.mockserver.client.SocketConnectionException: Channel handler removed before valid response has been received - that resolved as soon as I updated the org.mock-server and org.testcontainers dependencies.

As part of updating org.mock-server, I had to update the MediaType passed for the MockServerClient to org.mockserver.model.MediaType instead of com.google.common.net.MediaType.

Finally, the SpanApiIntegrationTest was failing due to the request body including a decimal point for duration.ms. I'm assuming this is safe and not related to the other updates, but can't say for sure because I couldn't run the tests prior to the updates.