EasyHttp\MockBuilder\Expectations\HeaderExistsExpectation::from()

Complexity

Conditions	3
Paths	1

Size

Total Lines	12
Code Lines	5

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	6
CRAP Score	3

Importance

Changes	1
Bugs	0	Features	0

<?php
    public static function from(Expectation $expectation): callable
    {
        return function ($request) use ($expectation) {
            /** @var RequestInterface $request */

            foreach ($expectation->emptyHeadersIterator() as $param => $value) {
                if (!$request->hasHeader($param)) {

It seems like $param can also be of type null; however, parameter $name of Psr\Http\Message\MessageInterface::hasHeader() does only seem to accept string, maybe add an additional type check?

                    return new RejectedPromise('header \'' . $param . '\' is missing');
                }
            }

            return $request;
        };