View GitHub Repository BackwardCompatibilityCheck

Schedule Inspection
Subscribe
Completed
Push — master ( 85853a...5df06c )
by James
20s queued 12s
created
Status
Category
Toggle Patches
test/unit/DetectChanges/BCBreak/TraitBased/ExcludeInternalTraitTest.php 1 patch
Indentation   +3 added lines, -3 removed lines patch added patch discarded remove patch 
@@ -32,9 +32,9 @@
 
 block discarded – undo
32 32
33 33 
         $check = $this->createMock(TraitBased::class);
34 34 
         $check->expects(self::once())
35 
-              ->method('__invoke')
36 
-              ->with($reflection, $reflection)
37 
-              ->willReturn(Changes::fromList(Change::removed('foo', true)));
35 
+                ->method('__invoke')
36 
+                ->with($reflection, $reflection)
37 
+                ->willReturn(Changes::fromList(Change::removed('foo', true)));
38 38
39 39 
         self::assertEquals(
40 40 
             Changes::fromList(Change::removed('foo', true)),
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 
@@ -35,9 +35,9 @@
 
 block discarded – undo
35 35 
         /** @var ClassBased&MockObject $check */
36 36 
         $check = $this->createMock(ClassBased::class);
37 37 
         $check->expects(self::once())
38 
-              ->method('__invoke')
39 
-              ->with($fromReflection, $toReflection)
40 
-              ->willReturn(Changes::fromList(Change::removed('foo', true)));
38 
+                ->method('__invoke')
39 
+                ->with($fromReflection, $toReflection)
40 
+                ->willReturn(Changes::fromList(Change::removed('foo', true)));
41 41
42 42 
         self::assertEquals(
43 43 
             Changes::fromList(Change::removed('foo', true)),
Please login to merge, or discard this patch.
test/unit/DetectChanges/BCBreak/ClassBased/ClassBecameInternalTest.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch 
@@ -31,7 +31,7 @@  discard block
 
 block discarded – undo
31 31
32 32 
         self::assertSame(
33 33 
             $expectedMessages,
34 
-            array_map(static function (Change $change) : string {
34 
+            array_map(static function(Change $change) : string {
35 35 
                 return $change->__toString();
36 36 
             }, iterator_to_array($changes))
37 37 
         );
 
@@ -53,7 +53,7 @@  discard block
 
 block discarded – undo
53 53 
             ,
54 54 
             $locator
55 55 
         ));
56 
-        $toReflector   = new ClassReflector(new StringSourceLocator(
56 
+        $toReflector = new ClassReflector(new StringSourceLocator(
57 57 
             <<<'PHP'
58 58 
 <?php
59 59
Please login to merge, or discard this patch.
unit/DetectChanges/BCBreak/InterfaceBased/ExcludeInternalInterfaceTest.php 1 patch
Indentation   +3 added lines, -3 removed lines patch added patch discarded remove patch 
@@ -32,9 +32,9 @@
 
 block discarded – undo
32 32
33 33 
         $check = $this->createMock(InterfaceBased::class);
34 34 
         $check->expects(self::once())
35 
-              ->method('__invoke')
36 
-              ->with($reflection, $reflection)
37 
-              ->willReturn(Changes::fromList(Change::removed('foo', true)));
35 
+                ->method('__invoke')
36 
+                ->with($reflection, $reflection)
37 
+                ->willReturn(Changes::fromList(Change::removed('foo', true)));
38 38
39 39 
         self::assertEquals(
40 40 
             Changes::fromList(Change::removed('foo', true)),
Please login to merge, or discard this patch.
unit/DetectChanges/BCBreak/FunctionBased/FunctionBecameInternalTest.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch 
@@ -35,7 +35,7 @@  discard block
 
 block discarded – undo
35 35
36 36 
         self::assertSame(
37 37 
             $expectedMessages,
38 
-            array_map(static function (Change $change) : string {
38 
+            array_map(static function(Change $change) : string {
39 39 
                 return $change->__toString();
40 40 
             }, iterator_to_array($changes))
41 41 
         );
 
@@ -96,7 +96,7 @@  discard block
 
 block discarded – undo
96 96 
         return array_combine(
97 97 
             array_keys($functions),
98 98 
             array_map(
99 
-                static function (string $function, array $errorMessages) use ($fromReflector, $toReflector) : array {
99 
+                static function(string $function, array $errorMessages) use ($fromReflector, $toReflector) : array {
100 100 
                     return [
101 101 
                         $fromReflector->reflect($function),
102 102 
                         $toReflector->reflect($function),
Please login to merge, or discard this patch.
unit/DetectChanges/BCBreak/FunctionBased/ExcludeInternalFunctionTest.php 2 patches
Indentation   +4 added lines, -4 removed lines patch added patch discarded remove patch 
@@ -33,9 +33,9 @@  discard block
 
 block discarded – undo
33 33
34 34 
         $check = $this->createMock(FunctionBased::class);
35 35 
         $check->expects(self::once())
36 
-              ->method('__invoke')
37 
-              ->with($function, $function)
38 
-              ->willReturn(Changes::fromList(Change::removed('foo', true)));
36 
+                ->method('__invoke')
37 
+                ->with($function, $function)
38 
+                ->willReturn(Changes::fromList(Change::removed('foo', true)));
39 39
40 40 
         self::assertEquals(
41 41 
             Changes::fromList(Change::removed('foo', true)),
 
@@ -61,7 +61,7 @@  discard block
 
 block discarded – undo
61 61
62 62 
         $check = $this->createMock(FunctionBased::class);
63 63 
         $check->expects(self::never())
64 
-              ->method('__invoke');
64 
+                ->method('__invoke');
65 65
66 66 
         self::assertEquals(
67 67 
             Changes::empty(),
Please login to merge, or discard this patch.
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch 
@@ -19,7 +19,7 @@  discard block
 
 block discarded – undo
19 19 
 {
20 20 
     public function testNormalFunctionsAreNotExcluded() : void
21 21 
     {
22 
-        $source   = new StringSourceLocator(
22 
+        $source = new StringSourceLocator(
23 23 
             <<<'PHP'
24 24 
 <?php
25 25 
 
@@ -46,7 +46,7 @@  discard block
 
 block discarded – undo
46 46
47 47 
     public function testInternalFunctionsAreExcluded() : void
48 48 
     {
49 
-        $source   = new StringSourceLocator(
49 
+        $source = new StringSourceLocator(
50 50 
             <<<'PHP'
51 51 
 <?php
52 52
Please login to merge, or discard this patch.
unit/DetectChanges/BCBreak/PropertyBased/ExcludeInternalPropertyTest.php 1 patch
Indentation   +4 added lines, -4 removed lines patch added patch discarded remove patch 
@@ -36,9 +36,9 @@  discard block
 
 block discarded – undo
36 36
37 37 
         $check = $this->createMock(PropertyBased::class);
38 38 
         $check->expects(self::once())
39 
-              ->method('__invoke')
40 
-              ->with($property, $property)
41 
-              ->willReturn(Changes::fromList(Change::removed('foo', true)));
39 
+                ->method('__invoke')
40 
+                ->with($property, $property)
41 
+                ->willReturn(Changes::fromList(Change::removed('foo', true)));
42 42
43 43 
         self::assertEquals(
44 44 
             Changes::fromList(Change::removed('foo', true)),
 
@@ -68,7 +68,7 @@  discard block
 
 block discarded – undo
68 68
69 69 
         $check = $this->createMock(PropertyBased::class);
70 70 
         $check->expects(self::never())
71 
-              ->method('__invoke');
71 
+                ->method('__invoke');
72 72
73 73 
         self::assertEquals(
74 74 
             Changes::empty(),
Please login to merge, or discard this patch.
unit/DetectChanges/BCBreak/PropertyBased/PropertyBecameInternalTest.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch 
@@ -36,7 +36,7 @@  discard block
 
 block discarded – undo
36 36
37 37 
         self::assertSame(
38 38 
             $expectedMessages,
39 
-            array_map(static function (Change $change) : string {
39 
+            array_map(static function(Change $change) : string {
40 40 
                 return $change->__toString();
41 41 
             }, iterator_to_array($changes))
42 42 
         );
 
@@ -108,7 +108,7 @@  discard block
 
 block discarded – undo
108 108 
         return array_combine(
109 109 
             array_keys($properties),
110 110 
             array_map(
111 
-                static function (string $property, array $errorMessages) use ($fromClass, $toClass) : array {
111 
+                static function(string $property, array $errorMessages) use ($fromClass, $toClass) : array {
112 112 
                     return [
113 113 
                         $fromClass->getProperty($property),
114 114 
                         $toClass->getProperty($property),
Please login to merge, or discard this patch.
test/unit/DetectChanges/BCBreak/MethodBased/ExcludeInternalMethodTest.php 1 patch
Indentation   +4 added lines, -4 removed lines patch added patch discarded remove patch 
@@ -34,9 +34,9 @@  discard block
 
 block discarded – undo
34 34
35 35 
         $check = $this->createMock(MethodBased::class);
36 36 
         $check->expects(self::once())
37 
-              ->method('__invoke')
38 
-              ->with($method, $method)
39 
-              ->willReturn(Changes::fromList(Change::removed('foo', true)));
37 
+                ->method('__invoke')
38 
+                ->with($method, $method)
39 
+                ->willReturn(Changes::fromList(Change::removed('foo', true)));
40 40
41 41 
         self::assertEquals(
42 42 
             Changes::fromList(Change::removed('foo', true)),
 
@@ -64,7 +64,7 @@  discard block
 
 block discarded – undo
64 64
65 65 
         $check = $this->createMock(MethodBased::class);
66 66 
         $check->expects(self::never())
67 
-              ->method('__invoke');
67 
+                ->method('__invoke');
68 68
69 69 
         self::assertEquals(
70 70 
             Changes::empty(),
Please login to merge, or discard this patch.