HttpClient::sendRequest() - Code Metrics - Inspection of "Add sendRequest method to test client" - FriendsOfSymfony/FOSHttpCache - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Pull Request — master (#340)

by David de

created 2016-12-05 14:26 UTC

HttpClient::sendRequest() A

↳ Parent: HttpClient

Complexity

Conditions	1
Paths	1

Size

Total Lines	8
Code Lines	3

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	3
CRAP Score	1

Importance

Changes

Metric	Value
dl	0
loc	8
ccs	3
cts	3
cp	1
rs	9.4285
c	0
b	0
f	0
cc	1
eloc	3
nc	1
nop	1
crap	1

<?php

/*
 * This file is part of the FOSHttpCache package.
 *
 * (c) FriendsOfSymfony <http://friendsofsymfony.github.com/>
 *
 * For the full copyright and license information, please view the LICENSE
 * file that was distributed with this source code.
 */

namespace FOS\HttpCache\Test;

use Http\Discovery\HttpClientDiscovery;
use Http\Discovery\MessageFactoryDiscovery;
use Http\Discovery\UriFactoryDiscovery;
use Http\Client\HttpClient as PhpHttpClient;
use Psr\Http\Message\RequestInterface;
use Psr\Http\Message\ResponseInterface;
use Psr\Http\Message\UriInterface;

/**
 * Provides a very simple way to fetch HTTP responses, auto-creating a client.
 */
class HttpClient
{
    /**
     * HTTP client for requests to the application.
     *
     * @var PhpHttpClient
     */
    private $httpClient;

    /**
     * @var string
     */
    private $hostname;

    /**
     * @var string
     */
    private $port;

    /**
     * @param string $hostname Default hostname if not specified in the URL
     * @param string $port     Default port if not specified in the URL
     */
    public function __construct($hostname, $port)
    {
        $this->hostname = $hostname;
        $this->port = $port;
    }

    /**
     * Get HTTP response from your application.
     *
     * @param string $uri     HTTP URI
     * @param array  $headers HTTP headers
     * @param string $method  HTTP method
     *
     * @return ResponseInterface
     */
    public function getResponse($uri, array $headers, $method)
    {
        $request = $this->createRequest($method, $uri, $headers);

        return $this->sendRequest($request);
    }

    /**
     * Send HTTP request to your application.
     *
     * @param RequestInterface $request
     *
     * @return ResponseInterface
     */
    public function sendRequest(RequestInterface $request)
    {
        // Close connections to make sure invalidation (PURGE/BAN) requests will
        // not interfere with content (GET) requests.
        $request = $request->withHeader('Connection', 'Close');

        return $this->getHttpClient()->sendRequest($request);
    }

    /**
     * Get HTTP client for your application.
     *
     * @return PhpHttpClient
     */
    private function getHttpClient()
    {
        if ($this->httpClient === null) {
            $this->httpClient = HttpClientDiscovery::find();
        }

        return $this->httpClient;
    }

    /**
     * Create a request.
     *
     * @param string $method
     * @param string $uri
     * @param array  $headers
     *
     * @return RequestInterface
     *
     * @throws \Exception
     */
    private function createRequest($method, $uri, $headers)
    {
        $uri = $this->createUri($uri);
        if ($uri->getHost() === '') {
            // Add base URI host
            $uri = $uri->withHost($this->hostname);
        }

        if (!$uri->getPort()) {
0   == false // true
0   == null  // true
123 == false // false
123 == null  // false

// It is often better to use strict comparison
0 === false // false
0 === null  // false
            $uri = $uri->withPort($this->port);
        }

        if ($uri->getScheme() === '') {
            $uri = $uri->withScheme('http');
        }

        return MessageFactoryDiscovery::find()->createRequest(
            $method,
            $uri,
            $headers
        );
    }

    /**
     * Create PSR-7 URI object from URI string.
     *
     * @param string $uriString
     *
     * @return UriInterface
     */
    private function createUri($uriString)
    {
        return UriFactoryDiscovery::find()->createUri($uriString);
    }
}


1		<?php
2
3		/*
4		* This file is part of the FOSHttpCache package.
5		*
6		* (c) FriendsOfSymfony <http://friendsofsymfony.github.com/>
7		*
8		* For the full copyright and license information, please view the LICENSE
9		* file that was distributed with this source code.
10		*/
11
12		namespace FOS\HttpCache\Test;
13
14		use Http\Discovery\HttpClientDiscovery;
15		use Http\Discovery\MessageFactoryDiscovery;
16		use Http\Discovery\UriFactoryDiscovery;
17		use Http\Client\HttpClient as PhpHttpClient;
18		use Psr\Http\Message\RequestInterface;
19		use Psr\Http\Message\ResponseInterface;
20		use Psr\Http\Message\UriInterface;
21
22		/**
23		* Provides a very simple way to fetch HTTP responses, auto-creating a client.
24		*/
25		class HttpClient
26		{
27		/**
28		* HTTP client for requests to the application.
29		*
30		* @var PhpHttpClient
31		*/
32		private $httpClient;
33
34		/**
35		* @var string
36		*/
37		private $hostname;
38
39		/**
40		* @var string
41		*/
42		private $port;
43
44		/**
45		* @param string $hostname Default hostname if not specified in the URL
46		* @param string $port Default port if not specified in the URL
47		*/
48	33	public function __construct($hostname, $port)
49		{
50	33	$this->hostname = $hostname;
51	33	$this->port = $port;
52	33	}
53
54		/**
55		* Get HTTP response from your application.
56		*
57		* @param string $uri HTTP URI
58		* @param array $headers HTTP headers
59		* @param string $method HTTP method
60		*
61		* @return ResponseInterface
62		*/
63	33	public function getResponse($uri, array $headers, $method)
64		{
65	33	$request = $this->createRequest($method, $uri, $headers);
66
67	33	return $this->sendRequest($request);
68		}
69
70		/**
71		* Send HTTP request to your application.
72		*
73		* @param RequestInterface $request
74		*
75		* @return ResponseInterface
76		*/
77	33	public function sendRequest(RequestInterface $request)
78		{
79		// Close connections to make sure invalidation (PURGE/BAN) requests will
80		// not interfere with content (GET) requests.
81	33	$request = $request->withHeader('Connection', 'Close');
82
83	33	return $this->getHttpClient()->sendRequest($request);
84		}
85
86		/**
87		* Get HTTP client for your application.
88		*
89		* @return PhpHttpClient
90		*/
91	33	private function getHttpClient()
92		{
93	33	if ($this->httpClient === null) {
94	33	$this->httpClient = HttpClientDiscovery::find();
95	33	}
96
97	33	return $this->httpClient;
98		}
99
100		/**
101		* Create a request.
102		*
103		* @param string $method
104		* @param string $uri
105		* @param array $headers
106		*
107		* @return RequestInterface
108		*
109		* @throws \Exception
110		*/
111	33	private function createRequest($method, $uri, $headers)
112		{
113	33	$uri = $this->createUri($uri);
114	33	if ($uri->getHost() === '') {
115		// Add base URI host
116	33	$uri = $uri->withHost($this->hostname);
117	33	}
118
119	33	if (!$uri->getPort()) {
		0 ignored issues – show Bug Best Practice introduced 2015-11-23 22:08 UTC by Report Bug Copy Issue Report The expression `$uri->getPort()` of type `null\|integer` is loosely compared to `false`; this is ambiguous if the integer can be zero. You might want to explicitly use `=== null` instead. In PHP, under loose comparison (like `==`, or `!=`, or `switch` conditions), values of different types might be equal. For `integer` values, zero is a special case, in particular the following results might be unexpected: 0 == false // true 0 == null // true 123 == false // false 123 == null // false // It is often better to use strict comparison 0 === false // false 0 === null // false Loading history...
120	33	$uri = $uri->withPort($this->port);
121	33	}
122
123	33	if ($uri->getScheme() === '') {
124	33	$uri = $uri->withScheme('http');
125	33	}
126
127	33	return MessageFactoryDiscovery::find()->createRequest(
128	33	$method,
129	33	$uri,
130		$headers
131	33	);
132		}
133
134		/**
135		* Create PSR-7 URI object from URI string.
136		*
137		* @param string $uriString
138		*
139		* @return UriInterface
140		*/
141	33	private function createUri($uriString)
142		{
143	33	return UriFactoryDiscovery::find()->createUri($uriString);
144		}
145		}
146

FriendsOfSymfony / FOSHttpCache

Pull Request — master (#340)

HttpClient::sendRequest() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like