Passed
Push — master ( c9d411...7f3120 )
by Tim
01:33
created

TestSqlTest::testBogus()   A

Complexity

Conditions 1
Paths 1

Size

Total Lines 3
Code Lines 1

Duplication

Lines 0
Ratio 0 %

Importance

Changes 0
Metric Value
cc 1
eloc 1
nc 1
nop 0
dl 0
loc 3
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
    public function testBogus()
14
    {
15
        $this->assertEquals(true, true);
16
    }
17
18
    public function testSqlSuccess()
19
    {
20
        if (!method_exists('\SimpleSAML_Configuration', 'setPreLoadedConfig')) {
21
            $this->markTestIncomplete('A precondition was not met.');
22
        }
23
        $globalConfig_input = [
24
            'store.type' => 'sql',
25
            'store.sql.dsn' => 'sqlite:/modules/monitor/tests/files/test.sqlite',
26
            'store.sql.username' => 'test',
27
            'store.sql.password' => 'test',
28
            'store.sql.options' => null,
29
            'store.sql.prefix' => 'test'
30
        ];
31
32
        $globalConfig = \SimpleSAML_Configuration::loadFromArray($globalConfig_input);
33
        \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

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

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