SilverStripe\Control\Tests\HttpRequestMockBuilder::buildRequestMock()

Complexity

Conditions	8
Paths	6

Size

Total Lines	35
Code Lines	19

Duplication

Lines	0
Ratio	0 %

Importance

Changes

0

<?php
    public function buildRequestMock($url, $getVars = [], $postVars = [], $method = null, Session $session = null)
    {
        if (is_null($session)) {
            $session = new Session([]);
        }

        $request = $this->createMock(HTTPRequest::class);

It seems like createMock() must be provided by classes using this trait. How about adding it as abstract method to this trait?

        $request->method('getSession')->willReturn($session);

        $request->method('getURL')->will($this->returnCallback(static function ($addParams) use ($url, $getVars) {

It seems like returnCallback() must be provided by classes using this trait. How about adding it as abstract method to this trait?

            return $addParams && count($getVars) ? $url.'?'.http_build_query($getVars) : $url;
        }));

        $request->method('getVars')->willReturn($getVars);
        $request->method('getVar')->will($this->returnCallback(static function ($key) use ($getVars) {
            return isset($getVars[$key]) ? $getVars[$key] : null;
        }));

        $request->method('postVars')->willReturn($postVars);
        $request->method('postVar')->will($this->returnCallback(static function ($key) use ($postVars) {
            return isset($postVars[$key]) ? $postVars[$key] : null;
        }));

        if (is_null($method)) {
            if (count($postVars)) {
                $method = 'POST';
            } else {
                $method = 'GET';
            }
        }

        $request->method('httpMethod')->willReturn($method);

        return $request;
    }