AuthSources::invokeTest() - Code Metrics - Inspection of "Update composer.json" - tvdijen/simplesamlphp-module-monitor - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — master ( 4541b1...3f3f73 )

by Tim

created 2018-05-17 07:33 UTC

AuthSources::invokeTest() B

↳ Parent: AuthSources

Complexity

Conditions	7
Paths	11

Size

Total Lines	56
Code Lines	40

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
cc	7
eloc	40
nc	11
nop	0
dl	0
loc	56
rs	7.7489
c	0
b	0
f	0

How to fix Long Method

<?php

namespace SimpleSAML\Module\monitor\TestSuite;

use \SimpleSAML_Configuration as ApplicationConfiguration;
use \SimpleSAML\Module\monitor\TestConfiguration as TestConfiguration;
use \SimpleSAML\Module\monitor\TestResult as TestResult;
use \SimpleSAML\Module\monitor\TestData as TestData;

final class AuthSources extends \SimpleSAML\Module\monitor\TestSuiteFactory
{
    /**
     * @var ApplicationConfiguration
     */
    private $authSourceConfig;

    /**
     * @var array|bool
     */
    private $checkAuthSources;

    /**
     * @var array|null
     */
    private $authSourceSpecifics;

    /**
     * @param TestConfiguration $configuration
     */
    public function __construct($configuration)
    {
        $moduleConfig = $configuration->getModuleConfig();
        $this->authSourceConfig = $configuration->getAuthSourceConfig();
        $this->checkAuthSources = $moduleConfig->getValue('checkAuthSources', true);
        $this->authSourceSpecifics = $moduleConfig->getValue('authSourceSpecifics', null);
        $this->setCategory('Authentication sources');

        parent::__construct($configuration);
    }

    /**
     * @return void
     */
    public function invokeTest()
    {
        if ($this->checkAuthSources === true) {
            $authSources = $this->authSourceConfig->getOptions();
        } else if (is_array($this->checkAuthSources)) {
            $authSources = array_intersect($this->authSourceConfig->getOptions(), $this->checkAuthSources);
        } else { // false or invalid value
            return;
        }

        $configuration = $this->getConfiguration();
        $output = [];

        foreach ($authSources as $authSourceId) {
            $authSourceSpecifics = $this->getAuthSourceSpecifics($authSourceId);
            $authSourceData = $this->authSourceConfig->getValue($authSourceId);
            $input = [
                'authSourceId' => $authSourceId,
                'authSourceData' => ApplicationConfiguration::loadFromArray($this->authSourceConfig->getValue($authSourceId)),
                'authSourceSpecifics' => $authSourceSpecifics,
            ];
            $testData = new TestData($input);

            switch ($authSourceData[0]) {
                case 'ldap:LDAP':
                    $ldapTest = new AuthSource\Ldap($configuration, $testData);
                    $this->addTestResults($ldapTest->getTestResults());
                    $output[$authSourceId] = $ldapTest->getArrayizeTestResults();
                    break;
                case 'negotiate:Negotiate':
                    $negoTest = new AuthSource\Negotiate($configuration, $testData);
                    $this->addTestResults($negoTest->getTestResults());

                    // We need to do some convertions from Negotiate > LDAP
                    $this->convertAuthSourceData($authSourceData);
                    $testData->setInput(ApplicationConfiguration::loadFromArray($authSourceData), 'authSourceData');

                    $ldapTest = new AuthSource\Ldap($configuration, $testData);
                    $this->addTestResults($ldapTest->getTestResults());

                    $output[$authSourceId] = array_merge($negoTest->getArrayizeTestResults(), $ldapTest->getArrayizeTestResults());
                    break;
                case 'multiauth:MultiAuth':
                    // Relies on other authSources
                    continue 2;
                default:
                    // Not implemented
                    continue 2;
            }
        }

        $state = $this->calculateState();
        $testResult = new TestResult('Authentication sources');
        $testResult->setState($state);
        $testResult->setOutput($output);
        $this->setTestResult($testResult);
    }

    /**
     * @param string $authSourceId
     *
     * @return array|null
     */
    private function getAuthSourceSpecifics($authSourceId)
    {
        if (is_array($this->authSourceSpecifics)) {
            if (array_key_exists($authSourceId, $this->authSourceSpecifics)) {
                return $this->authSourceSpecifics[$authSourceId];
            }
        }
        return null;
    }

    /**
     * @param array $authSourceData
     *
     * @return void
     */
    private function convertAuthSourceData(&$authSourceData)
    {
        // LDAP and Negotiate authSources use different names for equal properties
        // Hopefully this function can go away in SSP 2.0
        if (isSet($authSourceData['debugLDAP'])) {
            $authSourceData['debug'] = $authSourceData['debugLDAP'];
            unset($authSourceData['debugLDAP']);
        }
        if (isSet($authSourceData['adminUser'])) {
            $authSourceData['search.username'] = $authSourceData['adminUser'];
            unset($authSourceData['adminUser']);
        }
        if (isSet($authSourceData['adminPassword'])) {
            $authSourceData['search.password'] = $authSourceData['adminPassword'];
            unset($authSourceData['adminPassword']);
        }
        if (isSet($authSourceData['base'])) {
            $authSourceData['search.base'] = $authSourceData['base'];
            unset($authSourceData['base']);
        }
    }
}


