StaticPagesQueueEventTest::tearDown() - Code Metrics - Inspection of "Added standard Scrutinizer config" - silverstripe/silverstripe-staticpublishqueue - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Pull Request — master (#53)

by Helpful

created 2016-03-22 06:41 UTC

StaticPagesQueueEventTest::tearDown() A

↳ Parent: StaticPagesQueueEventTest

Complexity

Conditions	1
Paths	1

Size

Total Lines	4
Code Lines	3

Duplication

Lines	0
Ratio	0 %

Metric	Value
dl	0
loc	4
rs	10
cc	1
eloc	3
nc	1
nop	0

<?php

/**
 * This is a Unittest class for EventTest
 * 
 */
class StaticPagesQueueEventTest extends SapphireTest {
namespace YourVendor;

class YourClass { }

	public function setUp() {
		parent::setUp();
		StaticPagesQueueEvent::clear();
		Config::inst()->nest();
		Config::inst()->update('StaticPagesQueue', 'realtime', true);
	}

	public function tearDown() {
		Config::inst()->unnest();
		parent::tearDown();
	}

	public function testRegisterEventWithNonExistingInterface() {
		$this->setExpectedException('StaticPagesQueueEvent_Exception');
		StaticPagesQueueEvent::register_event('EventTestClassImplementator', 'apa');
	}
	
	public function testRegisterEventWithNonExistingEventhandler() {
		$this->setExpectedException('StaticPagesQueueEvent_Exception');
		StaticPagesQueueEvent::register_event('apa', 'TestingEventListener');
	}
	
	public function testRegisterEventNoImplementators() {
		$this->setExpectedException('StaticPagesQueueEvent_Exception');
		StaticPagesQueueEvent::register_event('EventTestClass', 'TestingEventListenerMissing');
	}
	
	public function testRegisterEvent() {
		 StaticPagesQueueEvent::register_event('EventTestClass', 'TestingEventListener');
	}
	
	public function testFireEventNoEventClassRegistered() {
		StaticPagesQueueEvent::trigger_events_during_testing(true);
		$this->setExpectedException('StaticPagesQueueEvent_Exception');
		StaticPagesQueueEvent::fire_event(new EventTestClass());
	}
	
	public function testFireEvent() {
		StaticPagesQueueEvent::trigger_events_during_testing(true);
		StaticPagesQueueEvent::register_event('EventTestClass', 'TestingEventListener');
		$this->assertTrue(StaticPagesQueueEvent::fire_event(new EventTestClass()));
		
	}
	
	public function testTriggerEventsDuringTesting() {
		StaticPagesQueueEvent::trigger_events_during_testing(false);
		StaticPagesQueueEvent::register_event('EventTestClass', 'TestingEventListener');
		$this->assertFalse(StaticPagesQueueEvent::fire_event(new EventTestClass()));
	}
}

interface TestingEventListener extends TestOnly {}
namespace YourVendor;

class YourClass { }

class EventTestClassImplementator implements TestingEventListener {
namespace YourVendor;

class YourClass { }
	public function testingEvent() {
		return true;
	}
}
class EventTestClass extends StaticPagesQueueEvent implements TestOnly{
namespace YourVendor;

class YourClass { }
}

interface TestingEventListenerMissing extends TestOnly {}
namespace YourVendor;

class YourClass { }


1			<?php
2
3			/**
4			* This is a Unittest class for EventTest
5			*
6			*/
7			class StaticPagesQueueEventTest extends SapphireTest {
			0 ignored issues – show Coding Style Compatibility introduced 2016-03-22 06:43 UTC by Report Bug Copy Issue Report PSR1 recommends that each class must be in a namespace of at least one level to avoid collisions. You can fix this by adding a namespace to your class: namespace YourVendor; class YourClass { } When choosing a vendor namespace, try to pick something that is not too generic to avoid conflicts with other libraries. Loading history...
8
9			public function setUp() {
10			parent::setUp();
11			StaticPagesQueueEvent::clear();
12			Config::inst()->nest();
13			Config::inst()->update('StaticPagesQueue', 'realtime', true);
14			}
15
16			public function tearDown() {
17			Config::inst()->unnest();
18			parent::tearDown();
19			}
20
21			public function testRegisterEventWithNonExistingInterface() {
22			$this->setExpectedException('StaticPagesQueueEvent_Exception');
23			StaticPagesQueueEvent::register_event('EventTestClassImplementator', 'apa');
24			}
25
26			public function testRegisterEventWithNonExistingEventhandler() {
27			$this->setExpectedException('StaticPagesQueueEvent_Exception');
28			StaticPagesQueueEvent::register_event('apa', 'TestingEventListener');
29			}
30
31			public function testRegisterEventNoImplementators() {
32			$this->setExpectedException('StaticPagesQueueEvent_Exception');
33			StaticPagesQueueEvent::register_event('EventTestClass', 'TestingEventListenerMissing');
34			}
35
36			public function testRegisterEvent() {
37			StaticPagesQueueEvent::register_event('EventTestClass', 'TestingEventListener');
38			}
39
40			public function testFireEventNoEventClassRegistered() {
41			StaticPagesQueueEvent::trigger_events_during_testing(true);
42			$this->setExpectedException('StaticPagesQueueEvent_Exception');
43			StaticPagesQueueEvent::fire_event(new EventTestClass());
44			}
45
46			public function testFireEvent() {
47			StaticPagesQueueEvent::trigger_events_during_testing(true);
48			StaticPagesQueueEvent::register_event('EventTestClass', 'TestingEventListener');
49			$this->assertTrue(StaticPagesQueueEvent::fire_event(new EventTestClass()));
50
51			}
52
53			public function testTriggerEventsDuringTesting() {
54			StaticPagesQueueEvent::trigger_events_during_testing(false);
55			StaticPagesQueueEvent::register_event('EventTestClass', 'TestingEventListener');
56			$this->assertFalse(StaticPagesQueueEvent::fire_event(new EventTestClass()));
57			}
58			}
59
60			interface TestingEventListener extends TestOnly {}
			0 ignored issues – show Coding Style Compatibility introduced 2016-03-22 06:43 UTC by Report Bug Copy Issue Report PSR1 recommends that each class should be in its own file to aid autoloaders. Having each class in a dedicated file usually plays nice with PSR autoloaders and is therefore a well established practice. If you use other autoloaders, you might not want to follow this rule. Loading history... Coding Style Compatibility introduced 2016-03-22 06:43 UTC by Report Bug Copy Issue Report Each interface must be in a namespace of at least one level (a top-level vendor name) You can fix this by adding a namespace to your class: namespace YourVendor; class YourClass { } When choosing a vendor namespace, try to pick something that is not too generic to avoid conflicts with other libraries. Loading history...
61
62			class EventTestClassImplementator implements TestingEventListener {
			0 ignored issues – show Coding Style Compatibility introduced 2016-03-22 06:43 UTC by Report Bug Copy Issue Report Each interface must be in a file by itself Having each class in a dedicated file usually plays nice with PSR autoloaders and is therefore a well established practice. If you use other autoloaders, you might not want to follow this rule. Loading history... Coding Style Compatibility introduced 2016-03-22 06:43 UTC by Report Bug Copy Issue Report PSR1 recommends that each class must be in a namespace of at least one level to avoid collisions. You can fix this by adding a namespace to your class: namespace YourVendor; class YourClass { } When choosing a vendor namespace, try to pick something that is not too generic to avoid conflicts with other libraries. Loading history...
63			public function testingEvent() {
64			return true;
65			}
66			}
67			class EventTestClass extends StaticPagesQueueEvent implements TestOnly{
			0 ignored issues – show Coding Style Compatibility introduced 2016-03-22 06:43 UTC by Report Bug Copy Issue Report PSR1 recommends that each class should be in its own file to aid autoloaders. Having each class in a dedicated file usually plays nice with PSR autoloaders and is therefore a well established practice. If you use other autoloaders, you might not want to follow this rule. Loading history... Coding Style Compatibility introduced 2016-03-22 06:43 UTC by Report Bug Copy Issue Report PSR1 recommends that each class must be in a namespace of at least one level to avoid collisions. You can fix this by adding a namespace to your class: namespace YourVendor; class YourClass { } When choosing a vendor namespace, try to pick something that is not too generic to avoid conflicts with other libraries. Loading history...
68			}
69
70			interface TestingEventListenerMissing extends TestOnly {}
			0 ignored issues – show Coding Style Compatibility introduced 2016-03-22 06:43 UTC by Report Bug Copy Issue Report PSR1 recommends that each class should be in its own file to aid autoloaders. Having each class in a dedicated file usually plays nice with PSR autoloaders and is therefore a well established practice. If you use other autoloaders, you might not want to follow this rule. Loading history... Coding Style Compatibility introduced 2016-03-22 06:43 UTC by Report Bug Copy Issue Report Each interface must be in a namespace of at least one level (a top-level vendor name) You can fix this by adding a namespace to your class: namespace YourVendor; class YourClass { } When choosing a vendor namespace, try to pick something that is not too generic to avoid conflicts with other libraries. Loading history...
71

silverstripe / silverstripe-staticpublishqueue

Pull Request — master (#53)

StaticPagesQueueEventTest::tearDown() A

Complexity

Size

Duplication

Duplication Side-by-Side

Filter issues like