integration::testIntegration() - Code Metrics - Inspection of "Moved some resources out :)" - devtoolboxuk/soteria - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( d691dc...d14a3f )

by Rob

created 2019-06-02 11:19 UTC

integration::testIntegration() A

↳ Parent: integration

Complexity

Conditions	3
Paths	4

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
dl	0
loc	32
rs	9.408
c	0
b	0
f	0
cc	3
nc	4
nop	0

<?php

namespace devtoolboxuk\soteria;

use PHPUnit\Framework\TestCase;

class integration extends TestCase

{
    private $security;

    function __construct($name = null, array $data = [], $dataName = '')

    {
        parent::__construct($name, $data, $dataName);
        $this->security = new SoteriaService();
    }

    function testIntegration()

    {
//        $xss = $this->security->xss();
        $sanitise = $this->security->sanitise();


        echo "\nXSS";
        $string = 'Visit my website http://www.doajob.org?redirect=https://www.google.com';
//        echo "\nString: " . $string;
//        $cleanString = $xss->clean($string);
//        echo "\nString: " . $cleanString;
//        echo "\n";
//        echo "\nXSS Url";
//        $string = 'Visit my website http://www.doajob.org?redirect=https://www.google.com';
//        echo "\nString: " . $string;
//        $cleanString = $xss->cleanUrl($string);
//        echo "\nString: " . $cleanString;
//        echo "\n";
//        echo "\nSanitiser";

        echo "\nString: " . $sanitise->removeUrl($string);
        if ($sanitise->isSanitised()) {
            echo "\n1";
        }

        echo "\nString: " . $sanitise->removeUrl("Rob WIlson");
        if ($sanitise->isSanitised()) {
            echo "\n1";
        }


    }

}


1			<?php
2
3			namespace devtoolboxuk\soteria;
4
5			use PHPUnit\Framework\TestCase;
6
7			class integration extends TestCase
			0 ignored issues – show Coding Style introduced 2019-06-02 11:22 UTC by Report Bug Copy Issue Report This class is not in CamelCase format. Classes in PHP are usually named in CamelCase. In camelCase names are written without any punctuation, the start of each new word being marked by a capital letter. The whole name starts with a capital letter as well. Thus the name database provider becomes `DatabaseProvider`. Loading history...
8			{
9			private $security;
10
11			function __construct($name = null, array $data = [], $dataName = '')
			0 ignored issues – show Best Practice introduced 2019-06-02 11:22 UTC by Report Bug Copy Issue Report It is generally recommended to explicitly declare the visibility for methods. Adding explicit visibility (`private`, `protected`, or `public`) is generally recommend to communicate to other developers how, and from where this method is intended to be used. Loading history... Comprehensibility Best Practice introduced 2019-06-02 11:22 UTC by Report Bug Copy Issue Report It is recommend to declare an explicit visibility for `__construct`. Generally, we recommend to declare visibility for all methods in your source code. This has the advantage of clearly communication to other developers, and also yourself, how this method should be consumed. If you are not sure which visibility to choose, it is a good idea to start with the most restrictive visibility, and then raise visibility as needed, i.e. start with `private`, and only raise it to `protected` if a sub-class needs to have access, or `public` if an external class needs access. Loading history...
12			{
13			parent::__construct($name, $data, $dataName);
14			$this->security = new SoteriaService();
15			}
16
17			function testIntegration()
			0 ignored issues – show Best Practice introduced 2019-06-02 11:22 UTC by Report Bug Copy Issue Report It is generally recommended to explicitly declare the visibility for methods. Adding explicit visibility (`private`, `protected`, or `public`) is generally recommend to communicate to other developers how, and from where this method is intended to be used. Loading history... Comprehensibility Best Practice introduced 2019-06-02 11:22 UTC by Report Bug Copy Issue Report It is recommend to declare an explicit visibility for `testIntegration`. Generally, we recommend to declare visibility for all methods in your source code. This has the advantage of clearly communication to other developers, and also yourself, how this method should be consumed. If you are not sure which visibility to choose, it is a good idea to start with the most restrictive visibility, and then raise visibility as needed, i.e. start with `private`, and only raise it to `protected` if a sub-class needs to have access, or `public` if an external class needs access. Loading history...
18			{
19			// $xss = $this->security->xss();
20			$sanitise = $this->security->sanitise();
21
22
23			echo "\nXSS";
24			$string = 'Visit my website http://www.doajob.org?redirect=https://www.google.com';
25			// echo "\nString: " . $string;
26			// $cleanString = $xss->clean($string);
27			// echo "\nString: " . $cleanString;
28			// echo "\n";
29			// echo "\nXSS Url";
30			// $string = 'Visit my website http://www.doajob.org?redirect=https://www.google.com';
31			// echo "\nString: " . $string;
32			// $cleanString = $xss->cleanUrl($string);
33			// echo "\nString: " . $cleanString;
34			// echo "\n";
35			// echo "\nSanitiser";
36
37			echo "\nString: " . $sanitise->removeUrl($string);
38			if ($sanitise->isSanitised()) {
39			echo "\n1";
40			}
41
42			echo "\nString: " . $sanitise->removeUrl("Rob WIlson");
43			if ($sanitise->isSanitised()) {
44			echo "\n1";
45			}
46
47
48			}
49
50			}
51

devtoolboxuk / soteria

Push — master ( d691dc...d14a3f )

integration::testIntegration() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like