facile-it / paraunit

Run PHPUnit tests in parallel
https://engineering.facile.it/paraunit/
Apache License 2.0
134 stars 14 forks source link

Re evaluate PHPUnit CLI options #177

Open Jean85 opened 1 year ago

Jean85 commented 1 year ago

After #174, PHPUnit CLI options should be re evaluated. Any option that doesn't affect Paraunit's behavior should be dropped, while newer ones should be incorporated and passed through. In this issue, We will cover just writing a self-documenting test that will extrapolate the situation of each option (so that it could be linked from the docs) and emit a warning when a new option is discovered.

Unusable

Done in #195: These options should be incompatible with Paraunit; ~we should ~document~ fail when detecting those (see #174)~

This list is available in here in the code: https://github.com/facile-it/paraunit/blob/0810462027a8cbe57cf9f2df095265723a2ad4a4/src/Configuration/PassThrough.php#L9-L24

To be handled in the future

To be ignored / used with --passthrough

The following options ~should be only listed in the test as allowed, so that no warning is triggered in the self-documenting test:~ are listed in this test data provider for the self-documenting test: https://github.com/facile-it/paraunit/blob/0810462027a8cbe57cf9f2df095265723a2ad4a4/tests/Functional/Configuration/PassThroughTest.php#L74-L94

Usable

The following options are usable with --pass-through and listed in this test data provider for the self-documenting test: https://github.com/facile-it/paraunit/blob/0810462027a8cbe57cf9f2df095265723a2ad4a4/tests/Functional/Configuration/PassThroughTest.php#L126-L170