improve handling of general properties, signal if declarative config is available at all

Author: zeitlinger

javaagent-extension-api/src/test/java/io/opentelemetry/javaagent/extension/internal/DeclarativeConfigPropertiesBridgeTest.java

@@ -6,7 +6,6 @@
 package io.opentelemetry.javaagent.extension.internal;
 
 import static org.assertj.core.api.Assertions.assertThat;
-import static org.assertj.core.data.MapEntry.entry;
 
 import io.opentelemetry.sdk.autoconfigure.spi.ConfigProperties;
 import io.opentelemetry.sdk.extension.incubator.fileconfig.DeclarativeConfiguration;
@@ -20,7 +19,6 @@
 import java.util.Map;
 import java.util.Objects;
 import org.junit.jupiter.api.BeforeEach;
-import org.junit.jupiter.api.DisplayName;
 import org.junit.jupiter.api.Test;
 
 class DeclarativeConfigPropertiesBridgeTest {
@@ -121,24 +119,6 @@ void vendor() {
     assertThat(bridge.getBoolean("acme.full_name.preserved")).isTrue();
   }
 
-  @DisplayName("properties from the general instrumentation section in config.yaml")
-  @Test
-  void general() {
-    assertThat(bridge.getMap("otel.instrumentation.common.peer-service-mapping"))
-        .containsOnly(entry("1.2.3.4", "FooService"), entry("2.3.4.5", "BarService"));
-    // not supported in SDK yet (this is strictly typed)
-    //    assertThat(bridge.getList("otel.instrumentation.http.known-methods"))
-    //        .containsExactly("GET", "POST", "PUT");
-    assertThat(bridge.getList("otel.instrumentation.http.client.capture-request-headers"))
-        .containsExactly("Content-Type", "Accept");
-    assertThat(bridge.getList("otel.instrumentation.http.client.capture-response-headers"))
-        .containsExactly("Content-Type", "Content-Encoding");
-    assertThat(bridge.getList("otel.instrumentation.http.server.capture-request-headers"))
-        .containsExactly("Content-Type", "Accept");
-    assertThat(bridge.getList("otel.instrumentation.http.server.capture-response-headers"))
-        .containsExactly("Content-Type", "Content-Encoding");
-  }
-
   @Test
   void common() {
     assertThat(bridge.getBoolean("otel.instrumentation.common.default-enabled")).isFalse();