improve test for exporting of issues: check content of excel file

Author: tpokorra

tests/integration/issue_test.go

@@ -4,6 +4,7 @@
 package integration
 
 import (
+	"bytes"
 	"fmt"
 	"html/template"
 	"net/http"
@@ -29,6 +30,7 @@ import (
 
 	"github.com/PuerkitoBio/goquery"
 	"github.com/stretchr/testify/assert"
+	"github.com/xuri/excelize/v2"
 )
 
 func getIssuesSelection(t testing.TB, htmlDoc *HTMLDoc) *goquery.Selection {
@@ -374,7 +376,7 @@ func TestIssueReaction(t *testing.T) {
 func TestIssueListExport(t *testing.T) {
 	defer tests.PrepareTestEnv(t)()
 	session := loginUser(t, "user2")
-	_ = testNewIssue(t, session, "user2", "repo1", "Title", "Description")
+	_ = testNewIssue(t, session, "user2", "repo1", "Title1", "Description1")
 	_ = testNewIssue(t, session, "user2", "repo1", "Title2", "Description2")
 	_ = testNewIssue(t, session, "user2", "repo1", "Title3", "Description3")
 
@@ -390,6 +392,41 @@ func TestIssueListExport(t *testing.T) {
 	cd := resp.Header().Get("Content-Disposition")
 	assert.Contains(t, cd, "attachment")
 	assert.True(t, strings.Contains(cd, ".xlsx"))
+
+	// open bytes as XLSX with excelize
+	data := resp.Body.Bytes()
+	f, err := excelize.OpenReader(bytes.NewReader(data))
+	assert.NoError(t, err)
+	defer func() { _ = f.Close() }()
+
+	// get first sheet and rows
+	sheets := f.GetSheetList()
+	assert.NotEmpty(t, sheets)
+	rows, err := f.GetRows(sheets[0])
+	assert.NoError(t, err)
+	// there should be at least a header row + our three issues
+	assert.GreaterOrEqual(t, len(rows), 4)
+
+	// ensure header has some cells and that our created titles appear somewhere
+	foundTitle1, foundTitle2, foundTitle3 := false, false, false
+	if len(rows) > 0 {
+		for _, row := range rows {
+			for _, cell := range row {
+				if strings.Contains(cell, "Title1") {
+					foundTitle1 = true
+				}
+				if strings.Contains(cell, "Title2") {
+					foundTitle2 = true
+				}
+				if strings.Contains(cell, "Title3") {
+					foundTitle3 = true
+				}
+			}
+		}
+	}
+	assert.True(t, foundTitle1, "Exported XLSX should contain Title1")
+	assert.True(t, foundTitle2, "Exported XLSX should contain Title2")
+	assert.True(t, foundTitle3, "Exported XLSX should contain Title3")
 }
 
 func TestIssueCrossReference(t *testing.T) {