Fix snippet test

Author: Hectorhammett

Spanner/src/Middleware/BuiltInMetricsAttemptMiddleware.php

@@ -120,7 +120,7 @@ public function __invoke(Call $call, array $options)
         $originalCallback = $options['metadataCallback'] ?? null;
 
         // This gets the metadata on an ok status meaning we can get the GFE latency header for unary calls
-        $options['metadataCallback'] = function($metadata) use ($originalCallback, $call, $options) {
+        $options['metadataCallback'] = function ($metadata) use ($originalCallback, $call, $options) {
             $this->recordGfeLatency($metadata, $call, $options);
             if ($originalCallback) {
                 $originalCallback($metadata);
@@ -134,7 +134,7 @@ public function __invoke(Call $call, array $options)
             );
         } catch (Exception $e) {
             // In case that the call is not a unary call and it is a streaming call error.
-            $this->recordAttempt($startTime,$e->getCode(), $call->getMethod(), $options);
+            $this->recordAttempt($startTime, $e->getCode(), $call->getMethod(), $options);
             $this->recordGfeError($e, $call, $options);
             throw $e;
         }

Spanner/src/OpenTelemetry/BuiltInMetricsExporter.php

@@ -193,8 +193,7 @@ private function createTimeSeries(
         NumberDataPoint|HistogramDataPoint $otelPoint,
         ?string $unit,
         DataInterface $otelData
-    ): TimeSeries
-    {
+    ): TimeSeries {
         $ts = new TimeSeries();
         $unit = $unit ?? '1';
 

Spanner/src/SpannerClient.php

@@ -1056,7 +1056,7 @@ private function configureBuiltinMetrics(bool $disabled): void
         $this->meter = $this->meterProvider->getMeter('google-cloud-spanner');
         ShutdownHandler::register([$this->meterProvider, 'shutdown']);
 
-        $attemptMetricsMiddleware = function (MiddlewareInterface $handler) use ($metricsClientId){
+        $attemptMetricsMiddleware = function (MiddlewareInterface $handler) use ($metricsClientId) {
             return new BuiltInMetricsAttemptMiddleware(
                 $handler,
                 $this->meter,
@@ -1066,7 +1066,7 @@ private function configureBuiltinMetrics(bool $disabled): void
             );
         };
 
-        $operationMetricsMiddleware = function (MiddlewareInterface $handler) use ($metricsClientId){
+        $operationMetricsMiddleware = function (MiddlewareInterface $handler) use ($metricsClientId) {
             return new BuiltInMetricsOperationMiddleware(
                 $handler,
                 $this->meter,

Spanner/tests/Snippet/CommitTimestampTest.php

@@ -57,10 +57,13 @@ public function testClass()
     {
         $id = 'abc';
 
+        // One add for the SpannerMiddleware and one for the Metrics middleware
         $this->spannerClient->addMiddleware(Argument::type('callable'))
-            ->shouldBeCalledOnce();
+            ->shouldBeCalled(2);
+
+        // One prepend for the Spanner Header Id and one for the Metrics middleware
         $this->spannerClient->prependMiddleware(Argument::type('callable'))
-            ->shouldBeCalledOnce();
+            ->shouldBeCalled(2);
 
         // ensure cache hit
         $cacheItem = $this->prophesize(CacheItemInterface::class);