From 0faac08237ea02654b0a5ce1be9407bb479ce061 Mon Sep 17 00:00:00 2001
From: Julien Neuhart <neuhart.julien@gmail.com>
Date: Wed, 24 Nov 2021 16:39:21 +0100
Subject: [PATCH] chore(unoconv): improve code coverage

---
 pkg/modules/libreoffice/unoconv/unoconv_test.go | 7 ++++++-
 1 file changed, 6 insertions(+), 1 deletion(-)

diff --git a/pkg/modules/libreoffice/unoconv/unoconv_test.go b/pkg/modules/libreoffice/unoconv/unoconv_test.go
index 5d72d479..6395c667 100644
--- a/pkg/modules/libreoffice/unoconv/unoconv_test.go
+++ b/pkg/modules/libreoffice/unoconv/unoconv_test.go
@@ -89,15 +89,18 @@ func TestUnoconv_Unoconv(t *testing.T) {
 func TestUnoconv_PDF(t *testing.T) {
 	for i, tc := range []struct {
 		ctx       context.Context
+		logger    *zap.Logger
 		inputPath string
 		options   Options
 		expectErr bool
 	}{
 		{
+			logger:    zap.NewNop(),
 			expectErr: true,
 		},
 		{
 			ctx:       context.Background(),
+			logger:    zap.NewExample(),
 			inputPath: "/tests/test/testdata/libreoffice/sample1.docx",
 			options: Options{
 				Landscape:  true,
@@ -107,6 +110,7 @@ func TestUnoconv_PDF(t *testing.T) {
 		},
 		{
 			ctx:       context.Background(),
+			logger:    zap.NewNop(),
 			inputPath: "/tests/test/testdata/libreoffice/sample1.docx",
 			options: Options{
 				PageRanges: "foo",
@@ -120,6 +124,7 @@ func TestUnoconv_PDF(t *testing.T) {
 
 				return ctx
 			}(),
+			logger:    zap.NewNop(),
 			inputPath: "/tests/test/testdata/libreoffice/sample1.docx",
 			expectErr: true,
 		},
@@ -144,7 +149,7 @@ func TestUnoconv_PDF(t *testing.T) {
 				}
 			}()
 
-			err = mod.PDF(tc.ctx, zap.NewNop(), tc.inputPath, outputDir+"/foo.pdf", tc.options)
+			err = mod.PDF(tc.ctx, tc.logger, tc.inputPath, outputDir+"/foo.pdf", tc.options)
 
 			if tc.expectErr && err == nil {
 				t.Errorf("test %d: expected error but got: %v", i, err)