HybridSessionAbstractTest

Complexity

Total Complexity

6

Size/Duplication

Total Lines	105
Duplicated Lines	16.19 %

Coupling/Cohesion

Components	2
Dependencies	5

6 Methods

Rating	Name	Duplication	Size	Complexity
	getStore()	0	1	?
A	setUp()	0	18	2
A	tearDown()	0	6	1
A	testStoreLargeData()	17	17	1
B	testStoreData()	0	26	1
A	testExpiry()	0	17	1

<?php

abstract class HybridSessionAbstractTest extends SapphireTest {

PSR1 recommends that each class must be in a namespace of at least one level to avoid collisions.

	public function setUp() {
		parent::setUp();

		HybridSessionAbstractTest_TestCookieBackend::$override_headers_sent = false;

The property $override_headers_sent was declared of type string, but false is of type false. Maybe add a type cast?

		Injector::nest();
		Injector::inst()->registerService(
			new HybridSessionAbstractTest_TestCookieBackend(),

new \HybridSessionAbstractTest_TestCookieBackend() is of type object<HybridSessionAbstractTest_TestCookieBackend>, but the function expects a object<stdClass>.

			'HybridSessionStore_Cookie'
		);

		SS_Datetime::set_mock_now('2010-03-15 12:00:00');

		if(get_class() === get_class($this)) {
			$this->markTestSkipped("Skipping abstract test");

The method markTestSkipped() does not seem to exist on object<HybridSessionAbstractTest>.

			$this->skipTest = true;
		}
	}

	public function tearDown() {
		Injector::unnest();
		SS_Datetime::clear_mock_now();

		parent::tearDown();
	}

	/**
	 * @return HybridSessionStore_Base
	 */
	abstract protected function getStore();

	/**
	 * Test how this store handles large volumes of data (>1000 characters)
	 */
	public function testStoreLargeData() {

This method seems to be duplicated in your project.

		$session = uniqid();
		$store = $this->getStore();

		// Test new session is blank
		$result = $store->read($session);
		$this->assertEmpty($result);

The method assertEmpty() does not seem to exist on object<HybridSessionAbstractTest>.

		// Save data against session
		$data1 = array(
			'Large' => str_repeat('A', 600),
			'Content' => str_repeat('B', 600)
		);
		$store->write($session, serialize($data1));
		$result = $store->read($session);
		$this->assertEquals($data1, unserialize($result));

The method assertEquals() does not seem to exist on object<HybridSessionAbstractTest>.

	}

	/**
	 * Test storage of data
	 */
	public function testStoreData() {
		$session = uniqid();
		$store = $this->getStore();

		// Test new session is blank
		$result = $store->read($session);
		$this->assertEmpty($result);

The method assertEmpty() does not seem to exist on object<HybridSessionAbstractTest>.

		// Save data against session
		$data1 = array(
			'Color' => 'red',
			'Animal' => 'elephant'
		);
		$store->write($session, serialize($data1));
		$result = $store->read($session);
		$this->assertEquals($data1, unserialize($result));

The method assertEquals() does not seem to exist on object<HybridSessionAbstractTest>.

		// Save larger data
		$data2 = array(
			'Color' => 'blue',
			'Animal' => str_repeat('bat', 100)
		);
		$store->write($session, serialize($data2));
		$result = $store->read($session);
		$this->assertEquals($data2, unserialize($result));

The method assertEquals() does not seem to exist on object<HybridSessionAbstractTest>.

	}

	/**
	 * Test expiry of data
	 */
	public function testExpiry() {
		$session1 = uniqid();
		$store = $this->getStore();

		// Store data now
		$data1 = array(
			'Food' => 'Pizza'
		);
		$store->write($session1, serialize($data1));
		$result1 = $store->read($session1);
		$this->assertEquals($data1, unserialize($result1));

The method assertEquals() does not seem to exist on object<HybridSessionAbstractTest>.

		// Go to the future and test that the expiry is accurate
		SS_Datetime::set_mock_now('2040-03-16 12:00:00');
		$result2 = $store->read($session1);
		$this->assertEmpty($result2);

The method assertEmpty() does not seem to exist on object<HybridSessionAbstractTest>.

	}
}

class HybridSessionAbstractTest_TestCookieBackend extends HybridSessionStore_Cookie {

PSR1 recommends that each class should be in its own file to aid autoloaders.

PSR1 recommends that each class must be in a namespace of at least one level to avoid collisions.

	/**
	 * Override value of 'headers_sent' but only if running tests.
	 *
	 * Set to true or false, or null to not override
	 *
	 * @var string
	 */
	public static $override_headers_sent = null;

	protected function canWrite() {
		if(self::$override_headers_sent !== null) {
			return !self::$override_headers_sent;
		}
		parent::canWrite();
	}
}