Adds a test and the corresponding bugfix.

src/Models/Scene.php

@@ -202,11 +202,17 @@ public function hasConnectionGroup(string $name): bool
     /**
      * Returns a connection group entity associated with this scene by a given name.
      * @param string $name
-     * @return SceneConnectionGroup
+     * @return SceneConnectionGroup|null
      */
-    public function getConnectionGroup(string $name): SceneConnectionGroup
+    public function getConnectionGroup(string $name): ?SceneConnectionGroup
     {
-        return $this->filterConnectionGroupCollectionByName($name)->first();
+        $filtered = $this->filterConnectionGroupCollectionByName($name)->first();
+
+        if (!$filtered) {
+            return null;
+        } else {
+            return $filtered;
+        }
     }
 
     /**

tests/Models/SceneModelTest.php

@@ -364,4 +364,21 @@ public function testIfConnectingASceneToAnotherAlreadyConnectedSceneThrowsAnExce
         $this->expectException(ArgumentException::class);
         $scene1->getConnectionGroup("lotgd/tests/village/hidden")->connect($scene2->getConnectionGroup("lotgd/tests/forest/category"));
     }
+
+    /**
+     * @see https://github.com/lotgd/core/issues/150
+     */
+    public function testIfGetConnectionGroupReturnsNullAndDoesNotThrowATypeError()
+    {
+        $scene1 = $this->getEntityManager()->getRepository(Scene::class)->find("30000000-0000-0000-0000-000000000001");
+
+        # Positively assert that this usually works
+        $connectionGroup = $scene1->getConnectionGroup("lotgd/tests/village/empty");
+        $this->assertNotNull($connectionGroup);
+        $this->assertInstanceOf(SceneConnectionGroup::class, $connectionGroup);
+
+        # Assert that connectionGroup is null if it does not exist.
+        $connectionGroup = $scene1->getConnectionGroup("this-does-not-exist");
+        $this->assertNull($connectionGroup);
+    }
 }