[DX] Add FileProcessResult + kick of collectors (#4947)

e2e/applied-rule-change-docblock/expected-output.diff

@@ -1,40 +1,40 @@
 2 files with changes
 ====================
 
-1) src/UselessVarTag.php:1
+1) src/RenameDocblock.php:0
 
     ---------- begin diff ----------
 @@ @@
+ <?php
 
- final class UselessVarTag
+ /**
+- * @param DateTime $someOldClass
++ * @param \DateTimeInterface $someOldClass
+  */
+ function someFunction($someOldClass)
  {
--    /**
--     * @var string
--     */
-     public string $name = 'name';
- }
     ----------- end diff -----------
 
 Applied rules:
- * RemoveUselessVarTagRector
+ * RenameClassRector
 
 
-2) src/RenameDocblock.php:0
+2) src/UselessVarTag.php:1
 
     ---------- begin diff ----------
 @@ @@
- <?php
 
- /**
-- * @param DateTime $someOldClass
-+ * @param \DateTimeInterface $someOldClass
-  */
- function someFunction($someOldClass)
+ final class UselessVarTag
  {
+-    /**
+-     * @var string
+-     */
+     public string $name = 'name';
+ }
     ----------- end diff -----------
 
 Applied rules:
- * RenameClassRector
+ * RemoveUselessVarTagRector
 
 
  [OK] 2 files would have changed (dry-run) by Rector

e2e/applied-rule-removed-node-with-cache/expected-output.diff

@@ -1,40 +1,40 @@
 2 files with changes
 ====================
 
-1) src/DeadConstructor.php:1
+1) src/AlwaysTrue.php:3
 
     ---------- begin diff ----------
 @@ @@
-
- final class DeadConstructor
  {
--    public function __construct()
--    {
--    }
+     public function run()
+     {
+-        if (1 === 1) {
+-        }
+-
+         return 'no';
+     }
  }
     ----------- end diff -----------
 
 Applied rules:
- * RemoveEmptyClassMethodRector
+ * RemoveAlwaysTrueIfConditionRector
 
 
-2) src/AlwaysTrue.php:3
+2) src/DeadConstructor.php:1
 
     ---------- begin diff ----------
 @@ @@
+
+ final class DeadConstructor
  {
-     public function run()
-     {
--        if (1 === 1) {
--        }
--
-         return 'no';
-     }
+-    public function __construct()
+-    {
+-    }
  }
     ----------- end diff -----------
 
 Applied rules:
- * RemoveAlwaysTrueIfConditionRector
+ * RemoveEmptyClassMethodRector
 
 
  [OK] 2 files would have changed (dry-run) by Rector

e2e/applied-rule-removed-node/expected-output.diff

@@ -1,40 +1,40 @@
 2 files with changes
 ====================
 
-1) src/DeadConstructor.php:1
+1) src/AlwaysTrue.php:3
 
     ---------- begin diff ----------
 @@ @@
-
- final class DeadConstructor
  {
--    public function __construct()
--    {
--    }
+     public function run()
+     {
+-        if (1 === 1) {
+-        }
+-
+         return 'no';
+     }
  }
     ----------- end diff -----------
 
 Applied rules:
- * RemoveEmptyClassMethodRector
+ * RemoveAlwaysTrueIfConditionRector
 
 
-2) src/AlwaysTrue.php:3
+2) src/DeadConstructor.php:1
 
     ---------- begin diff ----------
 @@ @@
+
+ final class DeadConstructor
  {
-     public function run()
-     {
--        if (1 === 1) {
--        }
--
-         return 'no';
-     }
+-    public function __construct()
+-    {
+-    }
  }
     ----------- end diff -----------
 
 Applied rules:
- * RemoveAlwaysTrueIfConditionRector
+ * RemoveEmptyClassMethodRector
 
 
  [OK] 2 files would have changed (dry-run) by Rector

e2e/applied-rule-return-array-nodes/expected-output.diff

@@ -1,46 +1,46 @@
 2 files with changes
 ====================
 
