Add common TextMapGetter test

instrumentation/grpc-1.6/library/src/main/java/io/opentelemetry/instrumentation/grpc/v1_6/GrpcRequestGetter.java

@@ -10,14 +10,19 @@
 import io.grpc.Metadata;
 import io.opentelemetry.context.propagation.TextMapGetter;
 import java.util.Iterator;
+import java.util.stream.Collectors;
 import javax.annotation.Nullable;
 
 enum GrpcRequestGetter implements TextMapGetter<GrpcRequest> {
   INSTANCE;
 
   @Override
   public Iterable<String> keys(GrpcRequest request) {
-    return request.getMetadata().keys();
+    // Filter out HTTP/2 pseudo-headers (starting with ':') as they cannot be
+    // accessed via Metadata.Key.of() and would cause IllegalArgumentException
+    return request.getMetadata().keys().stream()
+        .filter(key -> !key.startsWith(":"))
+        .collect(Collectors.toList());
   }
 
   @Override

testing-common/src/main/java/io/opentelemetry/instrumentation/testing/LibraryTestRunner.java

@@ -25,6 +25,7 @@
 import io.opentelemetry.instrumentation.testing.provider.TestLogRecordExporterComponentProvider;
 import io.opentelemetry.instrumentation.testing.provider.TestMetricExporterComponentProvider;
 import io.opentelemetry.instrumentation.testing.provider.TestSpanExporterComponentProvider;
+import io.opentelemetry.instrumentation.testing.util.KeysVerifyingPropagator;
 import io.opentelemetry.sdk.OpenTelemetrySdk;
 import io.opentelemetry.sdk.common.CompletableResultCode;
 import io.opentelemetry.sdk.logs.SdkLoggerProvider;
@@ -102,9 +103,10 @@ public final class LibraryTestRunner extends InstrumentationTestRunner {
                     .build())
             .setPropagators(
                 ContextPropagators.create(
-                    TextMapPropagator.composite(
-                        W3CTraceContextPropagator.getInstance(),
-                        W3CBaggagePropagator.getInstance())))
+                    new KeysVerifyingPropagator(
+                        TextMapPropagator.composite(
+                            W3CTraceContextPropagator.getInstance(),
+                            W3CBaggagePropagator.getInstance()))))
             .buildAndRegisterGlobal();
     openTelemetry = wrap(openTelemetrySdk);
   }

testing-common/src/main/java/io/opentelemetry/instrumentation/testing/util/KeysVerifyingPropagator.java

@@ -0,0 +1,48 @@
+/*
+ * Copyright The OpenTelemetry Authors
+ * SPDX-License-Identifier: Apache-2.0
+ */
+
+package io.opentelemetry.instrumentation.testing.util;
+
+import io.opentelemetry.context.Context;
+import io.opentelemetry.context.propagation.TextMapGetter;
+import io.opentelemetry.context.propagation.TextMapPropagator;
+import io.opentelemetry.context.propagation.TextMapSetter;
+import java.util.Collection;
+import javax.annotation.Nullable;
+
+/**
+ * A TextMapPropagator wrapper that verifies TextMapGetter methods can be called without errors.
+ *
+ * <p>This propagator wraps another propagator and calls methods during extraction. This catches
+ * compatibility issues where underlying library APIs change (e.g., NoSuchMethodError).
+ */
+public final class KeysVerifyingPropagator implements TextMapPropagator {
+  private final TextMapPropagator delegate;
+
+  public KeysVerifyingPropagator(TextMapPropagator delegate) {
+    this.delegate = delegate;
+  }
+
+  @Override
+  public Collection<String> fields() {
+    return delegate.fields();
+  }
+
+  @Override
+  public <C> void inject(Context context, C carrier, TextMapSetter<C> setter) {
+    delegate.inject(context, carrier, setter);
+  }
+
+  @Override
+  public <C> Context extract(Context context, @Nullable C carrier, TextMapGetter<C> getter) {
+    // exercise methods to verify no errors
+    Iterable<String> keys = getter.keys(carrier);
+    if (keys != null) {
+      keys.iterator().forEachRemaining(key -> getter.get(carrier, key));
+    }
+
+    return delegate.extract(context, carrier, getter);
+  }
+}

