Add support of approved_by_ids in MergeRequests API (#600)

Co-authored-by: Valentinas Bartusevičius <[email protected]>

Author: vbartusevicius

src/Api/MergeRequests.php

@@ -115,6 +115,12 @@ public function all($project_id = null, array $parameters = [])
         $resolver->setDefined('with_merge_status_recheck')
             ->setAllowedTypes('with_merge_status_recheck', 'bool')
         ;
+        $resolver->setDefined('approved_by_ids')
+            ->setAllowedTypes('approved_by_ids', 'array')
+            ->setAllowedValues('approved_by_ids', function (array $value) {
+                return \count($value) === \count(\array_filter($value, 'is_int'));
+            })
+        ;
 
         $path = null === $project_id ? 'merge_requests' : $this->getProjectPath($project_id, 'merge_requests');
 

tests/Api/MergeRequestsTest.php

@@ -66,6 +66,7 @@ public function shouldGetAllWithParams(): void
                 'source_branch' => 'develop',
                 'target_branch' => 'master',
                 'with_merge_status_recheck' => true,
+                'approved_by_ids' => [1],
             ])
             ->will($this->returnValue($expectedArray))
         ;
@@ -84,6 +85,7 @@ public function shouldGetAllWithParams(): void
             'source_branch' => 'develop',
             'target_branch' => 'master',
             'with_merge_status_recheck' => true,
+            'approved_by_ids' => [1],
         ]));
     }