-1) src/RemoveAlwaysElse.php:5
+1) src/MultiRules.php:5
 
     ---------- begin diff ----------
 @@ @@
      {
-         if ($value) {
-             throw new \InvalidStateException;
+         if (true === false) {
+             return -1;
 -        } else {
--            return 10;
+-            echo 'a statement';
          }
-+        return 10;
+-    }
+-
+-    private function notUsed()
+-    {
++        echo 'a statement';
      }
  }
     ----------- end diff -----------
 
 Applied rules:
+ * RemoveUnusedPrivateMethodRector
  * RemoveAlwaysElseRector
 
 
-2) src/MultiRules.php:5
+2) src/RemoveAlwaysElse.php:5
 
     ---------- begin diff ----------
 @@ @@
      {
-         if (true === false) {
-             return -1;
+         if ($value) {
+             throw new \InvalidStateException;
 -        } else {
--            echo 'a statement';
+-            return 10;
          }
--    }
--
--    private function notUsed()
--    {
-+        echo 'a statement';
++        return 10;
      }
  }
     ----------- end diff -----------
 
 Applied rules:
- * RemoveUnusedPrivateMethodRector
  * RemoveAlwaysElseRector
 
 

e2e/no-parallel-reflection-resolver/expected-output.diff

@@ -1,12 +1,12 @@
 2 files with changes
 ====================
 
-1) src/SomeClassNotFound.php:3
+1) src/NamespacedSomeClassFound.php:5
 
     ---------- begin diff ----------
 @@ @@
 
- final class SomeClassNotFound
+ final class NamespacedSomeClassFound
  {
 -    private $foo;
  }
@@ -16,12 +16,12 @@ Applied rules:
  * RemoveUnusedPrivatePropertyRector
 
 
-2) src/NamespacedSomeClassFound.php:5
+2) src/SomeClassNotFound.php:3
 
     ---------- begin diff ----------
 @@ @@
 
- final class NamespacedSomeClassFound
+ final class SomeClassNotFound
  {
 -    private $foo;
  }

e2e/parallel-reflection-resolver/expected-output.diff

@@ -1,12 +1,12 @@
 2 files with changes
 ====================
 
-1) src/SomeClassNotFound.php:3
+1) src/NamespacedSomeClassFound.php:5
 
     ---------- begin diff ----------
 @@ @@
 
- final class SomeClassNotFound
+ final class NamespacedSomeClassFound
  {
 -    private $foo;
  }
@@ -16,12 +16,12 @@ Applied rules:
  * RemoveUnusedPrivatePropertyRector
 
 
-2) src/NamespacedSomeClassFound.php:5
+2) src/SomeClassNotFound.php:3
 
     ---------- begin diff ----------
 @@ @@
 
- final class NamespacedSomeClassFound
+ final class SomeClassNotFound
  {
 -    private $foo;
  }

e2e/wildcards-path-config/expected-output.diff

@@ -1,40 +1,40 @@
 2 files with changes
 ====================
 
-1) packages/target/src/UselessVarTag.php:1
+1) packages/target/src/RenameDocblock.php:0
 
     ---------- begin diff ----------
 @@ @@
+ <?php
 
- final class UselessVarTag
+ /**
+- * @param DateTime $someOldClass
++ * @param \DateTimeInterface $someOldClass
+  */
+ function someFunction($someOldClass)
  {
--    /**
--     * @var string
--     */
-     public string $name = 'name';
- }
     ----------- end diff -----------
 
 Applied rules:
- * RemoveUselessVarTagRector
+ * RenameClassRector
 
 
-2) packages/target/src/RenameDocblock.php:0
+2) packages/target/src/UselessVarTag.php:1
 
     ---------- begin diff ----------
 @@ @@
- <?php
 
- /**
-- * @param DateTime $someOldClass
-+ * @param \DateTimeInterface $someOldClass
-  */
- function someFunction($someOldClass)
+ final class UselessVarTag
  {
+-    /**
+-     * @var string
+-     */
+     public string $name = 'name';
+ }
     ----------- end diff -----------
 
 Applied rules:
- * RenameClassRector
+ * RemoveUselessVarTagRector
 
 
- [OK] 2 files would have changed (dry-run) by Rector
+ [OK] 2 files would have changed (dry-run) by Rector