Merge pull request #7 from aternosorg/fix-tests

pavog · web-flow · commit 9a0e0604f928 · 2025-11-11T16:47:42.000+01:00
Fix tests
diff --git a/test/Client/ClientTest.php b/test/Client/ClientTest.php
@@ -75,7 +75,6 @@ protected function assertValidProject($project): void
         $this->assertNotNull($project->getData()->getLastUpdated());
         $this->assertNotNull($project->getData()->getVisibility());
         $this->assertNotNull($project->getData()->getDescription());
-        $this->assertNotNull($project->getData()->getUserActions());
         $this->assertNotNull($project->getData()->getSettings());
     }
 
@@ -354,6 +353,8 @@ public function testGetDailyProjectStats()
     {
         if (!getenv("HANGAR_API_KEY")) {
             $this->markTestSkipped("This test requires authentication.");
+            // no API key provided will result in a 403 Forbidden error:
+            // You need the is_subject_member permission to view project statistics
         }
 
         $project = $this->apiClient->getProject("mclogs");

Original file line number	Diff line number	Diff line change
`@@ -75,7 +75,6 @@ protected function assertValidProject($project): void`
`75`	`75`	`$this->assertNotNull($project->getData()->getLastUpdated());`
`76`	`76`	`$this->assertNotNull($project->getData()->getVisibility());`
`77`	`77`	`$this->assertNotNull($project->getData()->getDescription());`
`78`		`- $this->assertNotNull($project->getData()->getUserActions());`
`79`	`78`	`$this->assertNotNull($project->getData()->getSettings());`
`80`	`79`	`}`
`81`	`80`
`@@ -354,6 +353,8 @@ public function testGetDailyProjectStats()`
`354`	`353`	`{`
`355`	`354`	`if (!getenv("HANGAR_API_KEY")) {`
`356`	`355`	`$this->markTestSkipped("This test requires authentication.");`
	`356`	`+ // no API key provided will result in a 403 Forbidden error:`
	`357`	`+ // You need the is_subject_member permission to view project statistics`
`357`	`358`	`}`
`358`	`359`
`359`	`360`	`$project = $this->apiClient->getProject("mclogs");`