Try running unit tests on phpunit6

[seamuslee001]

ping @eileenmcnaughton

[civibot]

(Standard links)

• If this is your first pull-request for CiviCRM, please browse CONTRIBUTING.md for information about the development and testing processes.
• If you are reviewing this pull-request, you may wish to consult the test sites and the Review Standards (long template, short template).

[eileenmcnaughton]

Nice - what about running phpunit 7 for 7.1 + (& maybe 8 for php 7.2+) - we need min phpunit 7 for 7.3 & ideally we would run the most recent version of phpunit that we can to keep our ability to support the latest from regressing

[seamuslee001]

@eileenmcnaughton this needs @totten to deploy latest buildkit on test-2 first but yeh but PHPUnit7 is throwing more errors (plus more code changes). I need to review in depth first

[eileenmcnaughton]

@seamuslee001 so do you have any reservations about me merging this?

[eileenmcnaughton]

I guess it will require some people to update their buildkit?

[seamuslee001]

Yeh pretty much that but i don't have a reservation as such just that we may see failures if something runs on max on test-2 without Tim deploying latest buildkit

[eileenmcnaughton]

let's see how it goes

[eileenmcnaughton]

@seamuslee001 I wonder if any docs need updating now?