facile-it / paraunit

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

--printer flag not respected #151

Closed SteveDesmond-ca closed 1 year ago

SteveDesmond-ca commented 4 years ago

Steps to reproduce:

  1. create a custom printer that does something like outputs the test name after each test result (e.g. ./E/F/etc.)
  2. run vendor/bin/phpunit --printer {printer_class_name}
  3. run vendor/bin/paraunit --printer={printer_class_name}

Expected result:

Observed result:

Alternatively, --testdox support would handle my use case

Jean85 commented 4 years ago

Usign --printer is not easily doable as of now, I would have to reproduce the whole PHPUnit behavior inside Paraunit; also, that extension point in PHPUnit itself is deprecated and will be removed in PHPUnit 9.

As for testdox, I fear that as of now, Paraunit doesn't collect enough info to print out that format; that would probably require a complete rewrite of the feature on this side too.

SteveDesmond-ca commented 4 years ago

:+1: Understandable, I think this issue then becomes "remove the --printer flag from the CLI help text so as to not imply that this functionality exists/works.`

Jean85 commented 1 year ago

Closing as this is completely solved in 2.0 thanks to the new event system, and the --passthrough option when needed. Won't do anything on 1.x, sorry.