You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Currently in some parts of the code, such as the new composite reporter, we assume that a reporter will have the setEventEmitter() method, which comes from HasEventEmitterTrait that we add in AbstractTest:
array_map(function (ReporterInterface$reporter) use ($eventEmitter) {
$reporter->setEventEmitter($eventEmitter);
}, $this->reporters);
There's been some other related stuff where we typehint on abstract classes when we should really use an interface instead. I think we need to audit our typehints, and interfaces.
The text was updated successfully, but these errors were encountered:
Currently in some parts of the code, such as the new composite reporter, we assume that a reporter will have the
setEventEmitter()
method, which comes fromHasEventEmitterTrait
that we add inAbstractTest
:There's been some other related stuff where we typehint on abstract classes when we should really use an interface instead. I think we need to audit our typehints, and interfaces.
The text was updated successfully, but these errors were encountered: