use stable semconv in tests (#1098)

* use stable semconv when we can

* spotless

Author: SylvainJuge

custom/src/test/java/co/elastic/otel/ElasticAutoConfigurationCustomizerProviderTest.java

@@ -31,6 +31,7 @@
 import io.opentelemetry.sdk.metrics.data.AggregationTemporality;
 import io.opentelemetry.sdk.metrics.export.MetricExporter;
 import io.opentelemetry.sdk.resources.Resource;
+import io.opentelemetry.semconv.DeploymentAttributes;
 import io.opentelemetry.semconv.incubating.DeploymentIncubatingAttributes;
 import java.util.HashMap;
 import java.util.List;
@@ -148,7 +149,7 @@ void legacyDeploymentEnvironment() {
         .hasSize(3)
         .describedAs("when legacy attribute set agent should send both")
         .containsEntry(DeploymentIncubatingAttributes.DEPLOYMENT_ENVIRONMENT, "test")
-        .containsEntry(DeploymentIncubatingAttributes.DEPLOYMENT_ENVIRONMENT_NAME, "test")
+        .containsEntry(DeploymentAttributes.DEPLOYMENT_ENVIRONMENT_NAME, "test")
         .containsEntry("other", "other");
   }
 
@@ -157,15 +158,15 @@ void legacyDeploymentRemoveLegacyWhenBothSet() {
     Resource input =
         Resource.builder()
             .put(DeploymentIncubatingAttributes.DEPLOYMENT_ENVIRONMENT, "legacy")
-            .put(DeploymentIncubatingAttributes.DEPLOYMENT_ENVIRONMENT_NAME, "new")
+            .put(DeploymentAttributes.DEPLOYMENT_ENVIRONMENT_NAME, "new")
             .put("other", "other")
             .build();
     Resource resource = resourceProviders().apply(input, null);
     assertThat(resource.getAttributes())
         .hasSize(3)
         .describedAs("when both attributes are set, agent should send them as-is")
         .containsEntry(DeploymentIncubatingAttributes.DEPLOYMENT_ENVIRONMENT, "legacy")
-        .containsEntry(DeploymentIncubatingAttributes.DEPLOYMENT_ENVIRONMENT_NAME, "new")
+        .containsEntry(DeploymentAttributes.DEPLOYMENT_ENVIRONMENT_NAME, "new")
         .containsEntry("other", "other");
   }
 }

custom/src/test/java/co/elastic/otel/SpanStackTraceTest.java

@@ -30,7 +30,7 @@
 import io.opentelemetry.api.trace.Tracer;
 import io.opentelemetry.contrib.stacktrace.StackTraceSpanProcessor;
 import io.opentelemetry.sdk.trace.data.SpanData;
-import io.opentelemetry.semconv.incubating.CodeIncubatingAttributes;
+import io.opentelemetry.semconv.CodeAttributes;
 import java.util.List;
 import org.assertj.core.api.AbstractCharSequenceAssert;
 import org.junit.jupiter.api.AfterEach;
@@ -126,11 +126,9 @@ private void checkSpanStackTrace(boolean stackTraceExpected) {
     if (stackTraceExpected) {
       assertThat(span)
           .hasAttribute(
-              satisfies(
-                  CodeIncubatingAttributes.CODE_STACKTRACE,
-                  AbstractCharSequenceAssert::isNotBlank));
+              satisfies(CodeAttributes.CODE_STACKTRACE, AbstractCharSequenceAssert::isNotBlank));
     } else {
-      assertThat(span.getAttributes().get(CodeIncubatingAttributes.CODE_STACKTRACE)).isNull();
+      assertThat(span.getAttributes().get(CodeAttributes.CODE_STACKTRACE)).isNull();
     }
   }