fix: return null when error type is null

Author: diegomarquezp

sdk-platform-java/gax-java/gax/src/main/java/com/google/api/gax/tracing/ErrorTypeUtil.java

@@ -119,12 +119,12 @@ enum ErrorType {
   // Requirement source: go/clo:product-requirements-v1
   public static String extractErrorType(@Nullable Throwable error) {
     if (error == null) {
-      // No information about the error; we default to INTERNAL.
-      return ErrorType.INTERNAL.toString();
+      // No information about the error; This attribute should not be recorded.
+      return null;
     }
 
     // 1. Unwrap standard wrapper exceptions if present
-    Throwable realError = getRealCause(error);
+    Throwable realError = unwrapError(error);
 
     // 2. Attempt to extract specific error type from the main exception
     String specificError = extractSpecificErrorType(realError);
@@ -143,7 +143,7 @@ public static String extractErrorType(@Nullable Throwable error) {
   }
 
   /** Unwraps standard execution wrappers to find the real cause. */
-  private static Throwable getRealCause(Throwable t) {
+  private static Throwable unwrapError(Throwable t) {
     if (t.getCause() == null) {
       return t;
     }

sdk-platform-java/gax-java/gax/src/test/java/com/google/api/gax/tracing/ErrorTypeUtilTest.java

@@ -54,8 +54,7 @@ class ErrorTypeUtilTest {
 
   @Test
   void testExtractErrorType_null() {
-    assertThat(ErrorTypeUtil.extractErrorType(null))
-        .isEqualTo(ErrorTypeUtil.ErrorType.INTERNAL.toString());
+    assertThat(ErrorTypeUtil.extractErrorType(null)).isNull();
   }
 
   @Test

sdk-platform-java/gax-java/gax/src/test/java/com/google/api/gax/tracing/SpanTracerTest.java

@@ -439,10 +439,8 @@ void testAttemptFailed_internalFallback_nullError() {
     // For an anonymous inner class Throwable, getSimpleName() is empty string,
     // which triggers the
     // fallback
-    verify(span)
-        .setAttribute(
-            ObservabilityAttributes.ERROR_TYPE_ATTRIBUTE,
-            ErrorTypeUtil.ErrorType.INTERNAL.toString());
+    verify(span, never())
+        .setAttribute(eq(ObservabilityAttributes.ERROR_TYPE_ATTRIBUTE), anyString());
     verify(span).end();
   }