fix test

Author: pmattmann

Diff for: api/src/Entity/ContentNode.php

@@ -194,7 +194,7 @@ public function setParent(?ContentNode $parent) {
     }
 
     public function getCamp(): ?Camp {
-        if ($this->getRoot()->campRootContentNodes->count() > 0) {
+        if ($this->getRoot()->campRootContentNodes != null && $this->getRoot()->campRootContentNodes->count() > 0) {
             return $this->getRoot()->campRootContentNodes[0]->camp;
         }
 

Diff for: api/tests/Api/MaterialItems/CreateMaterialItemTest.php

@@ -159,10 +159,18 @@ public function testCreateMaterialItemWithPeriodFromQueryParameter() {
     public function testCreateMaterialItemValidatesMissingPeriodAndMaterialNode() {
         static::createClientWithCredentials()->request('POST', '/material_items', ['json' => $this->getExampleWritePayload([], ['period', 'materialNode'])]);
 
-        $this->assertResponseStatusCodeSame(403);
+        $this->assertResponseStatusCodeSame(422);
         $this->assertJsonContains([
-            'title' => 'An error occurred',
-            'detail' => 'Access Denied.',
+            'violations' => [
+                [
+                    'propertyPath' => 'period',
+                    'message' => 'Either this value or materialNode should not be null.',
+                ],
+                [
+                    'propertyPath' => 'materialNode',
+                    'message' => 'Either this value or period should not be null.',
+                ],
+            ],
         ]);
     }
 

Diff for: api/tests/Api/MaterialItems/UpdateMaterialItemTest.php

@@ -201,8 +201,12 @@ public function testPatchMaterialItemValidatesMissingPeriodAndMaterialNode() {
         $this->assertJsonContains([
             'violations' => [
                 [
-                    'propertyPath' => 'materialList',
-                    'message' => 'Must belong to the same camp.',
+                    'propertyPath' => 'period',
+                    'message' => 'Either this value or materialNode should not be null.',
+                ],
+                [
+                    'propertyPath' => 'materialNode',
+                    'message' => 'Either this value or period should not be null.',
                 ],
             ],
         ]);