fix: csv format priority (#882)

Author: ciricc

internal/view/issues.go

@@ -39,6 +39,12 @@ type IssueList struct {
 
 // Render renders the view.
 func (l *IssueList) Render() error {
+	// Prioritize CSV format when explicitly requested
+	if l.Display.CSV {
+		w := os.Stdout
+		return l.renderCSV(w)
+	}
+
 	if l.Display.Plain || tui.IsDumbTerminal() || tui.IsNotTTY() {
 		// custom delimiter is used only in plain mode, otherwise \t is used
 		delimeter := "\t"
@@ -49,11 +55,6 @@ func (l *IssueList) Render() error {
 		return l.renderPlain(w, delimeter)
 	}
 
-	if l.Display.CSV {
-		w := os.Stdout
-		return l.renderCSV(w)
-	}
-
 	renderer, err := MDRenderer()
 	if err != nil {
 		return err

internal/view/issues_test.go

@@ -148,6 +148,49 @@ TEST-2	Story	Open	2020-12-13 14:05:20
 	assert.Equal(t, expected, b.String())
 }
 
+func TestIssueRenderInCSVFormat(t *testing.T) {
+	var b bytes.Buffer
+
+	issue := IssueList{
+		Project: "TEST",
+		Server:  "https://test.local",
+		Data:    getIssues(),
+		Display: DisplayFormat{
+			CSV:        true,
+			NoHeaders:  false,
+			NoTruncate: true,
+		},
+	}
+	assert.NoError(t, issue.renderCSV(&b))
+
+	expected := `TYPE,KEY,SUMMARY,STATUS,ASSIGNEE,REPORTER,PRIORITY,RESOLUTION,CREATED,UPDATED,LABELS
+Bug,TEST-1,This is a test,Done,Person A,Person Z,High,Fixed,2020-12-13 14:05:20,2020-12-13 14:07:20,krakatit
+Story,TEST-2,This is another test,Open,,Person A,Normal,,2020-12-13 14:05:20,2020-12-13 14:07:20,"pat,mat"
+`
+	assert.Equal(t, expected, b.String())
+}
+
+func TestIssueRenderInCSVFormatWithoutHeaders(t *testing.T) {
+	var b bytes.Buffer
+
+	issue := IssueList{
+		Project: "TEST",
+		Server:  "https://test.local",
+		Data:    getIssues(),
+		Display: DisplayFormat{
+			CSV:        true,
+			NoHeaders:  true,
+			NoTruncate: true,
+		},
+	}
+	assert.NoError(t, issue.renderCSV(&b))
+
+	expected := `Bug,TEST-1,This is a test,Done,Person A,Person Z,High,Fixed,2020-12-13 14:05:20,2020-12-13 14:07:20,krakatit
+Story,TEST-2,This is another test,Open,,Person A,Normal,,2020-12-13 14:05:20,2020-12-13 14:07:20,"pat,mat"
+`
+	assert.Equal(t, expected, b.String())
+}
+
 func getIssues() []*jira.Issue {
 	return []*jira.Issue{
 		{