smartboxgroup / core-bundle

Serializer/Handler/DateHandler.php

Spacing +1 added lines, -1 removed lines

@@ -31,7 +31,7 @@
 
         $types = ['DateTime', 'DateInterval'];
 
-        $newFormats = ['array'];   // Add here any new format you wish to support
+        $newFormats = ['array']; // Add here any new format you wish to support
 
         foreach ($newFormats as $format) {
             $methods[] = [

Tests/Serializer/ArraySerializationVisitorFunctionalTest.php

Spacing +2 added lines, -2 removed lines

@@ -45,7 +45,7 @@ 
         $arrayData = $this->serializer->serialize($data, 'array');
         $this->assertEquals(
             [
-                'internalType' => TestEntity::class,            // We are using the IdenticalPropertyNamingStrategy
+                'internalType' => TestEntity::class, // We are using the IdenticalPropertyNamingStrategy
                 'title' => 'some title',
                 'description' => 'some description',
                 'note' => 'some note',
@@ -70,7 +70,7 @@ 
         $arrayData = $this->serializer->serialize($data, 'array', $context);
         $this->assertEquals(
             [
-                'internalType' => TestEntity::class,        // We are using the IdenticalPropertyNamingStrategy
+                'internalType' => TestEntity::class, // We are using the IdenticalPropertyNamingStrategy
                 'title' => 'some title',
                 'note' => 'some note',
             ],

Tests/Symfony/Validator/Abstract2Dot5ApiTest.php

Spacing +23 added lines, -23 removed lines

@@ -87,15 +87,15 @@ 
     {
         $entity = new Entity();
 
-        $callback1 = function ($value, ExecutionContextInterface $context) {
+        $callback1 = function($value, ExecutionContextInterface $context) {
             $context->addViolation('Message 1');
         };
-        $callback2 = function ($value, ExecutionContextInterface $context) {
+        $callback2 = function($value, ExecutionContextInterface $context) {
             $context->addViolation('Message 2');
         };
 
         $this->metadata->addConstraint(new Callback([
-            'callback' => function () {},
+            'callback' => function() {},
             'groups' => 'Group 1',
         ]));
         $this->metadata->addConstraint(new Callback([
@@ -120,10 +120,10 @@ 
         $entity = new Entity();
         $entity->reference = new Reference();
 
-        $callback1 = function ($value, ExecutionContextInterface $context) {
+        $callback1 = function($value, ExecutionContextInterface $context) {
             $context->addViolation('Reference violation 1');
         };
-        $callback2 = function ($value, ExecutionContextInterface $context) {
+        $callback2 = function($value, ExecutionContextInterface $context) {
             $context->addViolation('Reference violation 2');
         };
 
@@ -151,7 +151,7 @@ 
         $entity = new Entity();
         $entity->reference = new Reference();
 
-        $callback1 = function ($value, ExecutionContextInterface $context) use ($test, $entity) {
+        $callback1 = function($value, ExecutionContextInterface $context) use ($test, $entity) {
             $violations = $context
                 ->getValidator()
                 // Since the validator is not context aware, the group must
@@ -175,7 +175,7 @@ 
             $context->addViolation('Separate violation');
         };
 
-        $callback2 = function ($value, ExecutionContextInterface $context) use ($test, $entity) {
+        $callback2 = function($value, ExecutionContextInterface $context) use ($test, $entity) {
             $test->assertSame($test::REFERENCE_CLASS, $context->getClassName());
             $test->assertNull($context->getPropertyName());
             $test->assertSame('', $context->getPropertyPath());
@@ -210,7 +210,7 @@ 
         $entity = new Entity();
         $entity->reference = new Reference();
 
-        $callback1 = function ($value, ExecutionContextInterface $context) use ($test) {
+        $callback1 = function($value, ExecutionContextInterface $context) use ($test) {
             $previousValue = $context->getValue();
             $previousObject = $context->getObject();
             $previousMetadata = $context->getMetadata();
@@ -232,7 +232,7 @@ 
             $test->assertSame($previousGroup, $context->getGroup());
         };
 
-        $callback2 = function ($value, ExecutionContextInterface $context) use ($test, $entity) {
+        $callback2 = function($value, ExecutionContextInterface $context) use ($test, $entity) {
             $test->assertSame($test::REFERENCE_CLASS, $context->getClassName());
             $test->assertNull($context->getPropertyName());
             $test->assertSame('subpath', $context->getPropertyPath());
@@ -274,7 +274,7 @@ 
         $entity = new Entity();
         $entity->reference = new Reference();
 
-        $callback1 = function ($value, ExecutionContextInterface $context) use ($test) {
+        $callback1 = function($value, ExecutionContextInterface $context) use ($test) {
             $previousValue = $context->getValue();
             $previousObject = $context->getObject();
             $previousMetadata = $context->getMetadata();
@@ -296,7 +296,7 @@ 
             $test->assertSame($previousGroup, $context->getGroup());
         };
 
-        $callback2 = function ($value, ExecutionContextInterface $context) use ($test, $entity) {
+        $callback2 = function($value, ExecutionContextInterface $context) use ($test, $entity) {
             $test->assertSame($test::REFERENCE_CLASS, $context->getClassName());
             $test->assertNull($context->getPropertyName());
             $test->assertSame('subpath[key]', $context->getPropertyPath());
@@ -338,7 +338,7 @@ 
         $entity = new Entity();
         $traversable = new \ArrayIterator(['key' => $entity]);
 
-        $callback = function ($value, ExecutionContextInterface $context) use ($test, $entity, $traversable) {
+        $callback = function($value, ExecutionContextInterface $context) use ($test, $entity, $traversable) {
             $test->assertSame($test::ENTITY_CLASS, $context->getClassName());
             $test->assertNull($context->getPropertyName());
             $test->assertSame('[key]', $context->getPropertyPath());
@@ -376,7 +376,7 @@ 
         $entity = new Entity();
         $traversable = new \ArrayIterator(['key' => $entity]);
 
-        $callback = function ($value, ExecutionContextInterface $context) {
+        $callback = function($value, ExecutionContextInterface $context) {
             $context->addViolation('Message');
         };
 
@@ -401,7 +401,7 @@ 
         $entity = new Entity();
         $traversable = new \ArrayIterator(['key' => $entity]);
 
-        $callback = function ($value, ExecutionContextInterface $context) use ($test) {
+        $callback = function($value, ExecutionContextInterface $context) use ($test) {
             $test->fail('Should not be called');
         };
 
@@ -438,7 +438,7 @@ 
         $entity = new Entity();
         $entity->reference = new \ArrayIterator(['key' => new Reference()]);
 
-        $callback = function ($value, ExecutionContextInterface $context) use ($test) {
+        $callback = function($value, ExecutionContextInterface $context) use ($test) {
             $test->fail('Should not be called');
         };
 
@@ -464,7 +464,7 @@ 
         $entity = new Entity();
         $entity->reference = new \ArrayIterator(['key' => new Reference()]);
 
-        $callback = function ($value, ExecutionContextInterface $context) use ($test) {
+        $callback = function($value, ExecutionContextInterface $context) use ($test) {
             $test->fail('Should not be called');
         };
 
@@ -492,7 +492,7 @@ 
         $entity = new Entity();
         $entity->reference = new \ArrayIterator(['key' => new Reference()]);
 
-        $callback = function ($value, ExecutionContextInterface $context) use ($test) {
+        $callback = function($value, ExecutionContextInterface $context) use ($test) {
             $test->fail('Should not be called');
         };
 
@@ -518,7 +518,7 @@ 
     {
         $entity = new Entity();
 
-        $callback = function ($value, ExecutionContextInterface $context) {
+        $callback = function($value, ExecutionContextInterface $context) {
             $context->buildViolation('Message %param%')
                 ->setParameter('%param%', 'value')
                 ->setInvalidValue('Invalid value')
@@ -604,7 +604,7 @@ 
     {
         $entity = new Entity();
 
-        $callback = function ($value, ExecutionContextInterface $context) {
+        $callback = function($value, ExecutionContextInterface $context) {
             $context->addViolation('Message');
         };
 
@@ -623,7 +623,7 @@ 
     {
         $entity = new Entity();
 
-        $callback = function ($value, ExecutionContextInterface $context) {
+        $callback = function($value, ExecutionContextInterface $context) {
             $context->addViolation('Message');
         };
 
@@ -653,7 +653,7 @@ 
         $entity = new Entity();
         $entity->firstName = 'Bernhard';
 
-        $callback = function ($value, ExecutionContextInterface $context) use ($test, $entity, &$called) {
+        $callback = function($value, ExecutionContextInterface $context) use ($test, $entity, &$called) {
             $called = true;
             $test->assertSame($entity, $context->getObject());
         };
@@ -679,7 +679,7 @@ 
         $initializer1->expects($this->once())
             ->method('initialize')
             ->with($entity)
-            ->will($this->returnCallback(function ($object) {
+            ->will($this->returnCallback(function($object) {
                 $object->initialized = true;
             }));
 
@@ -695,7 +695,7 @@ 
         // prepare constraint which
         // * checks that "initialized" is set to true
         // * validates the object again
-        $callback = function ($object, ExecutionContextInterface $context) use ($test) {
+        $callback = function($object, ExecutionContextInterface $context) use ($test) {
             $test->assertTrue($object->initialized);
 
             // validate again in same group

Tests/Symfony/Validator/AbstractValidatorTest.php

Spacing +44 added lines, -44 removed lines

@@ -75,7 +75,7 @@ 
     {
         $test = $this;
 
-        $callback = function ($value, ExecutionContextInterface $context) use ($test) {
+        $callback = function($value, ExecutionContextInterface $context) use ($test) {
             $test->assertNull($context->getClassName());
             $test->assertNull($context->getPropertyName());
             $test->assertSame('', $context->getPropertyPath());
@@ -111,7 +111,7 @@ 
         $test = $this;
         $entity = new Entity();
 
-        $callback = function ($value, ExecutionContextInterface $context) use ($test, $entity) {
+        $callback = function($value, ExecutionContextInterface $context) use ($test, $entity) {
             $test->assertSame($test::ENTITY_CLASS, $context->getClassName());
             $test->assertNull($context->getPropertyName());
             $test->assertSame('', $context->getPropertyPath());
@@ -149,7 +149,7 @@ 
         $entity = new Entity();
         $entity->firstName = 'Bernhard';
 
-        $callback = function ($value, ExecutionContextInterface $context) use ($test, $entity) {
+        $callback = function($value, ExecutionContextInterface $context) use ($test, $entity) {
             $propertyMetadatas = $test->metadata->getPropertyMetadata('firstName');
 
             $test->assertSame($test::ENTITY_CLASS, $context->getClassName());
@@ -189,7 +189,7 @@ 
         $entity = new Entity();
         $entity->setLastName('Schussek');
 
-        $callback = function ($value, ExecutionContextInterface $context) use ($test, $entity) {
+        $callback = function($value, ExecutionContextInterface $context) use ($test, $entity) {
             $propertyMetadatas = $test->metadata->getPropertyMetadata('lastName');
 
             $test->assertSame($test::ENTITY_CLASS, $context->getClassName());
@@ -229,7 +229,7 @@ 
         $entity = new Entity();
         $array = ['key' => $entity];
 
-        $callback = function ($value, ExecutionContextInterface $context) use ($test, $entity, $array) {
+        $callback = function($value, ExecutionContextInterface $context) use ($test, $entity, $array) {
             $test->assertSame($test::ENTITY_CLASS, $context->getClassName());
             $test->assertNull($context->getPropertyName());
             $test->assertSame('[key]', $context->getPropertyPath());
@@ -267,7 +267,7 @@ 
         $entity = new Entity();
         $array = [2 => ['key' => $entity]];
 
-        $callback = function ($value, ExecutionContextInterface $context) use ($test, $entity, $array) {
+        $callback = function($value, ExecutionContextInterface $context) use ($test, $entity, $array) {
             $test->assertSame($test::ENTITY_CLASS, $context->getClassName());
             $test->assertNull($context->getPropertyName());
             $test->assertSame('[2][key]', $context->getPropertyPath());
@@ -305,7 +305,7 @@ 
         $entity = new Entity();
         $traversable = new \ArrayIterator(['key' => $entity]);
 
-        $callback = function ($value, ExecutionContextInterface $context) use ($test, $entity, $traversable) {
+        $callback = function($value, ExecutionContextInterface $context) use ($test, $entity, $traversable) {
             $test->assertSame($test::ENTITY_CLASS, $context->getClassName());
             $test->assertNull($context->getPropertyName());
             $test->assertSame('[key]', $context->getPropertyPath());
@@ -345,7 +345,7 @@ 
             2 => new \ArrayIterator(['key' => $entity]),
         ]);
 
-        $callback = function ($value, ExecutionContextInterface $context) use ($test, $entity, $traversable) {
+        $callback = function($value, ExecutionContextInterface $context) use ($test, $entity, $traversable) {
             $test->assertSame($test::ENTITY_CLASS, $context->getClassName());
             $test->assertNull($context->getPropertyName());
             $test->assertSame('[2][key]', $context->getPropertyPath());
@@ -383,7 +383,7 @@ 
         $entity = new Entity();
         $entity->reference = new Reference();
 
-        $callback = function ($value, ExecutionContextInterface $context) use ($test, $entity) {
+        $callback = function($value, ExecutionContextInterface $context) use ($test, $entity) {
             $test->assertSame($test::REFERENCE_CLASS, $context->getClassName());
             $test->assertNull($context->getPropertyName());
             $test->assertSame('reference', $context->getPropertyPath());
@@ -423,7 +423,7 @@ 
         $entity->reference = new Reference();
         $entity->reference->value = 'Foobar';
 
-        $callback = function ($value, ExecutionContextInterface $context) use ($test, $entity) {
+        $callback = function($value, ExecutionContextInterface $context) use ($test, $entity) {
             $propertyMetadatas = $test->referenceMetadata->getPropertyMetadata('value');
 
             $test->assertSame($test::REFERENCE_CLASS, $context->getClassName());
@@ -465,7 +465,7 @@ 
         $entity->reference = new Reference();
         $entity->reference->setPrivateValue('Bamboo');
 
-        $callback = function ($value, ExecutionContextInterface $context) use ($test, $entity) {
+        $callback = function($value, ExecutionContextInterface $context) use ($test, $entity) {
             $propertyMetadatas = $test->referenceMetadata->getPropertyMetadata('privateValue');
 
             $test->assertSame($test::REFERENCE_CLASS, $context->getClassName());
@@ -532,7 +532,7 @@ 
         $entity = new Entity();
         $entity->reference = ['key' => new Reference()];
 
-        $callback = function ($value, ExecutionContextInterface $context) use ($test, $entity) {
+        $callback = function($value, ExecutionContextInterface $context) use ($test, $entity) {
             $test->assertSame($test::REFERENCE_CLASS, $context->getClassName());
             $test->assertNull($context->getPropertyName());
             $test->assertSame('reference[key]', $context->getPropertyPath());
@@ -572,7 +572,7 @@ 
         $entity = new Entity();
         $entity->reference = [2 => ['key' => new Reference()]];
 
-        $callback = function ($value, ExecutionContextInterface $context) use ($test, $entity) {
+        $callback = function($value, ExecutionContextInterface $context) use ($test, $entity) {
             $test->assertSame($test::REFERENCE_CLASS, $context->getClassName());
             $test->assertNull($context->getPropertyName());
             $test->assertSame('reference[2][key]', $context->getPropertyPath());
@@ -610,7 +610,7 @@ 
         $entity = new Entity();
         $entity->reference = ['key' => new Reference()];
 
-        $callback = function ($value, ExecutionContextInterface $context) {
+        $callback = function($value, ExecutionContextInterface $context) {
             $context->addViolation('Message %param%', ['%param%' => 'value']);
         };
 
@@ -630,7 +630,7 @@ 
         $entity = new Entity();
         $entity->reference = [2 => ['key' => new Reference()]];
 
-        $callback = function ($value, ExecutionContextInterface $context) {
+        $callback = function($value, ExecutionContextInterface $context) {
             $context->addViolation('Message %param%', ['%param%' => 'value']);
         };
 
@@ -677,7 +677,7 @@ 
         $entity = new Entity();
         $entity->reference = new \ArrayIterator(['key' => new Reference()]);
 
-        $callback = function ($value, ExecutionContextInterface $context) use ($test, $entity) {
+        $callback = function($value, ExecutionContextInterface $context) use ($test, $entity) {
             $test->assertSame($test::REFERENCE_CLASS, $context->getClassName());
             $test->assertNull($context->getPropertyName());
             $test->assertSame('reference[key]', $context->getPropertyPath());
@@ -715,7 +715,7 @@ 
         $entity = new Entity();
         $entity->reference = new \ArrayIterator(['key' => new Reference()]);
 
-        $callback = function ($value, ExecutionContextInterface $context) {
+        $callback = function($value, ExecutionContextInterface $context) {
             $context->addViolation('Message %param%', ['%param%' => 'value']);
         };
 
@@ -754,7 +754,7 @@ 
             2 => new \ArrayIterator(['key' => new Reference()]),
         ]);
 
-        $callback = function ($value, ExecutionContextInterface $context) use ($test, $entity) {
+        $callback = function($value, ExecutionContextInterface $context) use ($test, $entity) {
             $test->assertSame($test::REFERENCE_CLASS, $context->getClassName());
             $test->assertNull($context->getPropertyName());
             $test->assertSame('reference[2][key]', $context->getPropertyPath());
@@ -796,7 +796,7 @@ 
         $entity->firstName = 'Bernhard';
         $entity->setLastName('Schussek');
 
-        $callback1 = function ($value, ExecutionContextInterface $context) use ($test, $entity) {
+        $callback1 = function($value, ExecutionContextInterface $context) use ($test, $entity) {
             $propertyMetadatas = $test->metadata->getPropertyMetadata('firstName');
 
             $test->assertSame($test::ENTITY_CLASS, $context->getClassName());
@@ -811,7 +811,7 @@ 
             $context->addViolation('Message %param%', ['%param%' => 'value']);
         };
 
-        $callback2 = function ($value, ExecutionContextInterface $context) {
+        $callback2 = function($value, ExecutionContextInterface $context) {
             $context->addViolation('Other violation');
         };
 
@@ -871,7 +871,7 @@ 
         $entity = new Entity();
         $entity->setLastName('Schussek');
 
-        $callback1 = function ($value, ExecutionContextInterface $context) use ($test, $entity) {
+        $callback1 = function($value, ExecutionContextInterface $context) use ($test, $entity) {
             $propertyMetadatas = $test->metadata->getPropertyMetadata('firstName');
 
             $test->assertSame($test::ENTITY_CLASS, $context->getClassName());
@@ -886,7 +886,7 @@ 
             $context->addViolation('Message %param%', ['%param%' => 'value']);
         };
 
-        $callback2 = function ($value, ExecutionContextInterface $context) {
+        $callback2 = function($value, ExecutionContextInterface $context) {
             $context->addViolation('Other violation');
         };
 
@@ -922,7 +922,7 @@ 
     {
         $test = $this;
 
-        $callback1 = function ($value, ExecutionContextInterface $context) use ($test) {
+        $callback1 = function($value, ExecutionContextInterface $context) use ($test) {
             $propertyMetadatas = $test->metadata->getPropertyMetadata('firstName');
 
             $test->assertSame($test::ENTITY_CLASS, $context->getClassName());
@@ -937,7 +937,7 @@ 
             $context->addViolation('Message %param%', ['%param%' => 'value']);
         };
 
-        $callback2 = function ($value, ExecutionContextInterface $context) {
+        $callback2 = function($value, ExecutionContextInterface $context) {
             $context->addViolation('Other violation');
         };
 
@@ -1002,7 +1002,7 @@ 
         $entity->reference = new Reference();
         $entity->reference2 = $entity->reference;
 
-        $callback = function ($value, ExecutionContextInterface $context) {
+        $callback = function($value, ExecutionContextInterface $context) {
             $context->addViolation('Message');
         };
 
@@ -1022,7 +1022,7 @@ 
         $entity->reference = new Reference();
         $entity->reference2 = new Reference();
 
-        $callback = function ($value, ExecutionContextInterface $context) {
+        $callback = function($value, ExecutionContextInterface $context) {
             $context->addViolation('Message');
         };
 
@@ -1040,7 +1040,7 @@ 
     {
         $entity = new Entity();
 
-        $callback = function ($value, ExecutionContextInterface $context) {
+        $callback = function($value, ExecutionContextInterface $context) {
             $context->addViolation('Message');
         };
 
@@ -1063,7 +1063,7 @@ 
     {
         $entity = new Entity();
 
-        $callback = function ($value, ExecutionContextInterface $context) {
+        $callback = function($value, ExecutionContextInterface $context) {
             $context->addViolation('Message');
         };
 
@@ -1086,15 +1086,15 @@ 
     {
         $entity = new Entity();
 
-        $callback1 = function ($value, ExecutionContextInterface $context) {
+        $callback1 = function($value, ExecutionContextInterface $context) {
             $context->addViolation('Violation in Group 2');
         };
-        $callback2 = function ($value, ExecutionContextInterface $context) {
+        $callback2 = function($value, ExecutionContextInterface $context) {
             $context->addViolation('Violation in Group 3');
         };
 
         $this->metadata->addConstraint(new Callback([
-            'callback' => function () {},
+            'callback' => function() {},
             'groups' => 'Group 1',
         ]));
         $this->metadata->addConstraint(new Callback([
@@ -1120,15 +1120,15 @@ 
     {
         $entity = new Entity();
 
-        $callback1 = function ($value, ExecutionContextInterface $context) {
+        $callback1 = function($value, ExecutionContextInterface $context) {
             $context->addViolation('Violation in Group 2');
         };
-        $callback2 = function ($value, ExecutionContextInterface $context) {
+        $callback2 = function($value, ExecutionContextInterface $context) {
             $context->addViolation('Violation in Group 3');
         };
 
         $this->metadata->addConstraint(new Callback([
-            'callback' => function () {},
+            'callback' => function() {},
             'groups' => 'Group 1',
         ]));
         $this->metadata->addConstraint(new Callback([
@@ -1155,10 +1155,10 @@ 
         $entity = new Entity();
         $entity->reference = new Reference();
 
-        $callback1 = function ($value, ExecutionContextInterface $context) {
+        $callback1 = function($value, ExecutionContextInterface $context) {
             $context->addViolation('Violation in Default group');
         };
-        $callback2 = function ($value, ExecutionContextInterface $context) {
+        $callback2 = function($value, ExecutionContextInterface $context) {
             $context->addViolation('Violation in group sequence');
         };
 
@@ -1186,10 +1186,10 @@ 
     {
         $entity = new Entity();
 
-        $callback1 = function ($value, ExecutionContextInterface $context) {
+        $callback1 = function($value, ExecutionContextInterface $context) {
             $context->addViolation('Violation in other group');
         };
-        $callback2 = function ($value, ExecutionContextInterface $context) {
+        $callback2 = function($value, ExecutionContextInterface $context) {
             $context->addViolation('Violation in group sequence');
         };
 
@@ -1217,16 +1217,16 @@ 
         $sequence = new GroupSequence(['Group 1', 'Group 2', 'Group 3', 'Entity']);
         $entity = new GroupSequenceProviderEntity($sequence);
 
-        $callback1 = function ($value, ExecutionContextInterface $context) {
+        $callback1 = function($value, ExecutionContextInterface $context) {
             $context->addViolation('Violation in Group 2');
         };
-        $callback2 = function ($value, ExecutionContextInterface $context) {
+        $callback2 = function($value, ExecutionContextInterface $context) {
             $context->addViolation('Violation in Group 3');
         };
 
         $metadata = new ClassMetadata(get_class($entity));
         $metadata->addConstraint(new Callback([
-            'callback' => function () {},
+            'callback' => function() {},
             'groups' => 'Group 1',
         ]));
         $metadata->addConstraint(new Callback([
@@ -1253,16 +1253,16 @@ 
         $sequence = ['Group 1', 'Group 2', 'Group 3', 'Entity'];
         $entity = new GroupSequenceProviderEntity($sequence);
 
-        $callback1 = function ($value, ExecutionContextInterface $context) {
+        $callback1 = function($value, ExecutionContextInterface $context) {
             $context->addViolation('Violation in Group 2');
         };
-        $callback2 = function ($value, ExecutionContextInterface $context) {
+        $callback2 = function($value, ExecutionContextInterface $context) {
             $context->addViolation('Violation in Group 3');
         };
 
         $metadata = new ClassMetadata(get_class($entity));
         $metadata->addConstraint(new Callback([
-            'callback' => function () {},
+            'callback' => function() {},
             'groups' => 'Group 1',
         ]));
         $metadata->addConstraint(new Callback([

Command/SmokeTestRunCommand.php

Spacing +2 added lines, -2 removed lines

@@ -118,7 +118,7 @@ 
             $testNames = $this->getTestNames();
             $smokeTests = array_filter(
                 $smokeTests,
-                function ($key) use ($test) {
+                function($key) use ($test) {
                     return $key === $test;
                 },
                 ARRAY_FILTER_USE_KEY
@@ -151,7 +151,7 @@ 
             if (!empty($skipTests)) {
                 $smokeTests = array_filter(
                     $smokeTests,
-                    function ($smokeTestInfo) use ($skipTests, &$skipped) {
+                    function($smokeTestInfo) use ($skipTests, &$skipped) {
                         $key = $smokeTestInfo['id'];
                         $skipping = in_array($key, $skipTests);
                         if ($skipping) {