Merge pull request #33 from nii236/feature/fix_source_target_project_id_types

Feature/fix source target project id types

Author: Raphaël Benitte

merge_requests.go

@@ -32,8 +32,8 @@ type MergeRequest struct {
 	Description     string `json:"description,omitempty"`
 	WorkInProgress  bool   `json:"work_in_progress,omitempty"`
 	MergeStatus     string `json:"merge_status,omitempty"`
-	SourceProjectID string `json:"source_project_id,omitempty"`
-	TargetProjectID string `json:"target_project_id,omitempty"`
+	SourceProjectID int    `json:"source_project_id,omitempty"`
+	TargetProjectID int    `json:"target_project_id,omitempty"`
 }
 
 type ChangeItem struct {

merge_requests_test.go

@@ -22,8 +22,8 @@ func TestProjectMergeRequest(t *testing.T) {
 	assert.Equal(t, err, nil)
 	assert.Equal(t, mr.TargetBranch, "master")
 	assert.Equal(t, mr.MergeStatus, "can_be_merged")
-	assert.Equal(t, mr.SourceProjectID, "2")
-	assert.Equal(t, mr.TargetProjectID, "3")
+	assert.Equal(t, mr.SourceProjectID, 2)
+	assert.Equal(t, mr.TargetProjectID, 3)
 	defer ts.Close()
 }
 

stubs/merge_requests/show.json

@@ -26,8 +26,8 @@
     "state": "active",
     "created_at": "2012-04-29T08:46:00Z"
   },
-  "source_project_id": "2",
-  "target_project_id": "3",
+  "source_project_id": 2,
+  "target_project_id": 3,
   "description":"fixed login page css paddings",
   "work_in_progress": false,
   "merge_status": "can_be_merged"