View GitHub Repository BackwardCompatibilityCheck

Schedule Inspection
Subscribe
Completed
Pull Request — master (#249)
by
unknown
05:18 queued 02:01
created
Status
Category
Toggle Patches
test/unit/DetectChanges/BCBreak/ClassBased/ClassBecameInterfaceTest.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch 
@@ -37,7 +37,7 @@  discard block
 
 block discarded – undo
37 37
38 38 
         self::assertSame(
39 39 
             $expectedMessages,
40 
-            array_map(static function (Change $change) : string {
40 
+            array_map(static function(Change $change) : string {
41 41 
                 return $change->__toString();
42 42 
             }, iterator_to_array($changes))
43 43 
         );
 
@@ -68,7 +68,7 @@  discard block
 
 block discarded – undo
68 68 
             ,
69 69 
             $locator
70 70 
         ));
71 
-        $toReflector   = new ClassReflector(new StringSourceLocator(
71 
+        $toReflector = new ClassReflector(new StringSourceLocator(
72 72 
             <<<'PHP'
73 73 
 <?php
74 74 
 
@@ -102,7 +102,7 @@  discard block
 
 block discarded – undo
102 102 
             array_keys($classes),
103 103 
             array_map(
104 104 
                 /** @psalm-param list<string> $errors https://github.com/vimeo/psalm/issues/2772 */
105 
-                static function (string $className, array $errors) use ($fromReflector, $toReflector) : array {
105 
+                static function(string $className, array $errors) use ($fromReflector, $toReflector) : array {
106 106 
                     return [
107 107 
                         $fromReflector->reflect($className),
108 108 
                         $toReflector->reflect($className),
Please login to merge, or discard this patch.
test/unit/DetectChanges/BCBreak/ClassBased/AncestorRemovedTest.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch 
@@ -37,7 +37,7 @@  discard block
 
 block discarded – undo
37 37
38 38 
         self::assertSame(
39 39 
             $expectedMessages,
40 
-            array_map(static function (Change $change) : string {
40 
+            array_map(static function(Change $change) : string {
41 41 
                 return $change->__toString();
42 42 
             }, iterator_to_array($changes))
43 43 
         );
 
@@ -68,7 +68,7 @@  discard block
 
 block discarded – undo
68 68 
             ,
69 69 
             $locator
70 70 
         ));
71 
-        $toReflector   = new ClassReflector(new StringSourceLocator(
71 
+        $toReflector = new ClassReflector(new StringSourceLocator(
72 72 
             <<<'PHP'
73 73 
 <?php
74 74 
 
@@ -102,7 +102,7 @@  discard block
 
 block discarded – undo
102 102 
             array_keys($classes),
103 103 
             array_map(
104 104 
                 /** @psalm-param list<string> $errors https://github.com/vimeo/psalm/issues/2772 */
105 
-                static function (string $className, array $errors) use ($fromReflector, $toReflector) : array {
105 
+                static function(string $className, array $errors) use ($fromReflector, $toReflector) : array {
106 106 
                     return [
107 107 
                         $fromReflector->reflect($className),
108 108 
                         $toReflector->reflect($className),
Please login to merge, or discard this patch.
test/unit/DetectChanges/BCBreak/TraitBased/TraitBecameInterfaceTest.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch 
@@ -37,7 +37,7 @@  discard block
 
 block discarded – undo
37 37
38 38 
         self::assertSame(
39 39 
             $expectedMessages,
40 
-            array_map(static function (Change $change) : string {
40 
+            array_map(static function(Change $change) : string {
41 41 
                 return $change->__toString();
42 42 
             }, iterator_to_array($changes))
43 43 
         );
 
@@ -68,7 +68,7 @@  discard block
 
 block discarded – undo
68 68 
             ,
69 69 
             $locator
70 70 
         ));
71 
-        $toReflector   = new ClassReflector(new StringSourceLocator(
71 
+        $toReflector = new ClassReflector(new StringSourceLocator(
72 72 
             <<<'PHP'
73 73 
 <?php
74 74 
 
@@ -102,7 +102,7 @@  discard block
 
 block discarded – undo
102 102 
             array_keys($classes),
103 103 
             array_map(
104 104 
                 /** @psalm-param list<string> $errors https://github.com/vimeo/psalm/issues/2772 */
105 
-                static function (string $className, array $errors) use ($fromReflector, $toReflector) : array {
105 
+                static function(string $className, array $errors) use ($fromReflector, $toReflector) : array {
106 106 
                     return [
107 107 
                         $fromReflector->reflect($className),
108 108 
                         $toReflector->reflect($className),
Please login to merge, or discard this patch.
test/unit/DetectChanges/BCBreak/TraitBased/TraitBecameClassTest.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch 
@@ -37,7 +37,7 @@  discard block
 
 block discarded – undo
37 37
38 38 
         self::assertSame(
39 39 
             $expectedMessages,
40 
-            array_map(static function (Change $change) : string {
40 
+            array_map(static function(Change $change) : string {
41 41 
                 return $change->__toString();
42 42 
             }, iterator_to_array($changes))
43 43 
         );
 
@@ -68,7 +68,7 @@  discard block
 
 block discarded – undo
68 68 
             ,
69 69 
             $locator
70 70 
         ));
71 
-        $toReflector   = new ClassReflector(new StringSourceLocator(
71 
+        $toReflector = new ClassReflector(new StringSourceLocator(
72 72 
             <<<'PHP'
73 73 
 <?php
74 74 
 
@@ -102,7 +102,7 @@  discard block
 
 block discarded – undo
102 102 
             array_keys($classes),
103 103 
             array_map(
104 104 
                 /** @psalm-param list<string> $errors https://github.com/vimeo/psalm/issues/2772 */
105 
-                static function (string $className, array $errors) use ($fromReflector, $toReflector) : array {
105 
+                static function(string $className, array $errors) use ($fromReflector, $toReflector) : array {
106 106 
                     return [
107 107 
                         $fromReflector->reflect($className),
108 108 
                         $toReflector->reflect($className),
Please login to merge, or discard this patch.
test/unit/DetectChanges/BCBreak/MethodBased/MethodScopeChangedTest.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch 
@@ -37,7 +37,7 @@  discard block
 
 block discarded – undo
37 37
38 38 
         self::assertSame(
39 39 
             $expectedMessages,
40 
-            array_map(static function (Change $change) : string {
40 
+            array_map(static function(Change $change) : string {
41 41 
                 return $change->__toString();
42 42 
             }, iterator_to_array($changes))
43 43 
         );
 
@@ -128,7 +128,7 @@  discard block
 
 block discarded – undo
128 128 
             array_keys($properties),
129 129 
             array_map(
130 130 
                 /** @psalm-param list<string> $errorMessages https://github.com/vimeo/psalm/issues/2772 */
131 
-                static function (string $methodName, array $errorMessages) use ($fromClass, $toClass) : array {
131 
+                static function(string $methodName, array $errorMessages) use ($fromClass, $toClass) : array {
132 132 
                     return [
133 133 
                         $fromClass->getMethod($methodName),
134 134 
                         $toClass->getMethod($methodName),
Please login to merge, or discard this patch.
unit/DetectChanges/BCBreak/MethodBased/MethodConcretenessChangedTest.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch 
@@ -37,7 +37,7 @@  discard block
 
 block discarded – undo
37 37
38 38 
         self::assertSame(
39 39 
             $expectedMessages,
40 
-            array_map(static function (Change $change) : string {
40 
+            array_map(static function(Change $change) : string {
41 41 
                 return $change->__toString();
42 42 
             }, iterator_to_array($changes))
43 43 
         );
 
@@ -128,7 +128,7 @@  discard block
 
 block discarded – undo
128 128 
             array_keys($properties),
129 129 
             array_map(
130 130 
                 /** @psalm-param list<string> $errorMessages https://github.com/vimeo/psalm/issues/2772 */
131 
-                static function (string $methodName, array $errorMessages) use ($fromClass, $toClass) : array {
131 
+                static function(string $methodName, array $errorMessages) use ($fromClass, $toClass) : array {
132 132 
                     return [
133 133 
                         $fromClass->getMethod($methodName),
134 134 
                         $toClass->getMethod($methodName),
Please login to merge, or discard this patch.
test/unit/DetectChanges/BCBreak/ClassBased/ExcludeInternalClassTest.php 1 patch
Indentation   +3 added lines, -3 removed lines patch added patch discarded remove patch 
@@ -33,9 +33,9 @@
 
 block discarded – undo
33 33
34 34 
         $check = $this->createMock(ClassBased::class);
35 35 
         $check->expects(self::once())
36 
-              ->method('__invoke')
37 
-              ->with($fromReflection, $toReflection)
38 
-              ->willReturn(Changes::fromList(Change::removed('foo', true)));
36 
+                ->method('__invoke')
37 
+                ->with($fromReflection, $toReflection)
38 
+                ->willReturn(Changes::fromList(Change::removed('foo', true)));
39 39
40 40 
         self::assertEquals(
41 41 
             Changes::fromList(Change::removed('foo', true)),
Please login to merge, or discard this patch.