fix(spanner): fix tests

Author: harshachinta

google-cloud-spanner/src/main/java/com/google/cloud/spanner/DatabaseClientImpl.java

@@ -282,8 +282,6 @@ public TransactionRunner readWriteTransaction(TransactionOption... options) {
       span.setStatus(e);
       span.end();
       throw e;
-    } finally {
-      span.end();
     }
   }
 
@@ -310,8 +308,6 @@ public AsyncRunner runAsync(TransactionOption... options) {
       span.setStatus(e);
       span.end();
       throw e;
-    } finally {
-      span.end();
     }
   }
 
@@ -324,8 +320,6 @@ public AsyncTransactionManager transactionManagerAsync(TransactionOption... opti
       span.setStatus(e);
       span.end();
       throw e;
-    } finally {
-      span.end();
     }
   }
 

google-cloud-spanner/src/test/java/com/google/cloud/spanner/DatabaseClientImplTest.java

@@ -4022,10 +4022,7 @@ public void testRunAsync_usesOptions() {
     when(pool.getSession()).thenReturn(session);
     TransactionOption option = mock(TransactionOption.class);
 
-    TraceWrapper traceWrapper =
-        new TraceWrapper(Tracing.getTracer(), OpenTelemetry.noop().getTracer(""), false);
-
-    DatabaseClientImpl client = new DatabaseClientImpl(pool, traceWrapper);
+    DatabaseClientImpl client = new DatabaseClientImpl(pool, mock(TraceWrapper.class));
     client.runAsync(option);
 
     verify(session).runAsync(option);
@@ -4038,10 +4035,7 @@ public void testTransactionManagerAsync_usesOptions() {
     when(pool.getSession()).thenReturn(session);
     TransactionOption option = mock(TransactionOption.class);
 
-    TraceWrapper traceWrapper =
-        new TraceWrapper(Tracing.getTracer(), OpenTelemetry.noop().getTracer(""), false);
-
-    DatabaseClientImpl client = new DatabaseClientImpl(pool, traceWrapper);
+    DatabaseClientImpl client = new DatabaseClientImpl(pool, mock(TraceWrapper.class));
     try (AsyncTransactionManager ignore = client.transactionManagerAsync(option)) {
       verify(session).transactionManagerAsync(option);
     }