add section.findByParent

Seretos · Oct 25, 2018 · 9d74539 · 9d74539
1 parent ffccb8a
commit 9d74539
Show file tree

Hide file tree

Showing 2 changed files with 24 additions and 0 deletions.
diff --git a/src/api/Sections.php b/src/api/Sections.php
@@ -47,6 +47,17 @@ public function findByNameAndParent(int $projectId, int $suiteId, string $name,
         return [];
     }
 
+    public function findByParent(int $projectId, int $suiteId, int $parentId = null){
+        $allSections = $this->all($projectId,$suiteId);
+        $sections = [];
+        foreach ($allSections as $section) {
+            if ((!isset($section['parent_id']) && $parentId == null) || (isset($section['parent_id']) && $section['parent_id'] == $parentId)) {
+                $sections[] = $section;
+            }
+        }
+        return $sections;
+    }
+
     public function create(int $projectId, int $suiteId, string $name, string $description = null, int $parent_id = null)
     {
         $section = $this->connector->send_post('add_section/'.$this->encodePath($projectId),

diff --git a/tests/api/SectionsTest.php b/tests/api/SectionsTest.php
@@ -66,6 +66,19 @@ public function findByNameAndParent(){
         $this->assertSame(['id' => 2,'name' => 'section2','parent_id' => null],$this->sections->findByNameAndParent(1,2,'section2'));
     }
 
+    /**
+     * @test
+     */
+    public function findByParent(){
+        $this->mockApiConnector->expects($this->any())
+            ->method('send_get')
+            ->with('get_sections/1&suite_id=2')
+            ->will($this->returnValue([['id' => 1,'name' => 'section1'],['id' => 2,'name' => 'section2','parent_id' => null],['id' => 3,'name' => 'section3','parent_id' => 1],['id' => 4,'name' => 'section4','parent_id' => 1]]));
+
+        $this->assertSame([['id' => 3,'name' => 'section3','parent_id' => 1],['id' => 4,'name' => 'section4','parent_id' => 1]],$this->sections->findByParent(1,2,1));
+        $this->assertSame([['id' => 1,'name' => 'section1'],['id' => 2,'name' => 'section2','parent_id' => null]],$this->sections->findByParent(1,2));
+    }
+
     /**
      * @test
      */