Fix bug in Scene constructor

This commit is contained in:
Austen McDonald
2016-08-30 20:40:09 +00:00
parent 019a248333
commit c585dcb33a
2 changed files with 23 additions and 18 deletions
+1 -1
View File
@@ -55,7 +55,7 @@ class Scene implements CreateableInterface
*/
public function __construct()
{
$this->children = ArrayCollection();
$this->children = new ArrayCollection();
}
/**
+22 -17
View File
@@ -13,7 +13,12 @@ class SceneModelTest extends CoreModelTestCase
{
/** @var string default data set */
protected $dataset = "scene";
public function testCreate()
{
$scene = new Scene();
}
/**
* Test getter methods
*/
@@ -21,71 +26,71 @@ class SceneModelTest extends CoreModelTestCase
{
$em = $this->getEntityManager();
$scene = $em->getRepository(Scene::class)->find(2);
$this->assertEquals("The Forest", $scene->getTitle());
$this->assertEquals("This is a very dangerous and dark forest", $scene->getDescription());
$this->assertInstanceOf(Scene::class, $scene->getParent());
$this->assertEquals(true, $scene->hasParent());
$this->assertEquals(false, $scene->hasChildren());
$em->flush();
}
/**
* Test if parent<=>child relationship is working.
*/
public function testChildParentRelationships()
{
$em = $this->getEntityManager();
$parentScene = $em->getRepository(Scene::class)->find(1);
$childScene = $em->getRepository(Scene::class)->find(2);
$this->assertEquals($parentScene, $childScene->getParent());
$this->assertContains($childScene, $parentScene->getChildren());
$em->flush();
}
/**
* Test if the scene can be removed.
*/
public function testMoveScene()
{
$em = $this->getEntityManager();
$parentScene1 = $em->getRepository(Scene::class)->find(1);
$parentScene2 = $em->getRepository(Scene::class)->find(4);
$orphanScene = $em->getRepository(scene::class)->find(5);
$this->assertEquals(false, $orphanScene->hasParent());
$this->assertEquals(false, $orphanScene->hasChildren());
// Assign orphanScene to parentScene1 and check relationships
$orphanScene->setParent($parentScene1);
$this->assertEquals(true, $orphanScene->hasParent());
$this->assertCount(3, $parentScene1->getChildren());
$this->assertEquals($parentScene1, $orphanScene->getParent());
$this->assertContains($orphanScene, $parentScene1->getChildren());
// Move the scene now to parentScene2 and check relationships
$orphanScene->setParent($parentScene2);
$this->assertCount(2, $parentScene1->getChildren());
$this->assertCount(1, $parentScene2->getChildren());
$this->assertEquals($parentScene2, $orphanScene->getParent());
$this->assertContains($orphanScene, $parentScene2->getChildren());
// Make an orphan out of it again
$orphanScene->setParent(null);
$this->assertCount(2, $parentScene1->getChildren());
$this->assertCount(0, $parentScene2->getChildren());
$this->assertEquals(false, $orphanScene->hasParent());
$this->assertNotContains($orphanScene, $parentScene1->getChildren());
$this->assertNotContains($orphanScene, $parentScene2->getChildren());
$em->flush();
}
}