Fix ComparisonFailure constructor compatibility with sebastian/comparator v8

[jorgsowa]

sebastian/comparator < 5 has a 6-param constructor with $identical (bool) as 5th param, while >= 8 also has 6 params but with $contextLines (int) as 6th param. The previous check only looked at parameter count (>= 6), causing wrong types to be passed on v8. Now also checks the parameter name to distinguish the two signatures.

Fixes #669

[stof]

Closing this in favor of #671 in which I implemented a better fix, to actually support the new feature of sebastian/comparator 8.1+.

[jorgsowa]

Thanks!

I think that workaround for sebastian/comparator < 5 may be even removed from the code. I didn't think about it previously.

[stof]

@jorgsowa it cannot be removed as long as we don't remove support for using Prophecy alongside those versions of sebastian/comparator (which ultimately restrict the compatibility with phpunit):

prophecy/composer.json

Line 23 in 09c2e59

"sebastian/comparator": "^3.0 || ^4.0 || ^5.0 || ^6.0 || ^7.0 || ^8.0",