fix(report): wrong pdf data type assertion during generation (#3681)

Signed-off-by: Rogério Peixoto <[email protected]>
Checkmarx · Jun 16, 2021 · 5174a8b · 5174a8b
1 parent 38e78ef
commit 5174a8b
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/pkg/report/pdf.go b/pkg/report/pdf.go
@@ -175,7 +175,7 @@ func PrintPdfReport(path, filename string, body interface{}) error {
 	startTime := time.Now()
 	log.Info().Msg("Started generating pdf report")
 
-	summary := body.(model.Summary)
+	summary := body.(*model.Summary)
 	basePath, err := os.Getwd()
 	if err != nil {
 		return err
@@ -196,7 +196,7 @@ func PrintPdfReport(path, filename string, body interface{}) error {
 
 	m.SetBackgroundColor(color.NewWhite())
 
-	createFirstPageHeader(m, &summary)
+	createFirstPageHeader(m, summary)
 
 	m.Line(1.0)
 

diff --git a/pkg/report/pdf_test.go b/pkg/report/pdf_test.go
@@ -32,7 +32,7 @@ func TestPrintPdfReport(t *testing.T) {
 			if err := os.MkdirAll(test.caseTest.path, os.ModePerm); err != nil {
 				t.Fatal(err)
 			}
-			err := PrintPdfReport(test.caseTest.path, test.caseTest.filename, test.caseTest.summary)
+			err := PrintPdfReport(test.caseTest.path, test.caseTest.filename, &test.caseTest.summary)
 			checkFileExists(t, err, &test, "pdf")
 			os.RemoveAll(test.caseTest.path)
 		})