SimpleSAML\Module\monitor\TestCase\AuthSource\Ldap\Bind

Complexity

Total Complexity

5

Size/Duplication

Total Lines	67
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	3

Importance

Changes	5
Bugs	0	Features	0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	initialize()	0	10	1
B	invokeTest()	0	27	4

<?php

namespace SimpleSAML\Module\monitor\TestCase\AuthSource\Ldap;

use \SimpleSAML\Module\monitor\State as State;
use \SimpleSAML\Module\monitor\TestData as TestData;
use \SimpleSAML\Module\monitor\TestResult as TestResult;

final class Bind extends \SimpleSAML\Module\monitor\TestCaseFactory
{
    /*
     * @var \SimpleSAML_Auth_LDAP|null
     */
    private $connection = null;

    /*
     * @var string|null
     */
    private $username = null;

    /*
     * @var string|null
     */

    private $password = null;


    /*
     * @param TestData $testData
     *
     * @return void
     */
    protected function initialize($testData)
    {
        $this->connection = $testData->getInput('connection');
        $authSourceData = $testData->getInput('authSourceData');

        $this->username = $authSourceData['search.username'];
        $this->password = $authSourceData['search.password'];

        parent::initialize($testData);
    }

   
    /*
     * @return void
     */
    protected function invokeTest()
    {
        $testResult = new TestResult('LDAP Bind', $this->username);
        $connection = $this->connection;

        try {
            $bind = $connection->bind($this->username, $this->password);
        } catch (\Exception $e) {
            // Fallthru
        }

        if (isSet($error)) {

The variable $error seems to never exist, and therefore isset should always return false. Did you maybe rename this variable?

            $msg = str_replace('Library - LDAP bind(): ', '', $error->getMessage());
            $state = State::FATAL;
        } elseif ($bind === true) {
            $msg = 'Bind succesful';
            $state = State::OK;
        } else {
            $msg = ldap_error($connection).' ('.ldap_errno($connection).')';
            $state = State::ERROR;
        }

        $testResult->setMessage($msg);
        $testResult->setState($state);

        $this->setTestResult($testResult);
    }
}