From 517cfc236573a7ff872dae9e010a4929d7374e9c Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Mislav=20Marohni=C4=87?= <mislav@github.com>
Date: Mon, 22 Feb 2021 16:15:02 +0100
Subject: [PATCH 1/6] Add `api --format` flag for specifying an output template

With the `--format` flag, the value of the flag is parsed as a Go
template which is then evaluated against parsed response data.
https://golang.org/pkg/text/template
---
 pkg/cmd/api/api.go | 25 ++++++++++++++++++++++++-
 1 file changed, 24 insertions(+), 1 deletion(-)

diff --git a/pkg/cmd/api/api.go b/pkg/cmd/api/api.go
index 025225825..79f5432db 100644
--- a/pkg/cmd/api/api.go
+++ b/pkg/cmd/api/api.go
@@ -14,6 +14,7 @@ import (
 	"strconv"
 	"strings"
 	"syscall"
+	"text/template"
 	"time"
 
 	"github.com/MakeNowJust/heredoc"
@@ -40,6 +41,7 @@ type ApiOptions struct {
 	ShowResponseHeaders bool
 	Paginate            bool
 	Silent              bool
+	Template            string
 	CacheTTL            time.Duration
 
 	HttpClient func() (*http.Client, error)
@@ -189,6 +191,7 @@ func NewCmdApi(f *cmdutil.Factory, runF func(*ApiOptions) error) *cobra.Command
 	cmd.Flags().BoolVar(&opts.Paginate, "paginate", false, "Make additional HTTP requests to fetch all pages of results")
 	cmd.Flags().StringVar(&opts.RequestInputFile, "input", "", "The `file` to use as body for the HTTP request")
 	cmd.Flags().BoolVar(&opts.Silent, "silent", false, "Do not print the response body")
+	cmd.Flags().StringVarP(&opts.Template, "format", "t", "", "Format the response using a Go template")
 	cmd.Flags().StringVar(&cacheDuration, "cache", "", "Cache the response for the specified duration")
 	return cmd
 }
@@ -316,7 +319,27 @@ func processResponse(resp *http.Response, opts *ApiOptions, headersOutputStream
 		responseBody = io.TeeReader(responseBody, bodyCopy)
 	}
 
-	if isJSON && opts.IO.ColorEnabled() {
+	if opts.Template != "" {
+		// TODO: reuse parsed template across pagination invocations
+		t := template.Must(template.New("").Parse(opts.Template))
+
+		var jsonData []byte
+		jsonData, err = ioutil.ReadAll(responseBody)
+		if err != nil {
+			return
+		}
+
+		var m interface{}
+		err = json.Unmarshal(jsonData, &m)
+		if err != nil {
+			return
+		}
+
+		err = t.Execute(opts.IO.Out, m)
+		if err != nil {
+			return
+		}
+	} else if isJSON && opts.IO.ColorEnabled() {
 		err = jsoncolor.Write(opts.IO.Out, responseBody, "  ")
 	} else {
 		_, err = io.Copy(opts.IO.Out, responseBody)

From fd82d621d5fa24b8de18d78d05ab93a5b978d71b Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Mislav=20Marohni=C4=87?= <mislav@github.com>
Date: Mon, 22 Feb 2021 16:54:27 +0100
Subject: [PATCH 2/6] Add `color` function to api templates

---
 pkg/cmd/api/api.go | 27 ++++++++++++++++++++++++++-
 1 file changed, 26 insertions(+), 1 deletion(-)

diff --git a/pkg/cmd/api/api.go b/pkg/cmd/api/api.go
index 79f5432db..53e6188bc 100644
--- a/pkg/cmd/api/api.go
+++ b/pkg/cmd/api/api.go
@@ -7,6 +7,7 @@ import (
 	"fmt"
 	"io"
 	"io/ioutil"
+	"math"
 	"net/http"
 	"os"
 	"regexp"
@@ -24,6 +25,7 @@ import (
 	"github.com/cli/cli/pkg/cmdutil"
 	"github.com/cli/cli/pkg/iostreams"
 	"github.com/cli/cli/pkg/jsoncolor"
+	"github.com/mgutz/ansi"
 	"github.com/spf13/cobra"
 )
 
@@ -320,8 +322,31 @@ func processResponse(resp *http.Response, opts *ApiOptions, headersOutputStream
 	}
 
 	if opts.Template != "" {
+		templateFuncs := map[string]interface{}{
+			"color": func(colorName string, input interface{}) (string, error) {
+				var text string
+				switch tt := input.(type) {
+				case string:
+					text = tt
+				case float64:
+					if math.Trunc(tt) == tt {
+						text = strconv.FormatFloat(tt, 'f', 0, 64)
+					} else {
+						text = strconv.FormatFloat(tt, 'f', 2, 64)
+					}
+				case nil:
+					text = ""
+				case bool:
+					text = fmt.Sprintf("%v", tt)
+				default:
+					return "", fmt.Errorf("cannot convert type to string: %v", tt)
+				}
+				return ansi.Color(text, colorName), nil
+			},
+		}
+
 		// TODO: reuse parsed template across pagination invocations
-		t := template.Must(template.New("").Parse(opts.Template))
+		t := template.Must(template.New("").Funcs(templateFuncs).Parse(opts.Template))
 
 		var jsonData []byte
 		jsonData, err = ioutil.ReadAll(responseBody)

From bf97c6e273ff419f22b4da0e9ff9a22bfccc0d12 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Mislav=20Marohni=C4=87?= <mislav@github.com>
Date: Tue, 2 Mar 2021 20:07:04 +0100
Subject: [PATCH 3/6] Add template functions, documentation, tests

---
 pkg/cmd/api/api.go           | 48 ++++++++----------
 pkg/cmd/api/api_test.go      | 39 +++++++++++++++
 pkg/cmd/api/template.go      | 96 ++++++++++++++++++++++++++++++++++++
 pkg/cmd/api/template_test.go | 60 ++++++++++++++++++++++
 4 files changed, 216 insertions(+), 27 deletions(-)
 create mode 100644 pkg/cmd/api/template.go
 create mode 100644 pkg/cmd/api/template_test.go

diff --git a/pkg/cmd/api/api.go b/pkg/cmd/api/api.go
index ac49d715b..7d7f23a92 100644
--- a/pkg/cmd/api/api.go
+++ b/pkg/cmd/api/api.go
@@ -7,7 +7,6 @@ import (
 	"fmt"
 	"io"
 	"io/ioutil"
-	"math"
 	"net/http"
 	"os"
 	"regexp"
@@ -25,7 +24,6 @@ import (
 	"github.com/cli/cli/pkg/cmdutil"
 	"github.com/cli/cli/pkg/iostreams"
 	"github.com/cli/cli/pkg/jsoncolor"
-	"github.com/mgutz/ansi"
 	"github.com/spf13/cobra"
 )
 
@@ -98,6 +96,17 @@ func NewCmdApi(f *cmdutil.Factory, runF func(*ApiOptions) error) *cobra.Command
 			there are no more pages of results. For GraphQL requests, this requires that the
 			original query accepts an %[1]s$endCursor: String%[1]s variable and that it fetches the
 			%[1]spageInfo{ hasNextPage, endCursor }%[1]s set of fields from a collection.
+
+			With %[1]s--template%[1]s, the provided Go template is rendered using the JSON data as input.
+			For the syntax of Go templates, see: https://golang.org/pkg/text/template/
+
+			The following functions are available in templates:
+			- %[1]scolor <style>, <input>%[1]s: colorize input using https://github.com/mgutz/ansi
+			- %[1]sautocolor%[1]s: like %[1]scolor%[1]s, but only emits color to terminals
+			- %[1]stimefmt <format> <time>%[1]s: formats a timestamp using Go's Time.Format function
+			- %[1]stimeago <time>%[1]s: renders a timestamp as relative to now
+			- %[1]spluck <field> <list>%[1]s: collects values of a field from all items in the input
+			- %[1]sjoin <sep> <list>%[1]s: joins values in the list using a separator
 		`, "`"),
 		Example: heredoc.Doc(`
 			# list releases in the current repository
@@ -112,6 +121,10 @@ func NewCmdApi(f *cmdutil.Factory, runF func(*ApiOptions) error) *cobra.Command
 			# set a custom HTTP header
 			$ gh api -H 'Accept: application/vnd.github.XYZ-preview+json' ...
 
+			# use a template for the output
+			$ gh api repos/:owner/:repo/issues --template \
+			  '{{range .}}{{.title}} ({{.labels | pluck "name" | join ", " | color "yellow"}}){{"\n"}}{{end}}'
+
 			# list releases with GraphQL
 			$ gh api graphql -F owner=':owner' -F name=':repo' -f query='
 			  query($name: String!, $owner: String!) {
@@ -183,7 +196,7 @@ func NewCmdApi(f *cmdutil.Factory, runF func(*ApiOptions) error) *cobra.Command
 	cmd.Flags().BoolVar(&opts.Paginate, "paginate", false, "Make additional HTTP requests to fetch all pages of results")
 	cmd.Flags().StringVar(&opts.RequestInputFile, "input", "", "The `file` to use as body for the HTTP request")
 	cmd.Flags().BoolVar(&opts.Silent, "silent", false, "Do not print the response body")
-	cmd.Flags().StringVarP(&opts.Template, "format", "t", "", "Format the response using a Go template")
+	cmd.Flags().StringVarP(&opts.Template, "template", "t", "", "Format the response using a Go template")
 	cmd.Flags().DurationVar(&opts.CacheTTL, "cache", 0, "Cache the response, e.g. \"3600s\", \"60m\", \"1h\"")
 	return cmd
 }
@@ -312,31 +325,12 @@ func processResponse(resp *http.Response, opts *ApiOptions, headersOutputStream
 	}
 
 	if opts.Template != "" {
-		templateFuncs := map[string]interface{}{
-			"color": func(colorName string, input interface{}) (string, error) {
-				var text string
-				switch tt := input.(type) {
-				case string:
-					text = tt
-				case float64:
-					if math.Trunc(tt) == tt {
-						text = strconv.FormatFloat(tt, 'f', 0, 64)
-					} else {
-						text = strconv.FormatFloat(tt, 'f', 2, 64)
-					}
-				case nil:
-					text = ""
-				case bool:
-					text = fmt.Sprintf("%v", tt)
-				default:
-					return "", fmt.Errorf("cannot convert type to string: %v", tt)
-				}
-				return ansi.Color(text, colorName), nil
-			},
-		}
-
 		// TODO: reuse parsed template across pagination invocations
-		t := template.Must(template.New("").Funcs(templateFuncs).Parse(opts.Template))
+		var t *template.Template
+		t, err = parseTemplate(opts.Template, opts.IO.ColorEnabled())
+		if err != nil {
+			return
+		}
 
 		var jsonData []byte
 		jsonData, err = ioutil.ReadAll(responseBody)
diff --git a/pkg/cmd/api/api_test.go b/pkg/cmd/api/api_test.go
index 69ffa100d..7dc508e64 100644
--- a/pkg/cmd/api/api_test.go
+++ b/pkg/cmd/api/api_test.go
@@ -8,9 +8,11 @@ import (
 	"net/http"
 	"os"
 	"path/filepath"
+	"strings"
 	"testing"
 	"time"
 
+	"github.com/MakeNowJust/heredoc"
 	"github.com/cli/cli/git"
 	"github.com/cli/cli/internal/ghrepo"
 	"github.com/cli/cli/pkg/cmdutil"
@@ -910,3 +912,40 @@ func Test_fillPlaceholders(t *testing.T) {
 		})
 	}
 }
+
+func Test_processResponse_template(t *testing.T) {
+	io, _, stdout, stderr := iostreams.Test()
+
+	resp := http.Response{
+		StatusCode: 200,
+		Header: map[string][]string{
+			"Content-Type": {"application/json"},
+		},
+		Body: ioutil.NopCloser(strings.NewReader(`[
+			{
+				"title": "First title",
+				"labels": [{"name":"bug"}, {"name":"help wanted"}]
+			},
+			{
+				"title": "Second but not last"
+			},
+			{
+				"title": "Alas, tis' the end",
+				"labels": [{}, {"name":"feature"}]
+			}
+		]`)),
+	}
+
+	_, err := processResponse(&resp, &ApiOptions{
+		IO:       io,
+		Template: `{{range .}}{{.title}} ({{.labels | pluck "name" | join ", " }}){{"\n"}}{{end}}`,
+	}, ioutil.Discard)
+	require.NoError(t, err)
+
+	assert.Equal(t, heredoc.Doc(`
+		First title (bug, help wanted)
+		Second but not last ()
+		Alas, tis' the end (, feature)
+	`), stdout.String())
+	assert.Equal(t, "", stderr.String())
+}
diff --git a/pkg/cmd/api/template.go b/pkg/cmd/api/template.go
new file mode 100644
index 000000000..823852e51
--- /dev/null
+++ b/pkg/cmd/api/template.go
@@ -0,0 +1,96 @@
+package api
+
+import (
+	"fmt"
+	"math"
+	"strconv"
+	"strings"
+	"text/template"
+	"time"
+
+	"github.com/cli/cli/utils"
+	"github.com/mgutz/ansi"
+)
+
+func parseTemplate(tpl string, colorEnabled bool) (*template.Template, error) {
+	now := time.Now()
+
+	templateFuncs := map[string]interface{}{
+		"color":     templateColor,
+		"autocolor": templateColor,
+
+		"timefmt": func(format, input string) (string, error) {
+			t, err := time.Parse(time.RFC3339, input)
+			if err != nil {
+				return "", err
+			}
+			return t.Format(format), nil
+		},
+		"timeago": func(input string) (string, error) {
+			t, err := time.Parse(time.RFC3339, input)
+			if err != nil {
+				return "", err
+			}
+			return utils.FuzzyAgoAbbr(now, t), nil
+		},
+
+		"pluck": templatePluck,
+		"join":  templateJoin,
+	}
+
+	if !colorEnabled {
+		templateFuncs["autocolor"] = func(colorName string, input interface{}) (string, error) {
+			return jsonScalarToString(input)
+		}
+	}
+
+	return template.New("").Funcs(templateFuncs).Parse(tpl)
+}
+
+func jsonScalarToString(input interface{}) (string, error) {
+	switch tt := input.(type) {
+	case string:
+		return tt, nil
+	case float64:
+		if math.Trunc(tt) == tt {
+			return strconv.FormatFloat(tt, 'f', 0, 64), nil
+		} else {
+			return strconv.FormatFloat(tt, 'f', 2, 64), nil
+		}
+	case nil:
+		return "", nil
+	case bool:
+		return fmt.Sprintf("%v", tt), nil
+	default:
+		return "", fmt.Errorf("cannot convert type to string: %v", tt)
+	}
+}
+
+func templateColor(colorName string, input interface{}) (string, error) {
+	text, err := jsonScalarToString(input)
+	if err != nil {
+		return "", err
+	}
+	return ansi.Color(text, colorName), nil
+}
+
+func templatePluck(field string, input []interface{}) []interface{} {
+	var results []interface{}
+	for _, item := range input {
+		obj := item.(map[string]interface{})
+		results = append(results, obj[field])
+	}
+	return results
+}
+
+func templateJoin(sep string, input []interface{}) (string, error) {
+	var results []string
+	for _, item := range input {
+		text, err := jsonScalarToString(item)
+		if err != nil {
+			return "", err
+		}
+		results = append(results, text)
+	}
+	return strings.Join(results, sep), nil
+}
diff --git a/pkg/cmd/api/template_test.go b/pkg/cmd/api/template_test.go
new file mode 100644
index 000000000..ff03d9a98
--- /dev/null
+++ b/pkg/cmd/api/template_test.go
@@ -0,0 +1,60 @@
+package api
+
+import "testing"
+
+func Test_jsonScalarToString(t *testing.T) {
+	tests := []struct {
+		name    string
+		input   interface{}
+		want    string
+		wantErr bool
+	}{
+		{
+			name:  "string",
+			input: "hello",
+			want:  "hello",
+		},
+		{
+			name:  "int",
+			input: float64(1234),
+			want:  "1234",
+		},
+		{
+			name:  "float",
+			input: float64(12.34),
+			want:  "12.34",
+		},
+		{
+			name:  "null",
+			input: nil,
+			want:  "",
+		},
+		{
+			name:  "true",
+			input: true,
+			want:  "true",
+		},
+		{
+			name:  "false",
+			input: false,
+			want:  "false",
+		},
+		{
+			name:    "object",
+			input:   map[string]interface{}{},
+			wantErr: true,
+		},
+	}
+	for _, tt := range tests {
+		t.Run(tt.name, func(t *testing.T) {
+			got, err := jsonScalarToString(tt.input)
+			if (err != nil) != tt.wantErr {
+				t.Errorf("jsonScalarToString() error = %v, wantErr %v", err, tt.wantErr)
+				return
+			}
+			if got != tt.want {
+				t.Errorf("jsonScalarToString() = %v, want %v", got, tt.want)
+			}
+		})
+	}
+}

From 0f27084f570995936fb63671ec247e70d4768808 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Mislav=20Marohni=C4=87?= <mislav@github.com>
Date: Thu, 4 Mar 2021 15:01:59 +0100
Subject: [PATCH 4/6] Add flag parsing test for `api --template`

---
 pkg/cmd/api/api_test.go | 31 +++++++++++++++++++++++++++++++
 1 file changed, 31 insertions(+)

diff --git a/pkg/cmd/api/api_test.go b/pkg/cmd/api/api_test.go
index 16df36c20..75132d7b4 100644
--- a/pkg/cmd/api/api_test.go
+++ b/pkg/cmd/api/api_test.go
@@ -47,6 +47,7 @@ func Test_NewCmdApi(t *testing.T) {
 				Paginate:            false,
 				Silent:              false,
 				CacheTTL:            0,
+				Template:            "",
 			},
 			wantsErr: false,
 		},
@@ -66,6 +67,7 @@ func Test_NewCmdApi(t *testing.T) {
 				Paginate:            false,
 				Silent:              false,
 				CacheTTL:            0,
+				Template:            "",
 			},
 			wantsErr: false,
 		},
@@ -85,6 +87,7 @@ func Test_NewCmdApi(t *testing.T) {
 				Paginate:            false,
 				Silent:              false,
 				CacheTTL:            0,
+				Template:            "",
 			},
 			wantsErr: false,
 		},
@@ -104,6 +107,7 @@ func Test_NewCmdApi(t *testing.T) {
 				Paginate:            false,
 				Silent:              false,
 				CacheTTL:            0,
+				Template:            "",
 			},
 			wantsErr: false,
 		},
@@ -123,6 +127,7 @@ func Test_NewCmdApi(t *testing.T) {
 				Paginate:            true,
 				Silent:              false,
 				CacheTTL:            0,
+				Template:            "",
 			},
 			wantsErr: false,
 		},
@@ -142,6 +147,7 @@ func Test_NewCmdApi(t *testing.T) {
 				Paginate:            false,
 				Silent:              true,
 				CacheTTL:            0,
+				Template:            "",
 			},
 			wantsErr: false,
 		},
@@ -166,6 +172,7 @@ func Test_NewCmdApi(t *testing.T) {
 				Paginate:            true,
 				Silent:              false,
 				CacheTTL:            0,
+				Template:            "",
 			},
 			wantsErr: false,
 		},
@@ -190,6 +197,7 @@ func Test_NewCmdApi(t *testing.T) {
 				Paginate:            false,
 				Silent:              false,
 				CacheTTL:            0,
+				Template:            "",
 			},
 			wantsErr: false,
 		},
@@ -214,6 +222,7 @@ func Test_NewCmdApi(t *testing.T) {
 				Paginate:            false,
 				Silent:              false,
 				CacheTTL:            0,
+				Template:            "",
 			},
 			wantsErr: false,
 		},
@@ -233,6 +242,27 @@ func Test_NewCmdApi(t *testing.T) {
 				Paginate:            false,
 				Silent:              false,
 				CacheTTL:            time.Minute * 5,
+				Template:            "",
+			},
+			wantsErr: false,
+		},
+		{
+			name: "with template",
+			cli:  "user -t 'hello {{.name}}'",
+			wants: ApiOptions{
+				Hostname:            "",
+				RequestMethod:       "GET",
+				RequestMethodPassed: false,
+				RequestPath:         "user",
+				RequestInputFile:    "",
+				RawFields:           []string(nil),
+				MagicFields:         []string(nil),
+				RequestHeaders:      []string(nil),
+				ShowResponseHeaders: false,
+				Paginate:            false,
+				Silent:              false,
+				CacheTTL:            0,
+				Template:            "hello {{.name}}",
 			},
 			wantsErr: false,
 		},
@@ -270,6 +300,7 @@ func Test_NewCmdApi(t *testing.T) {
 			assert.Equal(t, tt.wants.Paginate, opts.Paginate)
 			assert.Equal(t, tt.wants.Silent, opts.Silent)
 			assert.Equal(t, tt.wants.CacheTTL, opts.CacheTTL)
+			assert.Equal(t, tt.wants.Template, opts.Template)
 		})
 	}
 }

From f53ad7161ac956f94f1c4a6770f081472a0f8971 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Mislav=20Marohni=C4=87?= <mislav@github.com>
Date: Thu, 4 Mar 2021 16:35:08 +0100
Subject: [PATCH 5/6] Add more `api --template` tests

---
 pkg/cmd/api/api.go           |  21 +------
 pkg/cmd/api/template.go      |  43 +++++++++++++-
 pkg/cmd/api/template_test.go | 105 ++++++++++++++++++++++++++++++++++-
 3 files changed, 147 insertions(+), 22 deletions(-)

diff --git a/pkg/cmd/api/api.go b/pkg/cmd/api/api.go
index 106a564d2..72e1edb19 100644
--- a/pkg/cmd/api/api.go
+++ b/pkg/cmd/api/api.go
@@ -14,7 +14,6 @@ import (
 	"strconv"
 	"strings"
 	"syscall"
-	"text/template"
 	"time"
 
 	"github.com/MakeNowJust/heredoc"
@@ -335,25 +334,7 @@ func processResponse(resp *http.Response, opts *ApiOptions, headersOutputStream
 
 	if opts.Template != "" {
 		// TODO: reuse parsed template across pagination invocations
-		var t *template.Template
-		t, err = parseTemplate(opts.Template, opts.IO.ColorEnabled())
-		if err != nil {
-			return
-		}
-
-		var jsonData []byte
-		jsonData, err = ioutil.ReadAll(responseBody)
-		if err != nil {
-			return
-		}
-
-		var m interface{}
-		err = json.Unmarshal(jsonData, &m)
-		if err != nil {
-			return
-		}
-
-		err = t.Execute(opts.IO.Out, m)
+		err = executeTemplate(opts.IO.Out, responseBody, opts.Template, opts.IO.ColorEnabled())
 		if err != nil {
 			return
 		}
diff --git a/pkg/cmd/api/template.go b/pkg/cmd/api/template.go
index 823852e51..78c7a96c8 100644
--- a/pkg/cmd/api/template.go
+++ b/pkg/cmd/api/template.go
@@ -1,7 +1,10 @@
 package api
 
 import (
+	"encoding/json"
 	"fmt"
+	"io"
+	"io/ioutil"
 	"math"
 	"strconv"
 	"strings"
@@ -31,7 +34,7 @@ func parseTemplate(tpl string, colorEnabled bool) (*template.Template, error) {
 			if err != nil {
 				return "", err
 			}
-			return utils.FuzzyAgoAbbr(now, t), nil
+			return timeAgo(now.Sub(t)), nil
 		},
 
 		"pluck": templatePluck,
@@ -47,6 +50,25 @@ func parseTemplate(tpl string, colorEnabled bool) (*template.Template, error) {
 	return template.New("").Funcs(templateFuncs).Parse(tpl)
 }
 
+func executeTemplate(w io.Writer, input io.Reader, templateStr string, colorEnabled bool) error {
+	t, err := parseTemplate(templateStr, colorEnabled)
+	if err != nil {
+		return err
+	}
+
+	jsonData, err := ioutil.ReadAll(input)
+	if err != nil {
+		return err
+	}
+
+	var data interface{}
+	if err := json.Unmarshal(jsonData, &data); err != nil {
+		return err
+	}
+
+	return t.Execute(w, data)
+}
+
 func jsonScalarToString(input interface{}) (string, error) {
 	switch tt := input.(type) {
 	case string:
@@ -94,3 +116,22 @@ func templateJoin(sep string, input []interface{}) (string, error) {
 	}
 	return strings.Join(results, sep), nil
 }
+
+func timeAgo(ago time.Duration) string {
+	if ago < time.Minute {
+		return "just now"
+	}
+	if ago < time.Hour {
+		return utils.Pluralize(int(ago.Minutes()), "minute") + " ago"
+	}
+	if ago < 24*time.Hour {
+		return utils.Pluralize(int(ago.Hours()), "hour") + " ago"
+	}
+	if ago < 30*24*time.Hour {
+		return utils.Pluralize(int(ago.Hours())/24, "day") + " ago"
+	}
+	if ago < 365*24*time.Hour {
+		return utils.Pluralize(int(ago.Hours())/24/30, "month") + " ago"
+	}
+	return utils.Pluralize(int(ago.Hours()/24/365), "year") + " ago"
+}
diff --git a/pkg/cmd/api/template_test.go b/pkg/cmd/api/template_test.go
index ff03d9a98..067b87fcc 100644
--- a/pkg/cmd/api/template_test.go
+++ b/pkg/cmd/api/template_test.go
@@ -1,6 +1,15 @@
 package api
 
-import "testing"
+import (
+	"bytes"
+	"fmt"
+	"io"
+	"strings"
+	"testing"
+	"time"
+
+	"github.com/MakeNowJust/heredoc"
+)
 
 func Test_jsonScalarToString(t *testing.T) {
 	tests := []struct {
@@ -58,3 +67,97 @@ func Test_jsonScalarToString(t *testing.T) {
 		})
 	}
 }
+
+func Test_executeTemplate(t *testing.T) {
+	type args struct {
+		json     io.Reader
+		template string
+		colorize bool
+	}
+	tests := []struct {
+		name    string
+		args    args
+		wantW   string
+		wantErr bool
+	}{
+		{
+			name: "color",
+			args: args{
+				json:     strings.NewReader(`{}`),
+				template: `{{color "blue+h" "songs are like tattoos"}}`,
+				colorize: false,
+			},
+			wantW: "\x1b[0;94msongs are like tattoos\x1b[0m",
+		},
+		{
+			name: "autocolor enabled",
+			args: args{
+				json:     strings.NewReader(`{}`),
+				template: `{{autocolor "red" "stop"}}`,
+				colorize: true,
+			},
+			wantW: "\x1b[0;31mstop\x1b[0m",
+		},
+		{
+			name: "autocolor disabled",
+			args: args{
+				json:     strings.NewReader(`{}`),
+				template: `{{autocolor "red" "go"}}`,
+				colorize: false,
+			},
+			wantW: "go",
+		},
+		{
+			name: "timefmt",
+			args: args{
+				json:     strings.NewReader(`{"created_at":"2008-02-25T20:18:33Z"}`),
+				template: `{{.created_at | timefmt "Mon Jan 2, 2006"}}`,
+				colorize: false,
+			},
+			wantW: "Mon Feb 25, 2008",
+		},
+		{
+			name: "timeago",
+			args: args{
+				json:     strings.NewReader(fmt.Sprintf(`{"created_at":"%s"}`, time.Now().Add(-5*time.Minute).Format(time.RFC3339))),
+				template: `{{.created_at | timeago}}`,
+				colorize: false,
+			},
+			wantW: "5 minutes ago",
+		},
+		{
+			name: "pluck",
+			args: args{
+				json: strings.NewReader(heredoc.Doc(`[
+					{"name": "bug"},
+					{"name": "feature request"},
+					{"name": "chore"}
+				]`)),
+				template: `{{range(pluck "name" .)}}{{. | printf "%s\n"}}{{end}}`,
+				colorize: false,
+			},
+			wantW: "bug\nfeature request\nchore\n",
+		},
+		{
+			name: "join",
+			args: args{
+				json:     strings.NewReader(`[ "bug", "feature request", "chore" ]`),
+				template: `{{join "\t" .}}`,
+				colorize: false,
+			},
+			wantW: "bug\tfeature request\tchore",
+		},
+	}
+	for _, tt := range tests {
+		t.Run(tt.name, func(t *testing.T) {
+			w := &bytes.Buffer{}
+			if err := executeTemplate(w, tt.args.json, tt.args.template, tt.args.colorize); (err != nil) != tt.wantErr {
+				t.Errorf("executeTemplate() error = %v, wantErr %v", err, tt.wantErr)
+				return
+			}
+			if gotW := w.String(); gotW != tt.wantW {
+				t.Errorf("executeTemplate() = %q, want %q", gotW, tt.wantW)
+			}
+		})
+	}
+}

From eb08774370a0240f07a6235edceb53dd29aa2aa7 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Mislav=20Marohni=C4=87?= <mislav@github.com>
Date: Thu, 4 Mar 2021 16:48:06 +0100
Subject: [PATCH 6/6] Assert that `executeTemplate` is invoked

---
 pkg/cmd/api/api_test.go | 14 ++++++++++++++
 1 file changed, 14 insertions(+)

diff --git a/pkg/cmd/api/api_test.go b/pkg/cmd/api/api_test.go
index 75132d7b4..3b2acb6cf 100644
--- a/pkg/cmd/api/api_test.go
+++ b/pkg/cmd/api/api_test.go
@@ -426,6 +426,20 @@ func Test_apiRun(t *testing.T) {
 			stdout: "HTTP/1.1 200 Okey-dokey\nContent-Type: text/plain\r\n\r\n",
 			stderr: ``,
 		},
+		{
+			name: "output template",
+			options: ApiOptions{
+				Template: `{{.status}}`,
+			},
+			httpResponse: &http.Response{
+				StatusCode: 200,
+				Body:       ioutil.NopCloser(bytes.NewBufferString(`{"status":"not a cat"}`)),
+				Header:     http.Header{"Content-Type": []string{"application/json"}},
+			},
+			err:    nil,
+			stdout: "not a cat",
+			stderr: ``,
+		},
 	}
 
 	for _, tt := range tests {