Add more unit tests
This commit is contained in:
@@ -116,10 +116,10 @@ class GithubAuth extends Base implements OAuthAuthenticationProviderInterface
|
||||
/**
|
||||
* Get Github profile
|
||||
*
|
||||
* @access private
|
||||
* @access public
|
||||
* @return array
|
||||
*/
|
||||
private function getProfile()
|
||||
public function getProfile()
|
||||
{
|
||||
$this->getService()->getAccessToken($this->code);
|
||||
|
||||
|
||||
@@ -116,10 +116,10 @@ class GitlabAuth extends Base implements OAuthAuthenticationProviderInterface
|
||||
/**
|
||||
* Get Gitlab profile
|
||||
*
|
||||
* @access private
|
||||
* @access public
|
||||
* @return array
|
||||
*/
|
||||
private function getProfile()
|
||||
public function getProfile()
|
||||
{
|
||||
$this->getService()->getAccessToken($this->code);
|
||||
|
||||
|
||||
@@ -116,10 +116,10 @@ class GoogleAuth extends Base implements OAuthAuthenticationProviderInterface
|
||||
/**
|
||||
* Get Google profile
|
||||
*
|
||||
* @access private
|
||||
* @access public
|
||||
* @return array
|
||||
*/
|
||||
private function getProfile()
|
||||
public function getProfile()
|
||||
{
|
||||
$this->getService()->getAccessToken($this->code);
|
||||
|
||||
|
||||
@@ -66,6 +66,6 @@ class GroupManager
|
||||
}
|
||||
}
|
||||
|
||||
return $result;
|
||||
return array_values($result);
|
||||
}
|
||||
}
|
||||
|
||||
@@ -39,7 +39,7 @@ class DatabaseGroupProvider implements GroupProviderInterface
|
||||
*/
|
||||
public function getInternalId()
|
||||
{
|
||||
return $this->group['id'];
|
||||
return (int) $this->group['id'];
|
||||
}
|
||||
|
||||
/**
|
||||
|
||||
@@ -75,7 +75,7 @@ class Group extends Base
|
||||
*/
|
||||
public function search($input)
|
||||
{
|
||||
return $this->db->table(self::TABLE)->ilike('name', '%'.$input.'%')->findAll();
|
||||
return $this->db->table(self::TABLE)->ilike('name', '%'.$input.'%')->asc('name')->findAll();
|
||||
}
|
||||
|
||||
/**
|
||||
|
||||
Reference in New Issue
Block a user