Use parameters array instead of inline query param (#546)

Author: acrobat

lib/Github/Api/User.php

@@ -40,7 +40,7 @@ public function all($id = null)
             return $this->get('/users');
         }
 
-        return $this->get('/users?since=' . rawurldecode($id));
+        return $this->get('/users', ['since' => rawurldecode($id)]);
     }
 
     /**

test/Github/Tests/Api/UserTest.php

@@ -82,6 +82,25 @@ public function shouldGetAllUsers()
         $this->assertEquals($expectedArray, $api->all());
     }
 
+    /**
+     * @test
+     */
+    public function shouldGetAllUsersSince()
+    {
+        $expectedArray = array(
+            array('id' => 3, 'username' => 'test3'),
+            array('id' => 4, 'username' => 'test4')
+        );
+
+        $api = $this->getApiMock();
+        $api->expects($this->once())
+            ->method('get')
+            ->with('/users', ['since' => 2])
+            ->will($this->returnValue($expectedArray));
+
+        $this->assertEquals($expectedArray, $api->all(2));
+    }
+
     /**
      * @test
      */