Emarref\Jwt\HeaderParameter\CriticalTest

Complexity

Total Complexity

5

Size/Duplication

Total Lines	46
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	3

Importance

Changes	1
Bugs	0	Features	1

5 Methods

Rating	Name	Duplication	Size	Complexity
A	setUp()	0	4	1
A	testGetName()	0	4	1
A	testGetValue()	0	4	1
A	testSetValue()	0	7	1
A	testAddClaim()	0	12	1

<?php

namespace Emarref\Jwt\HeaderParameter;

use Emarref\Jwt\Claim;

class CriticalTest extends \PHPUnit_Framework_TestCase
{
    private static $name  = 'crit';

Consider using a different property name as you override a private property of the parent class.

    private static $value = ['foobar'];

    /**
     * @var Critical
     */
    private $parameter;

    public function setUp()
    {
        $this->parameter = new Critical(self::$value);
    }

    public function testGetName()
    {
        $this->assertSame(self::$name, $this->parameter->getName());
    }

    public function testGetValue()
    {
        $this->assertSame(self::$value, $this->parameter->getValue());
    }

    public function testSetValue()
    {
        $newValue = 'NewValue';

        $this->parameter->setValue($newValue);

$newValue is of type string, but the function expects a array.

        $this->assertSame([$newValue], $this->parameter->getValue());
    }

    public function testAddClaim()
    {
        $parameter = new Custom('foo', 'bar');

        $this->parameter->addParameter($parameter);
        $expectedValue = [self::$value[0], 'foo'];

        $this->assertSame($expectedValue, $this->parameter->getValue());

        $this->parameter->addParameter($parameter);
        $this->assertSame($expectedValue, $this->parameter->getValue(), 'Add parameter duplicates claims');
    }
}