Fix api repositories compare method (#597)

Co-authored-by: Paulius Petronis <[email protected]>

Author: pauliuspetronis

src/Api/Repositories.php

@@ -386,10 +386,13 @@ public function postCommitBuildStatus($project_id, string $sha, string $state, a
      */
     public function compare($project_id, string $fromShaOrMaster, string $toShaOrMaster, bool $straight = false)
     {
-        return $this->get($this->getProjectPath(
-            $project_id,
-            'repository/compare?from='.self::encodePath($fromShaOrMaster).'&to='.self::encodePath($toShaOrMaster).'&straight='.self::encodePath($straight ? 'true' : 'false')
-        ));
+        $params = [
+            'from' => self::encodePath($fromShaOrMaster),
+            'to' => self::encodePath($toShaOrMaster),
+            'straight' => self::encodePath($straight ? 'true' : 'false'),
+        ];
+
+        return $this->get($this->getProjectPath($project_id, 'repository/compare'), $params);
     }
 
     /**

tests/Api/RepositoriesTest.php

@@ -490,7 +490,7 @@ public function shouldCompareStraight(): void
         $api = $this->getApiMock();
         $api->expects($this->once())
             ->method('get')
-            ->with('projects/1/repository/compare?from=master&to=feature&straight=true')
+            ->with('projects/1/repository/compare', ['from' => 'master', 'to' => 'feature', 'straight' => 'true'])
             ->will($this->returnValue($expectedArray))
         ;
 
@@ -507,7 +507,7 @@ public function shouldNotCompareStraight(): void
         $api = $this->getApiMock();
         $api->expects($this->once())
             ->method('get')
-            ->with('projects/1/repository/compare?from=master&to=feature&straight=false')
+            ->with('projects/1/repository/compare', ['from' => 'master', 'to' => 'feature', 'straight' => 'false'])
             ->will($this->returnValue($expectedArray))
         ;