Merge pull request #22 from localheinz/fix/participation

Fix: Participation vs. partecipation
EvanDotPro · Mar 6, 2015 · 6fdb8f2 · 6fdb8f2
2 parents da9c895 + c74af1e
commit 6fdb8f2
Show file tree

Hide file tree

Showing 2 changed files with 25 additions and 3 deletions.
diff --git a/src/EdpGithub/Api/Repos.php b/src/EdpGithub/Api/Repos.php
@@ -78,14 +78,25 @@ public function codeFrequency($owner, $repo)
     }
 
     /**
-     * Get partecipation
+     * @param  string $owner
+     * @param  string $repo
+     * @return string
+     */
+    public function participation($owner, $repo)
+    {
+        return $this->get('repos/' . $owner . '/' . $repo . '/stats/participation');
+    }
+
+    /**
+     * @deprecated
+     *
      * @param  string $owner
      * @param  string $repo
      * @return string
      */
     public function partecipation($owner, $repo)
     {
-        return $this->get('repos/' . $owner . '/' . $repo . '/stats/partecipation');
+        return $this->participation($owner, $repo);
     }
 
     /**

diff --git a/test/EdpGithubTest/Api/ReposTest.php b/test/EdpGithubTest/Api/ReposTest.php
@@ -73,10 +73,21 @@ public function testCodeFrequency()
         $this->assertEquals($result, $expectedArray);
     }
 
+    public function testParticipation()
+    {
+        $expectedArray = array('id' => 1, 'name' => 'repo');
+        $client = $this->getClientMock('repos/owner/repo/stats/participation', $expectedArray);
+        $api = new Repos();
+        $api->setClient($client);
+        $result = $api->participation('owner', 'repo');
+
+        $this->assertEquals($result, $expectedArray);
+    }
+
     public function testPartecipation()
     {
         $expectedArray = array('id' => 1, 'name' => 'repo');
-        $client = $this->getClientMock('repos/owner/repo/stats/partecipation', $expectedArray);
+        $client = $this->getClientMock('repos/owner/repo/stats/participation', $expectedArray);
         $api = new Repos();
         $api->setClient($client);
         $result = $api->partecipation('owner', 'repo');