diff --git a/spanner/metrics.go b/spanner/metrics.go
index 30c369bf0838..b6a3d3d01063 100644
--- a/spanner/metrics.go
+++ b/spanner/metrics.go
@@ -264,6 +264,9 @@ func (tf *builtinMetricsTracerFactory) createInstruments(meter metric.Meter) err
 		metric.WithDescription("The count of database operations."),
 		metric.WithUnit(metricUnitCount),
 	)
+	if err != nil {
+		return err
+	}
 
 	// Create attempt_count
 	tf.attemptCount, err = meter.Int64Counter(
diff --git a/spanner/transaction_test.go b/spanner/transaction_test.go
index 00b5509ab685..983b9902f0b9 100644
--- a/spanner/transaction_test.go
+++ b/spanner/transaction_test.go
@@ -575,6 +575,9 @@ func TestReadWriteStmtBasedTransactionWithOptions(t *testing.T) {
 			break
 		} else if err == nil {
 			resp, err = tx.CommitWithReturnResp(ctx)
+			if err != nil {
+				t.Fatalf("failed to CommitWithReturnResp: %v", err)
+			}
 			break
 		}
 		// Set a default sleep time if the server delay is absent.