Passed
Push — master ( 250f83...0ecf6c )
by Tim
01:38
created

TestSqlTest::setup()   A

Complexity

Conditions 2
Paths 2

Size

Total Lines 4
Code Lines 2

Duplication

Lines 0
Ratio 0 %

Importance

Changes 0
Metric Value
cc 2
eloc 2
nc 2
nop 0
dl 0
loc 4
rs 10
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
class TestSqlTest extends \SimpleSAML\Test\Utils\ClearStateTestCase
12
{
13
    // This whole class relies on \SimpleSAML_Configuration::setPreLoadedConfig(), which is not available until after 1.15.4
14
    public function setup()
15
    {
16
        if (!method_exists('SimpleSAML_Configuration', 'setPreLoadedConfig')) {
17
            $this->markTestSkipped('Precondition not met');
18
        }
19
     
20
    }
21
22
    public function testSqlSuccess()
23
    {
24
        $globalConfig_input = [
25
            'store.type' => 'sql',
26
            'store.sql.dsn' => 'sqlite:/modules/monitor/tests/files/test.sqlite',
27
            'store.sql.username' => 'test',
28
            'store.sql.password' => 'test',
29
            'store.sql.options' => null,
30
            'store.sql.prefix' => 'test'
31
        ];
32
33
        $globalConfig = \SimpleSAML_Configuration::loadFromArray($globalConfig_input);
34
        \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

34
        \SimpleSAML_Configuration::/** @scrutinizer ignore-call */ 
35
                                   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...
35
36
        $testResult = $test->getTestResult();
0 ignored issues
show
Comprehensibility Best Practice introduced by
The variable $test seems to be never defined.
Loading history...
37
        $this->assertEquals(State::OK, $testResult->getState());
38
    }
39
40
    public function testSqlFailure()
41
    {
42
        $globalConfig_input = [
43
            'store.type' => 'sql',
44
            'store.sql.dsn' => '',
45
            'store.sql.username' => '',
46
            'store.sql.password' => '',
47
            'store.sql.options' => null,
48
            'store.sql.prefix' => 'test'
49
        ];
50
51
        $globalConfig = \SimpleSAML_Configuration::loadFromArray($globalConfig_input);
52
        \SimpleSAML_Configuration::setPreLoadedConfig($globalConfig, 'config.php');
53
54
        $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

54
        $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...
55
        $testResult = $test->getTestResult();
56
        $this->assertEquals(State::FATAL, $testResult->getState());
57
    }
58
}
59