SimpleSAML\Module\monitor\TestResult

Complexity

Total Complexity

3

Size/Duplication

Total Lines	52
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	0

Importance

Changes	2
Bugs	0	Features	0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	5	1
A	setState()	0	5	1
A	getState()	0	5	1

<?php

namespace SimpleSAML\Module\monitor;

final class TestResult
{
    /**
     * @var string
     */
    private $category;

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

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

The property $message is not used and could be removed.

    /**
     * @var State
     */
    private $state = State::NOSTATE;

    /**
     * @param string $category
     * @param string $subject
     */
    public function __construct($category = 'Unknown category', $subject = 'Unknown subject')
    {
        $this->category = $category;
        $this->subject = $subject;
    }

    /**
     * @param State $state
     *
     * @return void
     */
    public function setState($state = State::NOSTATE)
    {
        assert($state instanceof State);
        $this->$state = $state;
    }

    /**
     * @return State
     */
    public function getState()
    {
        assert($this->state instanceof State);
        return $this->$state;

The variable $state does not exist. Did you forget to declare it?

    }
}