From 5dd00b9a004d5a7bc867ed09387cbd88fed240e4 Mon Sep 17 00:00:00 2001
From: "Michael W. Mitton" <mmitton@yssd.org>
Date: Wed, 9 Jan 2019 10:12:53 -0500
Subject: Do not create a blank fill if no fill is specified in the style
 format

---
 styles_test.go | 34 ++++++++++++++++++++++++++++++++++
 1 file changed, 34 insertions(+)

(limited to 'styles_test.go')

diff --git a/styles_test.go b/styles_test.go
index baa66f0..c6fbbef 100644
--- a/styles_test.go
+++ b/styles_test.go
@@ -6,6 +6,40 @@ import (
 	"github.com/stretchr/testify/assert"
 )
 
+func TestStyleFill(t *testing.T) {
+	cases := []struct {
+		label      string
+		format     string
+		expectFill bool
+	}{{
+		label:      "no_fill",
+		format:     `{"alignment":{"wrap_text":true}}`,
+		expectFill: false,
+	}, {
+		label:      "fill",
+		format:     `{"fill":{"type":"pattern","pattern":1,"color":["#000000"]}}`,
+		expectFill: true,
+	}}
+
+	for _, testCase := range cases {
+		xl := NewFile()
+		const sheet = "Sheet1"
+
+		styleID, err := xl.NewStyle(testCase.format)
+		if err != nil {
+			t.Fatalf("%v", err)
+		}
+
+		styles := xl.stylesReader()
+		style := styles.CellXfs.Xf[styleID]
+		if testCase.expectFill {
+			assert.NotEqual(t, style.FillID, 0, testCase.label)
+		} else {
+			assert.Equal(t, style.FillID, 0, testCase.label)
+		}
+	}
+}
+
 func TestSetConditionalFormat(t *testing.T) {
 	cases := []struct {
 		label  string
-- 
cgit v1.2.1