Test Failed
Pull Request — master (#26)
by Christopher
02:56
created

xsBase64BinaryTest   A

Complexity

Total Complexity 8

Size/Duplication

Total Lines 70
Duplicated Lines 0 %

Coupling/Cohesion

Components 1
Dependencies 2

Importance

Changes 2
Bugs 0 Features 0
Metric Value
wmc 8
c 2
b 0
f 0
lcom 1
cbo 2
dl 0
loc 70
rs 10

6 Methods

Rating   Name   Duplication   Size   Complexity  
A setUp() 0 4 1
A tearDown() 0 4 1
A testxsBase64BinaryValidDataProvider() 0 10 1
A testxsBase64BinaryInvalidDataProvider() 0 7 1
A testxsBase64BinaryValid() 0 9 2
A testxsBase64BinaryInvalid() 0 10 2
1
<?php
2
3
4
namespace AlgoWeb\xsdTypes;
5
6
/**
7
 * Generated Test Class.
8
 */
9
class xsBase64BinaryTest extends \PHPUnit_Framework_TestCase
10
{
11
    /**
12
     * Sets up the fixture, for example, opens a network connection.
13
     * This method is called before a test is executed.
14
     */
15
    protected function setUp()
16
    {
17
        parent::setUp();
18
    }
19
20
    /**
21
     * Tears down the fixture, for example, closes a network connection.
22
     * This method is called after a test is executed.
23
     */
24
    protected function tearDown()
25
    {
26
        parent::tearDown();
27
    }
28
29
    /**
30
     * @dataProvider testxsBase64BinaryValidDataProvider
31
     * @param mixed $input
32
     * @param mixed $message
33
     */
34
    public function testxsBase64BinaryValid($input, $message)
35
    {
36
        try {
37
            $d = new xsBase64Binary($input);
38
            $e = (string)$d;
0 ignored issues
show
Unused Code introduced by
$e is not used, you could remove the assignment.

This check looks for variable assignements that are either overwritten by other assignments or where the variable is not used subsequently.

$myVar = 'Value';
$higher = false;

if (rand(1, 6) > 3) {
    $higher = true;
} else {
    $higher = false;
}

Both the $myVar assignment in line 1 and the $higher assignment in line 2 are dead. The first because $myVar is never used and the second because $higher is always overwritten for every possible time line.

Loading history...
39
        } catch (\Exception $e) {
40
            $this->fail($message . ' with Exception ' . $e->getMessage());
41
        }
42
    }
43
44
    public function testxsBase64BinaryValidDataProvider()
45
    {
46
        return array(
47
            array('0FB8', 'Uppercase base64'),
48
            array('0fb8', 'Lowercase base64'),
49
            array('0 FB8 0F+9', 'whitespace is allowed anywhere in the value'),
50
            array('0F+40A==', 'equals signs are used for padding'),
51
            array('', 'an empty value is valid'),
52
        );
53
    }
54
55
    /**
56
     * @dataProvider testxsBase64BinaryInvalidDataProvider
57
     * @param mixed $input
58
     * @param mixed $message
59
     */
60
    public function testxsBase64BinaryInvalid($input, $message)
61
    {
62
        try {
63
            $d = new xsBase64Binary($input);
64
            $s = (string)$d;
65
            $this->fail($message);
66
        } catch (\Exception $e) {
0 ignored issues
show
Coding Style Comprehensibility introduced by
Consider adding a comment why this CATCH block is empty.
Loading history...
67
        }
68
        $this->assertEquals('', $s, $message);
0 ignored issues
show
Bug introduced by
The variable $s does not seem to be defined for all execution paths leading up to this point.

If you define a variable conditionally, it can happen that it is not defined for all execution paths.

Let’s take a look at an example:

function myFunction($a) {
    switch ($a) {
        case 'foo':
            $x = 1;
            break;

        case 'bar':
            $x = 2;
            break;
    }

    // $x is potentially undefined here.
    echo $x;
}

In the above example, the variable $x is defined if you pass “foo” or “bar” as argument for $a. However, since the switch statement has no default case statement, if you pass any other value, the variable $x would be undefined.

Available Fixes

  1. Check for existence of the variable explicitly:

    function myFunction($a) {
        switch ($a) {
            case 'foo':
                $x = 1;
                break;
    
            case 'bar':
                $x = 2;
                break;
        }
    
        if (isset($x)) { // Make sure it's always set.
            echo $x;
        }
    }
    
  2. Define a default value for the variable:

    function myFunction($a) {
        $x = ''; // Set a default which gets overridden for certain paths.
        switch ($a) {
            case 'foo':
                $x = 1;
                break;
    
            case 'bar':
                $x = 2;
                break;
        }
    
        echo $x;
    }
    
  3. Add a value for the missing path:

    function myFunction($a) {
        switch ($a) {
            case 'foo':
                $x = 1;
                break;
    
            case 'bar':
                $x = 2;
                break;
    
            // We add support for the missing case.
            default:
                $x = '';
                break;
        }
    
        echo $x;
    }
    
Loading history...
69
    }
70
71
    public function testxsBase64BinaryInvalidDataProvider()
72
    {
73
        return array(
74
            array('FB8', 'an odd number of characters is not valid; characters appear in groups of four'),
75
            array('==0F', 'equals signs may only appear at the end'),
76
        );
77
    }
78
}
79