Move log total attribute count fix to separate PR

Author: trask

testing-common/src/main/java/io/opentelemetry/instrumentation/testing/internal/TelemetryConverter.java

@@ -313,9 +313,7 @@ private static LogRecordData createLogData(
                     TraceState.getDefault())) // logs proto doesn't have trace state
             .setSeverity(fromProto(logRecord.getSeverityNumber()))
             .setSeverityText(logRecord.getSeverityText())
-            .setAttributes(fromProto(logRecord.getAttributesList()))
-            .setTotalAttributeCount(
-                logRecord.getAttributesCount() + logRecord.getDroppedAttributesCount());
+            .setAttributes(fromProto(logRecord.getAttributesList()));
     if (canUseValue) {
       builder.setBodyValue(getBodyValue(logRecord.getBody()));
     } else {
@@ -342,9 +340,7 @@ private static LogRecordData createExtendedLogData(
             .setSeverity(fromProto(logRecord.getSeverityNumber()))
             .setSeverityText(logRecord.getSeverityText())
             .setEventName(logRecord.getEventName())
-            .setBodyValue(getBodyValue(logRecord.getBody()))
-            .setTotalAttributeCount(
-                logRecord.getAttributesCount() + logRecord.getDroppedAttributesCount());
+            .setBodyValue(getBodyValue(logRecord.getBody()));
     if (hasExtendedAttributes) {
       builder.setExtendedAttributes(fromProtoExtended(logRecord.getAttributesList()));
     } else {

testing/agent-for-testing/src/test/java/io/opentelemetry/javaagent/testing/AgentForTestingTest.java

@@ -5,7 +5,6 @@
 
 package io.opentelemetry.javaagent.testing;
 
-import static io.opentelemetry.api.common.AttributeKey.stringKey;
 import static org.assertj.core.api.Assertions.assertThat;
 
 import io.opentelemetry.api.GlobalOpenTelemetry;
@@ -52,13 +51,10 @@ void exportAndRetrieveMetrics() {
   @Test
   void exportAndRetrieveLogRecords() {
     Logger logger = GlobalOpenTelemetry.get().getLogsBridge().loggerBuilder("test").build();
-    logger.logRecordBuilder().setBody("testBody").setAttribute("testKey", "testValue").emit();
+    logger.logRecordBuilder().setBody("testBody").emit();
 
     List<LogRecordData> logRecords = AgentTestingExporterAccess.getExportedLogRecords();
-    assertThat(logRecords).hasSize(1);
-    LogRecordData logRecord = logRecords.get(0);
-    assertThat(logRecord.getBodyValue().getValue()).isEqualTo("testBody");
-    assertThat(logRecord.getAttributes().get(stringKey("testKey"))).isEqualTo("testValue");
-    assertThat(logRecord.getTotalAttributeCount()).isEqualTo(1);
+    assertThat(logRecords.size()).isEqualTo(1);
+    assertThat(logRecords.get(0).getBodyValue().getValue()).isEqualTo("testBody");
   }
 }