Merge pull request #49 from RachelTucker/GOSDK-12

GOSDK-12: Allowing multiple ranges to be set in get object

Author: GraciesPadre

ds3/ds3Client_test.go

@@ -328,12 +328,16 @@ func TestGetPartialObject(t *testing.T) {
 
     // Create and run the mocked client.
     requestHeaders := &http.Header{}
-    requestHeaders.Add("Range", "bytes=1-5")
+    requestHeaders.Add("Range", "bytes=1-5,7-8,20-500")
 
     response, err := mockedClient(t).
         Expecting(HTTP_VERB_GET, "/bucketName/object", &url.Values{}, requestHeaders, nil).
         Returning(206, stringResponse, nil).
-        GetObject(models.NewGetObjectRequest("bucketName", "object").WithRange(1, 5))
+        GetObject(models.NewGetObjectRequest("bucketName", "object").
+            WithRanges(
+                models.Range{Start: 1, End: 5},
+                models.Range{Start: 7, End: 8},
+                models.Range{Start: 20, End: 500}))
 
     // Check the error result.
     ds3Testing.AssertNilError(t, err)
@@ -357,7 +361,7 @@ func TestGetObjectRange(t *testing.T) {
 
     // Create and run the mocked client.
     request := models.NewGetObjectRequest("bucketName", "object").
-            WithRange(20, 179)
+            WithRanges(models.Range{Start: 20, End: 179})
 
     response, err := mockedClient(t).
         Expecting(HTTP_VERB_GET, "/bucketName/object", &url.Values{}, requestHeaders, nil).

ds3/models/getObjectRequest.go

@@ -15,10 +15,12 @@ package models
 
 import (
     "fmt"
+    "strings"
 )
 
-type rangeHeader struct {
-    start, end int
+type Range struct {
+    Start int64
+    End int64
 }
 
 type GetObjectRequest struct {
@@ -56,7 +58,11 @@ func (getObjectRequest *GetObjectRequest) WithChecksum(contentHash string, check
     return getObjectRequest
 }
 
-func (getObjectRequest *GetObjectRequest) WithRange(start, end int) *GetObjectRequest {
-    getObjectRequest.Metadata["Range"] = fmt.Sprintf("bytes=%d-%d", start, end)
+func (getObjectRequest *GetObjectRequest) WithRanges(ranges ...Range) *GetObjectRequest {
+    var rangeElements []string
+    for _, cur := range ranges {
+        rangeElements = append(rangeElements, fmt.Sprintf("%d-%d", cur.Start, cur.End))
+    }
+    getObjectRequest.Metadata["Range"] = fmt.Sprintf("bytes=%s", strings.Join(rangeElements[:], ","))
     return getObjectRequest
-}
+}