testing/agent-exporter/src/main/java/io/opentelemetry/javaagent/testing/exporter/AgentTestingExporterConfigSupplier.java

@@ -24,6 +24,7 @@ private static Map<String, String> getTestProperties() {
     properties.put("otel.logs.exporter", "none");
     properties.put("otel.metrics.exporter", "none");
     properties.put("otel.traces.exporter", "none");
+    properties.put("otel.propagators", "testing-verifying-propagator");
     return properties;
   }
 }

testing/agent-exporter/src/main/java/io/opentelemetry/javaagent/testing/propagator/KeysVerifyingPropagatorProvider.java

@@ -0,0 +1,32 @@
+/*
+ * Copyright The OpenTelemetry Authors
+ * SPDX-License-Identifier: Apache-2.0
+ */
+
+package io.opentelemetry.javaagent.testing.propagator;
+
+import com.google.auto.service.AutoService;
+import io.opentelemetry.api.baggage.propagation.W3CBaggagePropagator;
+import io.opentelemetry.api.trace.propagation.W3CTraceContextPropagator;
+import io.opentelemetry.context.propagation.TextMapPropagator;
+import io.opentelemetry.javaagent.testing.util.KeysVerifyingPropagator;
+import io.opentelemetry.sdk.autoconfigure.spi.ConfigProperties;
+import io.opentelemetry.sdk.autoconfigure.spi.ConfigurablePropagatorProvider;
+
+/** Provides a propagator that wraps the default W3C propagators with additional verification. */
+@AutoService(ConfigurablePropagatorProvider.class)
+public final class KeysVerifyingPropagatorProvider implements ConfigurablePropagatorProvider {
+  @Override
+  public TextMapPropagator getPropagator(ConfigProperties configProperties) {
+    TextMapPropagator composite =
+        TextMapPropagator.composite(
+            W3CTraceContextPropagator.getInstance(), W3CBaggagePropagator.getInstance());
+
+    return new KeysVerifyingPropagator(composite);
+  }
+
+  @Override
+  public String getName() {
+    return "testing-verifying-propagator";
+  }
+}

testing/agent-exporter/src/main/java/io/opentelemetry/javaagent/testing/util/KeysVerifyingPropagator.java

@@ -0,0 +1,50 @@
+/*
+ * Copyright The OpenTelemetry Authors
+ * SPDX-License-Identifier: Apache-2.0
+ */
+
+package io.opentelemetry.javaagent.testing.util;
+
+import io.opentelemetry.context.Context;
+import io.opentelemetry.context.propagation.TextMapGetter;
+import io.opentelemetry.context.propagation.TextMapPropagator;
+import io.opentelemetry.context.propagation.TextMapSetter;
+import java.util.Collection;
+import javax.annotation.Nullable;
+
+/**
+ * A TextMapPropagator wrapper that verifies TextMapGetter methods can be called without errors.
+ *
+ * <p>This propagator wraps another propagator and calls methods during extraction. This catches
+ * compatibility issues where underlying library APIs change (e.g., NoSuchMethodError).
+ *
+ * <p>Note: This is a copy of the class in testing-common to avoid dependency conflicts
+ */
+public final class KeysVerifyingPropagator implements TextMapPropagator {
+  private final TextMapPropagator delegate;
+
+  public KeysVerifyingPropagator(TextMapPropagator delegate) {
+    this.delegate = delegate;
+  }
+
+  @Override
+  public Collection<String> fields() {
+    return delegate.fields();
+  }
+
+  @Override
+  public <C> void inject(Context context, C carrier, TextMapSetter<C> setter) {
+    delegate.inject(context, carrier, setter);
+  }
+
+  @Override
+  public <C> Context extract(Context context, @Nullable C carrier, TextMapGetter<C> getter) {
+    // exercise methods to verify no errors
+    Iterable<String> keys = getter.keys(carrier);
+    if (keys != null) {
+      keys.iterator().forEachRemaining(key -> getter.get(carrier, key));
+    }
+
+    return delegate.extract(context, carrier, getter);
+  }
+}