Skip to content

Commit

Permalink
✨ Added the v2.Project.Version.Gets method
Browse files Browse the repository at this point in the history
  • Loading branch information
@ctreminiom
ctreminiom committed Nov 8, 2021
1 parent 2ef0fbe commit e5903d1
Show file tree
Hide file tree
Showing 2 changed files with 191 additions and 10 deletions.
31 changes: 29 additions & 2 deletions jira/v2/projectVersion.go
View file
Original file line number Diff line number Diff line change
Expand Up @@ -12,10 +12,37 @@ import (

type ProjectVersionService struct{ client *Client }

// Gets returns a paginated list of all versions in a project.
// Gets returns all versions in a project.
// The response is not paginated.
// Use Get project versions paginated if you want to get the versions in a project with pagination.
// Atlassian Docs: https://developer.atlassian.com/cloud/jira/platform/rest/v2/api-group-project-versions/#api-rest-api-2-project-projectidorkey-versions-get
func (p *ProjectVersionService) Gets(ctx context.Context, projectKeyOrID string) (result []*models.VersionScheme, response *ResponseScheme, err error) {

if len(projectKeyOrID) == 0 {
return nil, nil, models.ErrNoProjectIDError
}

endpoint := fmt.Sprintf("rest/api/2/project/%v/versions", projectKeyOrID)

request, err := p.client.newRequest(ctx, http.MethodGet, endpoint, nil)
if err != nil {
return
}

request.Header.Set("Accept", "application/json")

response, err = p.client.call(request, &result)
if err != nil {
return
}

return
}

// Search returns a paginated list of all versions in a project.
// Docs: https://docs.go-atlassian.io/jira-software-cloud/projects/versions#get-project-versions-paginated
// Atlassian Docs: https://developer.atlassian.com/cloud/jira/platform/rest/v2/api-group-project-versions/#api-rest-api-2-project-projectidorkey-version-get
func (p *ProjectVersionService) Gets(ctx context.Context, projectKeyOrID string, options *models.VersionGetsOptions, startAt,
func (p *ProjectVersionService) Search(ctx context.Context, projectKeyOrID string, options *models.VersionGetsOptions, startAt,
maxResults int) (result *models.VersionPageScheme, response *ResponseScheme, err error) {

if len(projectKeyOrID) == 0 {
Expand Down
170 changes: 162 additions & 8 deletions jira/v2/projectVersion_test.go
View file
Original file line number Diff line number Diff line change
Expand Up @@ -372,6 +372,160 @@ func TestProjectVersionService_Get(t *testing.T) {

func TestProjectVersionService_Gets(t *testing.T) {

testCases := []struct {
name string
projectKeyOrID string
mockFile string
wantHTTPMethod string
endpoint string
context context.Context
wantHTTPCodeReturn int
wantErr bool
}{
{
name: "GetProjectVersionsWhenTheParamsAreCorrect",
projectKeyOrID: "10001",
mockFile: "../mocks/get-project-versions.json",
wantHTTPMethod: http.MethodGet,
endpoint: "/rest/api/2/project/10001/versions",
context: context.Background(),
wantHTTPCodeReturn: http.StatusOK,
wantErr: false,
},

{
name: "GetProjectVersionsWhenTheProjectKeyIsNotProvided",
projectKeyOrID: "",
mockFile: "../mocks/get-project-versions.json",
wantHTTPMethod: http.MethodGet,
endpoint: "/rest/api/2/project/10001/versions",
context: context.Background(),
wantHTTPCodeReturn: http.StatusOK,
wantErr: true,
},

{
name: "GetProjectVersionsWhenTheExpandsIsNil",
projectKeyOrID: "10001",
mockFile: "../mocks/get-project-versions.json",
wantHTTPMethod: http.MethodGet,
endpoint: "/rest/api/2/project/10001/versions",
context: context.Background(),
wantHTTPCodeReturn: http.StatusOK,
wantErr: false,
},

{
name: "GetProjectVersionsWhenTheRequestMethodIsIncorrect",
projectKeyOrID: "10001",
mockFile: "../mocks/get-project-versions.json",
wantHTTPMethod: http.MethodPost,
endpoint: "/rest/api/2/project/10001/versions",
context: context.Background(),
wantHTTPCodeReturn: http.StatusOK,
wantErr: true,
},

{
name: "GetProjectVersionsWhenTheStatusCodeIsIncorrect",
projectKeyOrID: "10001",
mockFile: "../mocks/get-project-versions.json",
wantHTTPMethod: http.MethodGet,
endpoint: "/rest/api/2/project/10001/versions",
context: context.Background(),
wantHTTPCodeReturn: http.StatusBadRequest,
wantErr: true,
},

{
name: "GetProjectVersionsWhenTheContextIsNil",
projectKeyOrID: "10001",
mockFile: "../mocks/get-project-versions.json",
wantHTTPMethod: http.MethodGet,
endpoint: "/rest/api/2/project/10001/versions",
context: nil,
wantHTTPCodeReturn: http.StatusOK,
wantErr: true,
},

{
name: "GetProjectVersionsWhenTheResponseBodyHasADifferentFormat",
projectKeyOrID: "10001",
mockFile: "./mocks/empty_json.json",
wantHTTPMethod: http.MethodGet,
endpoint: "/rest/api/2/project/10001/versions",
context: context.Background(),
wantHTTPCodeReturn: http.StatusOK,
wantErr: true,
},
}

for _, testCase := range testCases {

t.Run(testCase.name, func(t *testing.T) {

//Init a new HTTP mock server
mockOptions := mockServerOptions{
Endpoint: testCase.endpoint,
MockFilePath: testCase.mockFile,
MethodAccepted: testCase.wantHTTPMethod,
ResponseCodeWanted: testCase.wantHTTPCodeReturn,
}

mockServer, err := startMockServer(&mockOptions)
if err != nil {
t.Fatal(err)
}

defer mockServer.Close()

//Init the library instance
mockClient, err := startMockClient(mockServer.URL)
if err != nil {
t.Fatal(err)
}

i := &ProjectVersionService{client: mockClient}

gotResult, gotResponse, err := i.Gets(testCase.context, testCase.projectKeyOrID)

if testCase.wantErr {

if err != nil {
t.Logf("error returned: %v", err.Error())
}

assert.Error(t, err)
} else {

assert.NoError(t, err)
assert.NotEqual(t, gotResponse, nil)
assert.NotEqual(t, gotResult, nil)

apiEndpoint, err := url.Parse(gotResponse.Endpoint)
if err != nil {
t.Fatal(err)
}

var endpointToAssert string

if apiEndpoint.Query().Encode() != "" {
endpointToAssert = fmt.Sprintf("%v?%v", apiEndpoint.Path, apiEndpoint.Query().Encode())
} else {
endpointToAssert = apiEndpoint.Path
}

t.Logf("HTTP Endpoint Wanted: %v, HTTP Endpoint Returned: %v", testCase.endpoint, endpointToAssert)
assert.Equal(t, testCase.endpoint, endpointToAssert)
}
})

}

}

func TestProjectVersionService_Search(t *testing.T) {

testCases := []struct {
name string
projectKeyOrID string
Expand All @@ -385,7 +539,7 @@ func TestProjectVersionService_Gets(t *testing.T) {
wantErr bool
}{
{
name: "GetProjectVersionsWhenTheParamsAreCorrect",
name: "SearchProjectVersionsWhenTheParamsAreCorrect",
projectKeyOrID: "DUMMY",
options: &models.VersionGetsOptions{
OrderBy: "description",
Expand All @@ -404,7 +558,7 @@ func TestProjectVersionService_Gets(t *testing.T) {
},

{
name: "GetProjectVersionsWhenTheOptionsAreNotProvided",
name: "SearchProjectVersionsWhenTheOptionsAreNotProvided",
projectKeyOrID: "DUMMY",
options: &models.VersionGetsOptions{
OrderBy: "",
Expand All @@ -423,7 +577,7 @@ func TestProjectVersionService_Gets(t *testing.T) {
},

{
name: "GetProjectVersionsWhenTheOptionsIsNil",
name: "SearchProjectVersionsWhenTheOptionsIsNil",
projectKeyOrID: "DUMMY",
options: nil,
startAt: 0,
Expand All @@ -437,7 +591,7 @@ func TestProjectVersionService_Gets(t *testing.T) {
},

{
name: "GetProjectVersionsWhenTheRequestMethodIsIncorrect",
name: "SearchProjectVersionsWhenTheRequestMethodIsIncorrect",
projectKeyOrID: "",
options: &models.VersionGetsOptions{
OrderBy: "description",
Expand All @@ -456,7 +610,7 @@ func TestProjectVersionService_Gets(t *testing.T) {
},

{
name: "GetProjectVersionsWhenTheStatusCodeIsIncorrect",
name: "SearchProjectVersionsWhenTheStatusCodeIsIncorrect",
projectKeyOrID: "DUMMY",
options: &models.VersionGetsOptions{
OrderBy: "description",
Expand All @@ -475,7 +629,7 @@ func TestProjectVersionService_Gets(t *testing.T) {
},

{
name: "GetProjectVersionsWhenTheContextIsNil",
name: "SearchProjectVersionsWhenTheContextIsNil",
projectKeyOrID: "DUMMY",
options: &models.VersionGetsOptions{
OrderBy: "description",
Expand All @@ -494,7 +648,7 @@ func TestProjectVersionService_Gets(t *testing.T) {
},

{
name: "GetProjectVersionsWhenTheResponseBodyHasADifferentFormat",
name: "SearchProjectVersionsWhenTheResponseBodyHasADifferentFormat",
projectKeyOrID: "DUMMY",
options: &models.VersionGetsOptions{
OrderBy: "description",
Expand Down Expand Up @@ -540,7 +694,7 @@ func TestProjectVersionService_Gets(t *testing.T) {

i := &ProjectVersionService{client: mockClient}

gotResult, gotResponse, err := i.Gets(testCase.context, testCase.projectKeyOrID, testCase.options,
gotResult, gotResponse, err := i.Search(testCase.context, testCase.projectKeyOrID, testCase.options,
testCase.startAt, testCase.maxResults)

if testCase.wantErr {
Expand Down

0 comments on commit e5903d1

Please sign in to comment.