SimpleSAML\Module\monitor\TestSuite

Complexity

Total Complexity

8

Size/Duplication

Total Lines	84
Duplicated Lines	0 %

Coupling/Cohesion

Components	2
Dependencies	1

Importance

Changes

0

8 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	11	1
	initialize()	0	1	?
A	setMonitor()	0	5	1
A	getMonitor()	0	5	1
A	addTest()	0	5	1
A	getTests()	0	5	1
A	calculateState()	0	12	3
	invokeTestSuite()	0	1	?

<?php

namespace SimpleSAML\Module\monitor;

abstract class TestSuite extends TestFactory

TestSuite does not seem to conform to the naming convention (^sspmod_monitor_([A-Z][a-zA-Z0-9_]+)+$).

TestSuite does not seem to conform to the naming convention (^Abstract|Factory$).

{
    private $monitor = null;
    private $tests = null;

    /**
     * @param sspmod_monitor_Monitor $monitor
     */
    public function __construct($monitor, $input)
    {
        assert($monitor instanceof sspmod_monitor_Monitor);

The class SimpleSAML\Module\monitor\sspmod_monitor_Monitor does not exist. Did you forget a USE statement, or did you not list all dependencies?

        assert(is_array($input));

        $this->setMonitor($monitor);
        $this->setInput($input);
        $this->initialize();
        $this->setInput(null);
        $this->invokeTestSuite();
    }

    /*
     * @return void
     */
    abstract protected function initialize();

    /*
     * @param sspmod_monitor_Monitor $monitor
     *
     * @return void
     */
    private function setMonitor($monitor)
    {
        assert($monitor instanceof Monitor);
        $this->monitor = $monitor;
    }

    /*
     * @return sspmod_monitor_Monitor

Consider making the return type a bit more specific; maybe use NoType.

     */
    public function getMonitor()
    {
        assert(is_a($this->monitor, 'sspmod_monitor_Monitor'));
        return $this->monitor;
    }

    /*
     * @return void
     */
    protected function addTest($test)
    {
        assert(is_a($test, 'sspmod_monitor_Test'));
        $this->tests[] = $test;
    }

    /*
     * @return array
     */
    public function getTests()
    {
        assert(is_array($this->tests));
        return $this->tests;
    }

    /*
     * @return void
     */
    protected function calculateState()
    {
        $tests = $this->getTests();

        if (!empty($tests)) {
            $overall = array();
            foreach ($tests as $test) {
                $overall[] = $test->getState();
            }
            $this->setState(min($overall));
        }
    }

    /*
     * @return void
     */
    abstract protected function invokeTestSuite();
}