Skip to content

Conversation

@jtojnar
Copy link
Contributor

@jtojnar jtojnar commented Nov 12, 2022

No description provided.

@samsonasik
Copy link
Member

You need to register new e2e dir to :

Otherwise `RemovedAndAddedFilesCollector` will print something like the following and that would need to be baked into `expected-output.diff`:

    [NOTE] File "/home/user/Projects/rector-src/e2e/removed-and-added-files-collector/test" will be added
Currently failing when parallel running is enabled.
@jtojnar jtojnar force-pushed the modifier-integration branch from 85f9d00 to 734da13 Compare November 12, 2022 19:44
]);

// TODO: Make it run in parallel.
$rectorConfig->disableParallel();
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

To reproduce the failure, remove this line.

@@ -0,0 +1,6 @@
! [NOTE] File "./removed-and-added-files-collector/test" will be added
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No idea why CI messes it up like this:

@@ @@
-! [NOTE] File
- !        "./removed-and-added-files-c
- !        ollector/test" will be added
+! [NOTE] File "./removed-and-added-files-collector/test" will be added

@samsonasik
Copy link
Member

$this->removedAndAddedFilesProcessor->run($configuration) seems missing in parallel process.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants