LocalhostMultiTest

Complexity

Total Complexity

34

Size/Duplication

Total Lines	226
Duplicated Lines	42.04 %

Coupling/Cohesion

Components	1
Dependencies	2

Importance

Changes

0

17 Methods

Rating	Name	Duplication	Size	Complexity
B	_runtests()	0	19	5
A	testDeflate()	11	11	2
A	testGzip()	11	11	2
A	testKeepAlives()	11	11	2
A	testProxy()	0	10	2
A	testHttp11()	14	14	2
A	testHttp11Gzip()	14	14	2
A	testHttp11Deflate()	14	14	2
A	testHttp11Proxy()	10	20	3
A	testHttps()	0	16	2
A	testHttpsProxy()	10	22	3
A	testUTF8Responses()	0	6	1
A	testUTF8Requests()	0	5	1
A	testISOResponses()	0	6	1
A	testISORequests()	0	5	1
A	testBasicAuth()	0	6	1
A	testDigestAuth()	0	13	2

<?php

For compatibility and reusability of your code, PSR1 recommends that a file should introduce either new symbols (like classes, functions, etc.) or have side-effects (like outputting something, or including other files), but not both at the same time. The first symbol is defined on line 14 and the first side effect is on line 3.

include_once __DIR__ . '/../lib/xmlrpc.inc';
include_once __DIR__ . '/../lib/xmlrpc_wrappers.inc';

include_once __DIR__ . '/parse_args.php';

include_once __DIR__ . '/3LocalhostTest.php';

/**
 * Tests which stress http features of the library.
 * Each of these tests iterates over (almost) all of the 'localhost' tests
 */
class LocalhostMultiTest extends LocalhostTest

PSR1 recommends that each class must be in a namespace of at least one level to avoid collisions.

{
    /**
     * @todo reintroduce skipping of tests which failed when executed individually if test runs happen as separate processes
     * @todo reintroduce skipping of tests within the loop
     */
    function _runtests()

It is generally recommended to explicitly declare the visibility for methods.

    {
        $unsafeMethods = array('testHttps', 'testCatchExceptions', 'testUtf8Method', 'testServerComments', 'testExoticCharsetsRequests',
            'testExoticCharsetsRequests2', 'testExoticCharsetsRequests3',
            // @todo the following are currently not compatible w Digest Auth (most likely because of client copy) and should be fixed
            'testcatchWarnings', 'testWrappedMethodAsSource', 'testTransferOfObjectViaWrapping');
        foreach(get_class_methods('LocalhostTest') as $method)
        {
            if(strpos($method, 'test') === 0 && !in_array($method, $unsafeMethods))
            {
                if (!isset(self::$failed_tests[$method]))
                    $this->$method();
            }
            /*if ($this->_failed)

65% of this comment could be valid code. Did you maybe forget this after debugging?

            {
                break;
            }*/
        }
    }

    function testDeflate()

It is generally recommended to explicitly declare the visibility for methods.

This method seems to be duplicated in your project.

    {
        if(!function_exists('gzdeflate'))
        {
            $this->markTestSkipped('Zlib missing: cannot test deflate functionality');
            return;
        }
        $this->client->accepted_compression = array('deflate');
        $this->client->request_compression = 'deflate';
        $this->_runtests();
    }

    function testGzip()

It is generally recommended to explicitly declare the visibility for methods.

This method seems to be duplicated in your project.

    {
        if(!function_exists('gzdeflate'))
        {
            $this->markTestSkipped('Zlib missing: cannot test gzip functionality');
            return;
        }
        $this->client->accepted_compression = array('gzip');
        $this->client->request_compression = 'gzip';
        $this->_runtests();
    }

    function testKeepAlives()

It is generally recommended to explicitly declare the visibility for methods.

This method seems to be duplicated in your project.

    {
        if(!function_exists('curl_init'))
        {
            $this->markTestSkipped('CURL missing: cannot test http 1.1');
            return;
        }
        $this->method = 'http11';
        $this->client->keepalive = true;
        $this->_runtests();
    }

    function testProxy()

It is generally recommended to explicitly declare the visibility for methods.

    {
        if ($this->args['PROXYSERVER'])
        {
            $this->client->setProxy($this->args['PROXYSERVER'], $this->args['PROXYPORT']);
            $this->_runtests();
        }
        else
            $this->markTestSkipped('PROXY definition missing: cannot test proxy');
    }

    function testHttp11()

It is generally recommended to explicitly declare the visibility for methods.

This method seems to be duplicated in your project.

    {
        if(!function_exists('curl_init'))
        {
            $this->markTestSkipped('CURL missing: cannot test http 1.1');
            return;
        }
        $this->method = 'http11'; // not an error the double assignment!
        $this->client->method = 'http11';
        //$this->client->verifyhost = 0;

50% of this comment could be valid code. Did you maybe forget this after debugging?

        //$this->client->verifypeer = 0;

50% of this comment could be valid code. Did you maybe forget this after debugging?

        $this->client->keepalive = false;
        $this->_runtests();
    }

    function testHttp11Gzip()

It is generally recommended to explicitly declare the visibility for methods.

This method seems to be duplicated in your project.

    {
        if(!function_exists('curl_init'))
        {
            $this->markTestSkipped('CURL missing: cannot test http 1.1');
            return;
        }
        $this->method = 'http11'; // not an error the double assignment!
        $this->client->method = 'http11';
        $this->client->keepalive = false;
        $this->client->accepted_compression = array('gzip');
        $this->client->request_compression = 'gzip';
        $this->_runtests();
    }

    function testHttp11Deflate()

It is generally recommended to explicitly declare the visibility for methods.

This method seems to be duplicated in your project.

    {
        if(!function_exists('curl_init'))
        {
            $this->markTestSkipped('CURL missing: cannot test http 1.1');
            return;
        }
        $this->method = 'http11'; // not an error the double assignment!
        $this->client->method = 'http11';
        $this->client->keepalive = false;
        $this->client->accepted_compression = array('deflate');
        $this->client->request_compression = 'deflate';
        $this->_runtests();
    }

    function testHttp11Proxy()

It is generally recommended to explicitly declare the visibility for methods.

    {
        if(!function_exists('curl_init'))

This code seems to be duplicated across your project.

        {
            $this->markTestSkipped('CURL missing: cannot test http 1.1 w. proxy');
            return;
        }
        else if ($this->args['PROXYSERVER'] == '')
        {
            $this->markTestSkipped('PROXY definition missing: cannot test proxy w. http 1.1');
            return;
        }
        $this->method = 'http11'; // not an error the double assignment!
        $this->client->method = 'http11';
        $this->client->setProxy($this->args['PROXYSERVER'], $this->args['PROXYPORT']);
        //$this->client->verifyhost = 0;

50% of this comment could be valid code. Did you maybe forget this after debugging?

        //$this->client->verifypeer = 0;

50% of this comment could be valid code. Did you maybe forget this after debugging?

        $this->client->keepalive = false;
        $this->_runtests();
    }

    function testHttps()

It is generally recommended to explicitly declare the visibility for methods.

    {
        if(!function_exists('curl_init'))
        {
            $this->markTestSkipped('CURL missing: cannot test https functionality');
            return;
        }
        $this->client->server = $this->args['HTTPSSERVER'];
        $this->method = 'https';
        $this->client->method = 'https';
        $this->client->path = $this->args['HTTPSURI'];
        $this->client->setSSLVerifyPeer(!$this->args['HTTPSIGNOREPEER']);
        $this->client->setSSLVerifyHost($this->args['HTTPSVERIFYHOST']);
        $this->client->setSSLVersion($this->args['SSLVERSION']);
        $this->_runtests();
    }

    function testHttpsProxy()

It is generally recommended to explicitly declare the visibility for methods.

    {
        if(!function_exists('curl_init'))

This code seems to be duplicated across your project.

        {
            $this->markTestSkipped('CURL missing: cannot test https functionality');
            return;
        }
        else if ($this->args['PROXYSERVER'] == '')
        {
            $this->markTestSkipped('PROXY definition missing: cannot test proxy w. http 1.1');
            return;
        }
        $this->client->server = $this->args['HTTPSSERVER'];
        $this->method = 'https';
        $this->client->method = 'https';
        $this->client->setProxy($this->args['PROXYSERVER'], $this->args['PROXYPORT']);
        $this->client->path = $this->args['HTTPSURI'];
        $this->client->setSSLVerifyPeer(!$this->args['HTTPSIGNOREPEER']);
        $this->client->setSSLVerifyHost($this->args['HTTPSVERIFYHOST']);
        $this->client->setSSLVersion($this->args['SSLVERSION']);
        $this->_runtests();
    }

    function testUTF8Responses()

It is generally recommended to explicitly declare the visibility for methods.

    {
        //$this->client->path = strpos($URI, '?') === null ? $URI.'?RESPONSE_ENCODING=UTF-8' : $URI.'&RESPONSE_ENCODING=UTF-8';

50% of this comment could be valid code. Did you maybe forget this after debugging?

        $this->client->path = $this->args['URI'].'?RESPONSE_ENCODING=UTF-8';
        $this->_runtests();
    }

    function testUTF8Requests()

It is generally recommended to explicitly declare the visibility for methods.

    {
        $this->client->request_charset_encoding = 'UTF-8';
        $this->_runtests();
    }

    function testISOResponses()

It is generally recommended to explicitly declare the visibility for methods.

    {
        //$this->client->path = strpos($URI, '?') === null ? $URI.'?RESPONSE_ENCODING=UTF-8' : $URI.'&RESPONSE_ENCODING=UTF-8';

50% of this comment could be valid code. Did you maybe forget this after debugging?

        $this->client->path = $this->args['URI'].'?RESPONSE_ENCODING=ISO-8859-1';
        $this->_runtests();
    }

    function testISORequests()

It is generally recommended to explicitly declare the visibility for methods.

    {
        $this->client->request_charset_encoding = 'ISO-8859-1';
        $this->_runtests();
    }

    function testBasicAuth()

It is generally recommended to explicitly declare the visibility for methods.

    {
        $this->client->setCredentials('test', 'test');
        $this->client->path = $this->args['URI'].'?FORCE_AUTH=Basic';
        $this->_runtests();
    }

    function testDigestAuth()

It is generally recommended to explicitly declare the visibility for methods.

    {
        if (!function_exists('curl_init'))
        {
            $this->markTestSkipped('CURL missing: cannot test digest auth functionality');
            return;
        }
        $this->client->setCredentials('test', 'test', CURLAUTH_DIGEST);
        $this->client->path = $this->args['URI'].'?FORCE_AUTH=Digest';
        $this->method = 'http11';
        $this->client->method = 'http11';
        $this->_runtests();
    }
}