From b442a1b4d20659bf4cf45613ebaa712288666cda Mon Sep 17 00:00:00 2001
From: Andy Feller <andyfeller@github.com>
Date: Mon, 11 Dec 2023 08:42:43 -0500
Subject: [PATCH] Fix run exported jobs error for completedAt

---
 pkg/cmd/run/shared/shared.go      |  6 +-
 pkg/cmd/run/shared/shared_test.go | 97 +++++++++++++++++++++++++++++++
 2 files changed, 100 insertions(+), 3 deletions(-)

diff --git a/pkg/cmd/run/shared/shared.go b/pkg/cmd/run/shared/shared.go
index 20846d60f..6c76714fd 100644
--- a/pkg/cmd/run/shared/shared.go
+++ b/pkg/cmd/run/shared/shared.go
@@ -186,9 +186,9 @@ func (r *Run) ExportData(fields []string) map[string]interface{} {
 						"number":     s.Number,
 					})
 				}
-				var completedAt *time.Time
+				var completedAt time.Time
 				if !j.CompletedAt.IsZero() {
-					completedAt = &j.CompletedAt
+					completedAt = j.CompletedAt
 				}
 				jobs = append(jobs, map[string]interface{}{
 					"databaseId":  j.ID,
@@ -200,8 +200,8 @@ func (r *Run) ExportData(fields []string) map[string]interface{} {
 					"completedAt": completedAt,
 					"url":         j.URL,
 				})
-				data[f] = jobs
 			}
+			data[f] = jobs
 		default:
 			sf := fieldByName(v, f)
 			data[f] = sf.Interface()
diff --git a/pkg/cmd/run/shared/shared_test.go b/pkg/cmd/run/shared/shared_test.go
index 5d7768207..815a6556c 100644
--- a/pkg/cmd/run/shared/shared_test.go
+++ b/pkg/cmd/run/shared/shared_test.go
@@ -1,8 +1,10 @@
 package shared
 
 import (
+	"bytes"
 	"encoding/json"
 	"net/http"
+	"strings"
 	"testing"
 	"time"
 
@@ -11,6 +13,7 @@ import (
 	"github.com/cli/cli/v2/internal/ghrepo"
 	"github.com/cli/cli/v2/pkg/httpmock"
 	"github.com/stretchr/testify/assert"
+	"github.com/stretchr/testify/require"
 )
 
 func TestPreciseAgo(t *testing.T) {
@@ -104,3 +107,97 @@ func TestRun_Duration(t *testing.T) {
 		})
 	}
 }
+
+func TestRunExportData(t *testing.T) {
+	oldestStartedAt, _ := time.Parse(time.RFC3339, "2022-07-20T11:20:13Z")
+	oldestCompletedAt, _ := time.Parse(time.RFC3339, "2022-07-20T11:21:16Z")
+	newestStartedAt, _ := time.Parse(time.RFC3339, "2022-07-20T11:20:55Z")
+	newestCompletedAt, _ := time.Parse(time.RFC3339, "2022-07-20T11:23:16Z")
+
+	tests := []struct {
+		name   string
+		fields []string
+		run    Run
+		output string
+	}{
+		{
+			name:   "exports workflow run's single job",
+			fields: []string{"jobs"},
+			run: Run{
+				Jobs: []Job{
+					{
+						ID:         123456,
+						Status:     "completed",
+						Conclusion: "success",
+						Name:       "macos",
+						Steps: []Step{
+							{
+								Name:       "Checkout",
+								Status:     "completed",
+								Conclusion: "success",
+								Number:     1,
+							},
+						},
+						StartedAt:   oldestStartedAt,
+						CompletedAt: oldestCompletedAt,
+						URL:         "https://example.com/OWNER/REPO/actions/runs/123456",
+					},
+				},
+			},
+			output: `{"jobs":[{"completedAt":"2022-07-20T11:21:16Z","conclusion":"success","databaseId":123456,"name":"macos","startedAt":"2022-07-20T11:20:13Z","status":"completed","steps":[{"conclusion":"success","name":"Checkout","number":1,"status":"completed"}],"url":"https://example.com/OWNER/REPO/actions/runs/123456"}]}`,
+		},
+		{
+			name:   "exports workflow run's multiple jobs",
+			fields: []string{"jobs"},
+			run: Run{
+				Jobs: []Job{
+					{
+						ID:         123456,
+						Status:     "completed",
+						Conclusion: "success",
+						Name:       "macos",
+						Steps: []Step{
+							{
+								Name:       "Checkout",
+								Status:     "completed",
+								Conclusion: "success",
+								Number:     1,
+							},
+						},
+						StartedAt:   oldestStartedAt,
+						CompletedAt: oldestCompletedAt,
+						URL:         "https://example.com/OWNER/REPO/actions/runs/123456",
+					},
+					{
+						ID:         234567,
+						Status:     "completed",
+						Conclusion: "error",
+						Name:       "windows",
+						Steps: []Step{
+							{
+								Name:       "Checkout",
+								Status:     "completed",
+								Conclusion: "error",
+								Number:     2,
+							},
+						},
+						StartedAt:   newestStartedAt,
+						CompletedAt: newestCompletedAt,
+						URL:         "https://example.com/OWNER/REPO/actions/runs/234567",
+					},
+				},
+			},
+			output: `{"jobs":[{"completedAt":"2022-07-20T11:21:16Z","conclusion":"success","databaseId":123456,"name":"macos","startedAt":"2022-07-20T11:20:13Z","status":"completed","steps":[{"conclusion":"success","name":"Checkout","number":1,"status":"completed"}],"url":"https://example.com/OWNER/REPO/actions/runs/123456"},{"completedAt":"2022-07-20T11:23:16Z","conclusion":"error","databaseId":234567,"name":"windows","startedAt":"2022-07-20T11:20:55Z","status":"completed","steps":[{"conclusion":"error","name":"Checkout","number":2,"status":"completed"}],"url":"https://example.com/OWNER/REPO/actions/runs/234567"}]}`,
+		},
+	}
+
+	for _, tt := range tests {
+		t.Run(tt.name, func(t *testing.T) {
+			exported := tt.run.ExportData(tt.fields)
+			buf := bytes.Buffer{}
+			enc := json.NewEncoder(&buf)
+			require.NoError(t, enc.Encode(exported))
+			assert.Equal(t, tt.output, strings.TrimSpace(buf.String()))
+		})
+	}
+}