SessionCheckTest::testSessionSet() - Code Metrics - Inspection of "Merge pull request #60 from silverstripe-terraform..." - silverstripe/silverstripe-environmentcheck - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 128ddb...c5783a )

unknown

created 2019-03-19 22:51 UTC

SessionCheckTest::testSessionSet() A

↳ Parent: SessionCheckTest

Complexity

Conditions	1
Paths	1

Size

Total Lines	17
Code Lines	8

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
cc	1
eloc	8
nc	1
nop	0
dl	0
loc	17
rs	10
c	0
b	0
f	0

<?php

namespace SilverStripe\EnvironmentCheck\Tests\Checks;

use GuzzleHttp\Client;
use GuzzleHttp\HandlerStack;
use GuzzleHttp\Psr7\Response;
use SilverStripe\Dev\SapphireTest;
use GuzzleHttp\Handler\MockHandler;
use SilverStripe\EnvironmentCheck\EnvironmentCheck;
use SilverStripe\EnvironmentCheck\Checks\SessionCheck;

/**
 * Test session checks.
 */
class SessionCheckTest extends SapphireTest
{
    /**
     * @var SilverStripe\EnvironmentCheck\Checks\SessionCheck

     */
    public $sessionCheck = null;

    /**
     * Create a session check for use by tests.
     *
     * @return void
     */
    protected function setUp()
    {
        parent::setUp();
        $this->sessionCheck = new SessionCheck('/');

    }

    /**
     * Env check reports error when session cookies are being set.
     *
     * @return void
     */
    public function testSessionSet()
    {
        // Create a mock and queue two responses.
        $mock = new MockHandler([
            new Response(200, ['Set-Cookie' => 'PHPSESSID:foo']),
            new Response(200, ['Set-Cookie' => 'SECSESSID:bar'])
        ]);

        $handler = HandlerStack::create($mock);
        $client = new Client(['handler' => $handler]);
        $this->sessionCheck->client = $client;

        // Check for PHPSESSID
        $this->assertContains(EnvironmentCheck::ERROR, $this->sessionCheck->check());

        // Check for SECSESSID
        $this->assertContains(EnvironmentCheck::ERROR, $this->sessionCheck->check());
    }

    /**
     * Env check responds OK when no session cookies are set in response.
     *
     * @return void
     */
    public function testSessionNotSet()
    {
        // Create a mock and queue two responses.
        $mock = new MockHandler([
            new Response(200)
        ]);

        $handler = HandlerStack::create($mock);
        $client = new Client(['handler' => $handler]);
        $this->sessionCheck->client = $client;

        $this->assertContains(EnvironmentCheck::OK, $this->sessionCheck->check());
    }
}


1			<?php
2
3			namespace SilverStripe\EnvironmentCheck\Tests\Checks;
4
5			use GuzzleHttp\Client;
6			use GuzzleHttp\HandlerStack;
7			use GuzzleHttp\Psr7\Response;
8			use SilverStripe\Dev\SapphireTest;
9			use GuzzleHttp\Handler\MockHandler;
10			use SilverStripe\EnvironmentCheck\EnvironmentCheck;
11			use SilverStripe\EnvironmentCheck\Checks\SessionCheck;
12
13			/**
14			* Test session checks.
15			*/
16			class SessionCheckTest extends SapphireTest
17			{
18			/**
19			* @var SilverStripe\EnvironmentCheck\Checks\SessionCheck
			0 ignored issues – show Bug introduced 2019-03-19 21:26 UTC by Report Bug Copy Issue Report The type `SilverStripe\Environment...eck\Checks\SessionCheck` was not found. Did you mean `SilverStripe\EnvironmentCheck\Checks\SessionCheck`? If so, make sure to prefix the type with `\`. Loading history...
20			*/
21			public $sessionCheck = null;
22
23			/**
24			* Create a session check for use by tests.
25			*
26			* @return void
27			*/
28			protected function setUp()
29			{
30			parent::setUp();
31			$this->sessionCheck = new SessionCheck('/');
			0 ignored issues – show Documentation Bug introduced 2019-03-19 21:26 UTC by Report Bug Copy Issue Report It seems like `new SilverStripe\Environ...hecks\SessionCheck('/')` of type `SilverStripe\EnvironmentCheck\Checks\SessionCheck` is incompatible with the declared type `SilverStripe\Environment...eck\Checks\SessionCheck` of property `$sessionCheck`. Our type inference engine has found an assignment to a property that is incompatible with the declared type of that property. Either this assignment is in error or the assigned type should be added to the documentation/type hint for that property.. Loading history...
32			}
33
34			/**
35			* Env check reports error when session cookies are being set.
36			*
37			* @return void
38			*/
39			public function testSessionSet()
40			{
41			// Create a mock and queue two responses.
42			$mock = new MockHandler([
43			new Response(200, ['Set-Cookie' => 'PHPSESSID:foo']),
44			new Response(200, ['Set-Cookie' => 'SECSESSID:bar'])
45			]);
46
47			$handler = HandlerStack::create($mock);
48			$client = new Client(['handler' => $handler]);
49			$this->sessionCheck->client = $client;
50
51			// Check for PHPSESSID
52			$this->assertContains(EnvironmentCheck::ERROR, $this->sessionCheck->check());
53
54			// Check for SECSESSID
55			$this->assertContains(EnvironmentCheck::ERROR, $this->sessionCheck->check());
56			}
57
58			/**
59			* Env check responds OK when no session cookies are set in response.
60			*
61			* @return void
62			*/
63			public function testSessionNotSet()
64			{
65			// Create a mock and queue two responses.
66			$mock = new MockHandler([
67			new Response(200)
68			]);
69
70			$handler = HandlerStack::create($mock);
71			$client = new Client(['handler' => $handler]);
72			$this->sessionCheck->client = $client;
73
74			$this->assertContains(EnvironmentCheck::OK, $this->sessionCheck->check());
75			}
76			}
77

silverstripe / silverstripe-environmentcheck

Push — master ( 128ddb...c5783a )

SessionCheckTest::testSessionSet() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like