Passed
Push — master ( 0ecf6c...b70a53 )
by Tim
01:45
created

TestSqlTest::testSqlFailure()   A

Complexity

Conditions 1
Paths 1

Size

Total Lines 17
Code Lines 12

Duplication

Lines 0
Ratio 0 %

Importance

Changes 0
Metric Value
cc 1
eloc 12
nc 1
nop 0
dl 0
loc 17
rs 9.4285
c 0
b 0
f 0
1
<?php
2
3
namespace SimpleSAML\Module\monitor\Test;
4
5
use \SimpleSAML\Module\monitor\TestCase\Store\Sql as Sql;
6
use \SimpleSAML\Module\monitor\State as State;
7
8
/**
9
 * Tests for Sql
10
 * 
11
 * @requires function \SimpleSAML_Configuration::setPreLoadedConfig
12
 */
13
class TestSqlTest extends \SimpleSAML\Test\Utils\ClearStateTestCase
14
{
15
    public function testSqlSuccess()
16
    {
17
        $globalConfig_input = [
18
            'store.type' => 'sql',
19
            'store.sql.dsn' => 'sqlite:/modules/monitor/tests/files/test.sqlite',
20
            'store.sql.username' => 'test',
21
            'store.sql.password' => 'test',
22
            'store.sql.options' => null,
23
            'store.sql.prefix' => 'test'
24
        ];
25
26
        $globalConfig = \SimpleSAML_Configuration::loadFromArray($globalConfig_input);
27
        \SimpleSAML_Configuration::setPreLoadedConfig($globalConfig, 'config.php');
0 ignored issues
show
Bug introduced by
The method setPreLoadedConfig() does not exist on SimpleSAML_Configuration. ( Ignorable by Annotation )

If this is a false-positive, you can also ignore this issue in your code via the ignore-call  annotation

27
        \SimpleSAML_Configuration::/** @scrutinizer ignore-call */ 
28
                                   setPreLoadedConfig($globalConfig, 'config.php');

This check looks for calls to methods that do not seem to exist on a given type. It looks for the method on the type itself as well as in inherited classes or implemented interfaces.

This is most likely a typographical error or the method has been renamed.

Loading history...
28
29
        $testResult = $test->getTestResult();
0 ignored issues
show
Comprehensibility Best Practice introduced by
The variable $test seems to be never defined.
Loading history...
30
        $this->assertEquals(State::OK, $testResult->getState());
31
    }
32
33
    public function testSqlFailure()
34
    {
35
        $globalConfig_input = [
36
            'store.type' => 'sql',
37
            'store.sql.dsn' => '',
38
            'store.sql.username' => '',
39
            'store.sql.password' => '',
40
            'store.sql.options' => null,
41
            'store.sql.prefix' => 'test'
42
        ];
43
44
        $globalConfig = \SimpleSAML_Configuration::loadFromArray($globalConfig_input);
45
        \SimpleSAML_Configuration::setPreLoadedConfig($globalConfig, 'config.php');
46
47
        $test = new Sql();
0 ignored issues
show
Bug introduced by
The call to SimpleSAML\Module\monito...tore\Sql::__construct() has too few arguments starting with testData. ( Ignorable by Annotation )

If this is a false-positive, you can also ignore this issue in your code via the ignore-call  annotation

47
        $test = /** @scrutinizer ignore-call */ new Sql();

This check compares calls to functions or methods with their respective definitions. If the call has less arguments than are defined, it raises an issue.

If a function is defined several times with a different number of parameters, the check may pick up the wrong definition and report false positives. One codebase where this has been known to happen is Wordpress. Please note the @ignore annotation hint above.

Loading history...
48
        $testResult = $test->getTestResult();
49
        $this->assertEquals(State::FATAL, $testResult->getState());
50
    }
51
}
52