New Relic Micrometer Spring Legacy Updates

Updates for New Relic properties added in Micrometer v1.3.0 related to
micrometer-metrics#1588<micrometer-metrics#1588>

Author: neiljpowell

micrometer-spring-legacy/src/main/java/io/micrometer/spring/autoconfigure/export/newrelic/NewRelicProperties.java

@@ -22,11 +22,26 @@
  * {@link ConfigurationProperties} for configuring New Relic metrics export.
  *
  * @author Jon Schneider
+ * @author Neil Powell
  * @since 1.0.0
  */
 @ConfigurationProperties(prefix = "management.metrics.export.newrelic")
 public class NewRelicProperties extends StepRegistryProperties {
 
+    /**
+     * Whether to send the meter name as the event type instead of using the 'event-type'
+     * configuration property value. Can be set to 'true' if New Relic guidelines are not
+     * being followed or event types consistent with previous Spring Boot releases are
+     * required.
+     */
+    private boolean meterNameEventTypeEnabled;
+
+    /**
+     * The event type that should be published. This property will be ignored if
+     * 'meter-name-event-type-enabled' is set to 'true'.
+     */
+    private String eventType = "SpringBootSample";
+
     /**
      * New Relic API key.
      */
@@ -42,6 +57,22 @@ public class NewRelicProperties extends StepRegistryProperties {
      */
     private String uri = "https://insights-collector.newrelic.com";
 
+    public boolean isMeterNameEventTypeEnabled() {
+        return this.meterNameEventTypeEnabled;
+    }
+
+    public void setMeterNameEventTypeEnabled(boolean meterNameEventTypeEnabled) {
+        this.meterNameEventTypeEnabled = meterNameEventTypeEnabled;
+    }
+
+    public String getEventType() {
+        return this.eventType;
+    }
+
+    public void setEventType(String eventType) {
+        this.eventType = eventType;
+    }
+
     public String getApiKey() {
         return this.apiKey;
     }
@@ -65,5 +96,4 @@ public String getUri() {
     public void setUri(String uri) {
         this.uri = uri;
     }
-
 }

micrometer-spring-legacy/src/main/java/io/micrometer/spring/autoconfigure/export/newrelic/NewRelicPropertiesConfigAdapter.java

@@ -22,6 +22,7 @@
  * Adapter to convert {@link NewRelicProperties} to a {@link NewRelicConfig}.
  *
  * @author Jon Schneider
+ * @author Neil Powell
  */
 class NewRelicPropertiesConfigAdapter extends StepRegistryPropertiesConfigAdapter<NewRelicProperties>
     implements NewRelicConfig {
@@ -30,6 +31,16 @@ class NewRelicPropertiesConfigAdapter extends StepRegistryPropertiesConfigAdapte
         super(properties);
     }
 
+    @Override
+    public boolean meterNameEventTypeEnabled() {
+        return get(NewRelicProperties::isMeterNameEventTypeEnabled, NewRelicConfig.super::meterNameEventTypeEnabled);
+    }
+
+    @Override
+    public String eventType() {
+        return get(NewRelicProperties::getEventType, NewRelicConfig.super::eventType);
+    }
+
     @Override
     public String apiKey() {
         return get(NewRelicProperties::getApiKey, NewRelicConfig.super::apiKey);

micrometer-spring-legacy/src/test/java/io/micrometer/spring/autoconfigure/export/newrelic/NewRelicPropertiesTests.java

@@ -0,0 +1,52 @@
+/*
+ * Copyright 2012-2019 the original author or authors.
+ *
+ * Licensed under the Apache License, Version 2.0 (the "License");
+ * you may not use this file except in compliance with the License.
+ * You may obtain a copy of the License at
+ *
+ *      https://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+
+package io.micrometer.spring.autoconfigure.export.newrelic;
+
+import io.micrometer.newrelic.NewRelicConfig;
+import org.junit.jupiter.api.Test;
+
+import io.micrometer.spring.autoconfigure.export.properties.StepRegistryPropertiesTest;
+
+import static org.assertj.core.api.Assertions.assertThat;
+
+/**
+ * Tests for {@link NewRelicProperties}.
+ *
+ * @author Neil Powell
+ */
+public class NewRelicPropertiesTests extends StepRegistryPropertiesTest {
+
+    @Test
+    public void defaultValuesAreConsistent() {
+        NewRelicProperties properties = new NewRelicProperties();
+        NewRelicConfig config = (key) -> null;
+        assertStepRegistryDefaultValues(properties, config);
+        // apiKey and account are mandatory
+        assertThat(properties.getUri()).isEqualTo(config.uri());
+        assertThat(properties.isMeterNameEventTypeEnabled()).isEqualTo(config.meterNameEventTypeEnabled());
+    }
+
+    @Test
+    void eventTypeDefaultValueIsOverriden() {
+        NewRelicProperties properties = new NewRelicProperties();
+        NewRelicConfig config = (key) -> null;
+        assertThat(properties.getEventType()).isNotEqualTo(config.eventType());
+        assertThat(properties.getEventType()).isEqualTo("SpringBootSample");
+        assertThat(config.eventType()).isEqualTo("MicrometerSample");
+    }
+
+}