go/go-jira
1
0
Fork 0
mirror of https://github.com/interviewstreet/go-jira.git synced 2025-04-21 12:07:01 +02:00
Code Issues Releases Activity

Merge branch 'manute-master'

Browse Source 
* manute-master:
  Applied NewJiraError for user api endpoints
  finding users by email, username or name
This commit is contained in:
Andy Grunwald 2017-12-10 19:56:36 +01:00
commit 451fade708
2 changed files with 48 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

29
user.go
View File

@ -46,7 +46,7 @@ func (s *UserService) Get(username string) (*User, *Response, error) {
user := new(User) user := new(User)
resp, err := s.client.Do(req, user) resp, err := s.client.Do(req, user)
if err != nil { if err != nil {
return nil, resp, err return nil, resp, NewJiraError(resp, err)
} }
return user, resp, nil return user, resp, nil
} }
@ -70,11 +70,13 @@ func (s *UserService) Create(user *User) (*User, *Response, error) {
defer resp.Body.Close() defer resp.Body.Close()
data, err := ioutil.ReadAll(resp.Body) data, err := ioutil.ReadAll(resp.Body)
if err != nil { if err != nil {
return nil, resp, fmt.Errorf("Could not read the returned data") e := fmt.Errorf("Could not read the returned data")
return nil, resp, NewJiraError(resp, e)
} }
err = json.Unmarshal(data, responseUser) err = json.Unmarshal(data, responseUser)
if err != nil { if err != nil {
return nil, resp, fmt.Errorf("Could not unmarshall the data into struct") e := fmt.Errorf("Could not unmarshall the data into struct")
return nil, resp, NewJiraError(resp, e)
} }
return responseUser, resp, nil return responseUser, resp, nil
} }
@ -92,7 +94,26 @@ func (s *UserService) GetGroups(username string) (*[]UserGroup, *Response, error
userGroups := new([]UserGroup) userGroups := new([]UserGroup)
resp, err := s.client.Do(req, userGroups) resp, err := s.client.Do(req, userGroups)
if err != nil { if err != nil {
return nil, resp, err return nil, resp, NewJiraError(resp, err)
} }
return userGroups, resp, nil return userGroups, resp, nil
} }
// Finds user info from JIRA:
// It can find users by email, username or name
//
// JIRA API docs: https://docs.atlassian.com/jira/REST/cloud/#api/2/user-findUsers
func (s *UserService) Find(property string) ([]User, *Response, error) {
apiEndpoint := fmt.Sprintf("/rest/api/2/user/search?username=%s", property)
req, err := s.client.NewRequest("GET", apiEndpoint, nil)
if err != nil {
return nil, nil, err
}
users := []User{}
resp, err := s.client.Do(req, &users)
if err != nil {
return nil, resp, NewJiraError(resp, err)
}
return users, resp, nil
}

23
user_test.go
View File

@ -73,3 +73,26 @@ func TestUserService_GetGroups(t *testing.T) {
t.Error("Expected user groups. []UserGroup is nil") t.Error("Expected user groups. []UserGroup is nil")
} }
} }
func TestUserService_Find_Success(t *testing.T) {
setup()
defer teardown()
testMux.HandleFunc("/rest/api/2/user/search", func(w http.ResponseWriter, r *http.Request) {
testMethod(t, r, "GET")
testRequestURL(t, r, "/rest/api/2/user/search?username=fred@example.com")
fmt.Fprint(w, `[{"self":"http://www.example.com/jira/rest/api/2/user?username=fred","key":"fred",
"name":"fred","emailAddress":"fred@example.com","avatarUrls":{"48x48":"http://www.example.com/jira/secure/useravatar?size=large&ownerId=fred",
"24x24":"http://www.example.com/jira/secure/useravatar?size=small&ownerId=fred","16x16":"http://www.example.com/jira/secure/useravatar?size=xsmall&ownerId=fred",
"32x32":"http://www.example.com/jira/secure/useravatar?size=medium&ownerId=fred"},"displayName":"Fred F. User","active":true,"timeZone":"Australia/Sydney","groups":{"size":3,"items":[
{"name":"jira-user","self":"http://www.example.com/jira/rest/api/2/group?groupname=jira-user"},{"name":"jira-admin",
"self":"http://www.example.com/jira/rest/api/2/group?groupname=jira-admin"},{"name":"important","self":"http://www.example.com/jira/rest/api/2/group?groupname=important"
}]},"applicationRoles":{"size":1,"items":[]},"expand":"groups,applicationRoles"}]`)
})
if user, _, err := testClient.User.Find("fred@example.com"); err != nil {
t.Errorf("Error given: %s", err)
} else if user == nil {
t.Error("Expected user. User is nil")
}
}