1			<?php
2
3			namespace SimpleSAML\Module\monitor\TestSuite;
4
5			use \SimpleSAML_Configuration as ApplicationConfiguration;
6			use \SimpleSAML\Module\monitor\TestConfiguration as TestConfiguration;
7			use \SimpleSAML\Module\monitor\TestResult as TestResult;
8			use \SimpleSAML\Module\monitor\TestData as TestData;
9
10			final class AuthSources extends \SimpleSAML\Module\monitor\TestSuiteFactory
11			{
12			/**
13			* @var ApplicationConfiguration
14			*/
15			private $authSourceConfig;
16
17			/**
18			* @var array\|bool
19			*/
20			private $checkAuthSources;
21
22			/**
23			* @var array\|null
24			*/
25			private $authSourceSpecifics;
26
27			/**
28			* @param TestConfiguration $configuration
29			*/
30			public function __construct($configuration)
31			{
32			$moduleConfig = $configuration->getModuleConfig();
33			$this->authSourceConfig = $configuration->getAuthSourceConfig();
34			$this->checkAuthSources = $moduleConfig->getValue('checkAuthSources', true);
35			$this->authSourceSpecifics = $moduleConfig->getValue('authSourceSpecifics', null);
36			$this->setCategory('Authentication sources');
37
38			parent::__construct($configuration);
39			}
40
41			/**
42			* @return void
43			*/
44			public function invokeTest()
45			{
46			if ($this->checkAuthSources === true) {
47			$authSources = $this->authSourceConfig->getOptions();
48			} else if (is_array($this->checkAuthSources)) {
49			$authSources = array_intersect($this->authSourceConfig->getOptions(), $this->checkAuthSources);
50			} else { // false or invalid value
51			return;
52			}
53
54			$configuration = $this->getConfiguration();
55			$output = [];
56
57			foreach ($authSources as $authSourceId) {
58			$authSourceSpecifics = $this->getAuthSourceSpecifics($authSourceId);
59			$authSourceData = $this->authSourceConfig->getValue($authSourceId);
60			$input = [
61			'authSourceId' => $authSourceId,
62			'authSourceData' => ApplicationConfiguration::loadFromArray($this->authSourceConfig->getValue($authSourceId)),
63			'authSourceSpecifics' => $authSourceSpecifics,
64			];
65			$testData = new TestData($input);
66
67			switch ($authSourceData[0]) {
68			case 'ldap:LDAP':
69			$ldapTest = new AuthSource\Ldap($configuration, $testData);
70			$this->addTestResults($ldapTest->getTestResults());
71			$output[$authSourceId] = $ldapTest->getArrayizeTestResults();
72			break;
73			case 'negotiate:Negotiate':
74			$negoTest = new AuthSource\Negotiate($configuration, $testData);
75			$this->addTestResults($negoTest->getTestResults());
76
77			// We need to do some convertions from Negotiate > LDAP
78			$this->convertAuthSourceData($authSourceData);
79			$testData->setInput(ApplicationConfiguration::loadFromArray($authSourceData), 'authSourceData');
80
81			$ldapTest = new AuthSource\Ldap($configuration, $testData);
82			$this->addTestResults($ldapTest->getTestResults());
83
84			$output[$authSourceId] = array_merge($negoTest->getArrayizeTestResults(), $ldapTest->getArrayizeTestResults());
85			break;
86			case 'multiauth:MultiAuth':
87			// Relies on other authSources
88			continue 2;
89			default:
90			// Not implemented
91			continue 2;
92			}
93			}
94
95			$state = $this->calculateState();
96			$testResult = new TestResult('Authentication sources');
97			$testResult->setState($state);
98			$testResult->setOutput($output);
99			$this->setTestResult($testResult);
100			}
101
102			/**
103			* @param string $authSourceId
104			*
105			* @return array\|null
106			*/
107			private function getAuthSourceSpecifics($authSourceId)
108			{
109			if (is_array($this->authSourceSpecifics)) {
110			if (array_key_exists($authSourceId, $this->authSourceSpecifics)) {
111			return $this->authSourceSpecifics[$authSourceId];
112			}
113			}
114			return null;
115			}
116
117			/**
118			* @param array $authSourceData
119			*
120			* @return void
121			*/
122			private function convertAuthSourceData(&$authSourceData)
123			{
124			// LDAP and Negotiate authSources use different names for equal properties
125			// Hopefully this function can go away in SSP 2.0
126			if (isSet($authSourceData['debugLDAP'])) {
127			$authSourceData['debug'] = $authSourceData['debugLDAP'];
128			unset($authSourceData['debugLDAP']);
129			}
130			if (isSet($authSourceData['adminUser'])) {
131			$authSourceData['search.username'] = $authSourceData['adminUser'];
132			unset($authSourceData['adminUser']);
133			}
134			if (isSet($authSourceData['adminPassword'])) {
135			$authSourceData['search.password'] = $authSourceData['adminPassword'];
136			unset($authSourceData['adminPassword']);
137			}
138			if (isSet($authSourceData['base'])) {
139			$authSourceData['search.base'] = $authSourceData['base'];
140			unset($authSourceData['base']);
141			}
142			}
143			}
144

tvdijen / simplesamlphp-module-monitor

Push — master ( 4541b1...3f3f73 )

AuthSources::invokeTest() B

Complexity

Size

Duplication

Importance

How to fix Long Method

Long Method

Duplication Side-by-Side

Filter issues like