From 2d1ed05a4419b5a6d004680e4a148eb3c6206020 Mon Sep 17 00:00:00 2001
From: Tom Hudson <mail@tomnomnom.com>
Date: Fri, 17 Jun 2016 00:32:30 +0100
Subject: [PATCH] Adds String method for Links

---
 examples_test.go | 13 +++++++++++++
 main.go          | 13 +++++++++++--
 main_test.go     | 15 +++++++++++++++
 3 files changed, 39 insertions(+), 2 deletions(-)

diff --git a/examples_test.go b/examples_test.go
index f6e4300..2dc48f4 100644
--- a/examples_test.go
+++ b/examples_test.go
@@ -61,3 +61,16 @@ func ExampleLink_String() {
 	// Output:
 	// Link: <http://example.com/page/2>; rel="next"
 }
+
+func ExampleLinks_String() {
+
+	links := linkheader.Links{
+		{URL: "http://example.com/page/3", Rel: "next"},
+		{URL: "http://example.com/page/1", Rel: "last"},
+	}
+
+	fmt.Printf("Link: %s\n", links.String())
+
+	// Output:
+	// Link: <http://example.com/page/3>; rel="next", <http://example.com/page/1>; rel="last"
+}
diff --git a/main.go b/main.go
index a86f2ed..fa2cb7a 100644
--- a/main.go
+++ b/main.go
@@ -33,8 +33,7 @@ func (l Link) Param(key string) (string, error) {
 	return "", fmt.Errorf("Could not find param '%s'", key)
 }
 
-// String returns the string representation of a Link header
-// for use in HTTP responses
+// String returns the string representation of a link
 func (l Link) String() string {
 
 	p := make([]string, len(l.Params))
@@ -61,6 +60,16 @@ func (l Links) FilterByRel(r string) Links {
 	return links
 }
 
+// String returns the string representation of multiple Links
+// for use in HTTP responses etc
+func (l Links) String() string {
+	strs := make([]string, 0)
+	for _, link := range l {
+		strs = append(strs, link.String())
+	}
+	return strings.Join(strs, ", ")
+}
+
 // Parse parses a raw Link header in the form:
 //   <url>; rel="foo", <url>; rel="bar"; wat="dis"
 // returning a slice of Link structs
diff --git a/main_test.go b/main_test.go
index 8ee2389..1b569f3 100644
--- a/main_test.go
+++ b/main_test.go
@@ -131,3 +131,18 @@ func TestLinkToString(t *testing.T) {
 		t.Errorf("Re-parsed link header should have matching rel, but has `%s`", parsed[0].Rel)
 	}
 }
+
+func TestLinksToString(t *testing.T) {
+	ls := Links{
+		{URL: "http://example.com/page/3", Rel: "next"},
+		{URL: "http://example.com/page/1", Rel: "last"},
+	}
+
+	have := ls.String()
+
+	want := "<http://example.com/page/3>; rel=\"next\", <http://example.com/page/1>; rel=\"last\""
+
+	if have != want {
+		t.Errorf("Want `%s`, have `%s`", want, have)
+	}
+}