From 2e791fa433def282ee2e7a5049a46fc4a76796cf Mon Sep 17 00:00:00 2001
From: xuri <xuri.me@gmail.com>
Date: Wed, 16 Oct 2019 01:03:29 +0800
Subject: Optimize code of Getting/Setting Page Margins

---
 sheetpr_test.go | 161 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++
 1 file changed, 161 insertions(+)

(limited to 'sheetpr_test.go')

diff --git a/sheetpr_test.go b/sheetpr_test.go
index 97a314c..d1ae2f1 100644
--- a/sheetpr_test.go
+++ b/sheetpr_test.go
@@ -146,3 +146,164 @@ func TestSheetPrOptions(t *testing.T) {
 		})
 	}
 }
+
+func TestSetSheetrOptions(t *testing.T) {
+	f := excelize.NewFile()
+	// Test SetSheetrOptions on not exists worksheet.
+	assert.EqualError(t, f.SetSheetPrOptions("SheetN"), "sheet SheetN is not exist")
+}
+
+func TestGetSheetPrOptions(t *testing.T) {
+	f := excelize.NewFile()
+	// Test GetSheetPrOptions on not exists worksheet.
+	assert.EqualError(t, f.GetSheetPrOptions("SheetN"), "sheet SheetN is not exist")
+}
+
+var _ = []excelize.PageMarginsOptions{
+	excelize.PageMarginBottom(1.0),
+	excelize.PageMarginFooter(1.0),
+	excelize.PageMarginHeader(1.0),
+	excelize.PageMarginLeft(1.0),
+	excelize.PageMarginRight(1.0),
+	excelize.PageMarginTop(1.0),
+}
+
+var _ = []excelize.PageMarginsOptionsPtr{
+	(*excelize.PageMarginBottom)(nil),
+	(*excelize.PageMarginFooter)(nil),
+	(*excelize.PageMarginHeader)(nil),
+	(*excelize.PageMarginLeft)(nil),
+	(*excelize.PageMarginRight)(nil),
+	(*excelize.PageMarginTop)(nil),
+}
+
+func ExampleFile_SetPageMargins() {
+	f := excelize.NewFile()
+	const sheet = "Sheet1"
+
+	if err := f.SetPageMargins(sheet,
+		excelize.PageMarginBottom(1.0),
+		excelize.PageMarginFooter(1.0),
+		excelize.PageMarginHeader(1.0),
+		excelize.PageMarginLeft(1.0),
+		excelize.PageMarginRight(1.0),
+		excelize.PageMarginTop(1.0),
+	); err != nil {
+		panic(err)
+	}
+	// Output:
+}
+
+func ExampleFile_GetPageMargins() {
+	f := excelize.NewFile()
+	const sheet = "Sheet1"
+
+	var (
+		marginBottom excelize.PageMarginBottom
+		marginFooter excelize.PageMarginFooter
+		marginHeader excelize.PageMarginHeader
+		marginLeft   excelize.PageMarginLeft
+		marginRight  excelize.PageMarginRight
+		marginTop    excelize.PageMarginTop
+	)
+
+	if err := f.GetPageMargins(sheet,
+		&marginBottom,
+		&marginFooter,
+		&marginHeader,
+		&marginLeft,
+		&marginRight,
+		&marginTop,
+	); err != nil {
+		panic(err)
+	}
+	fmt.Println("Defaults:")
+	fmt.Println("- marginBottom:", marginBottom)
+	fmt.Println("- marginFooter:", marginFooter)
+	fmt.Println("- marginHeader:", marginHeader)
+	fmt.Println("- marginLeft:", marginLeft)
+	fmt.Println("- marginRight:", marginRight)
+	fmt.Println("- marginTop:", marginTop)
+	// Output:
+	// Defaults:
+	// - marginBottom: 0.75
+	// - marginFooter: 0.3
+	// - marginHeader: 0.3
+	// - marginLeft: 0.7
+	// - marginRight: 0.7
+	// - marginTop: 0.75
+}
+
+func TestPageMarginsOption(t *testing.T) {
+	const sheet = "Sheet1"
+
+	testData := []struct {
+		container  excelize.PageMarginsOptionsPtr
+		nonDefault excelize.PageMarginsOptions
+	}{
+		{new(excelize.PageMarginTop), excelize.PageMarginTop(1.0)},
+		{new(excelize.PageMarginBottom), excelize.PageMarginBottom(1.0)},
+		{new(excelize.PageMarginLeft), excelize.PageMarginLeft(1.0)},
+		{new(excelize.PageMarginRight), excelize.PageMarginRight(1.0)},
+		{new(excelize.PageMarginHeader), excelize.PageMarginHeader(1.0)},
+		{new(excelize.PageMarginFooter), excelize.PageMarginFooter(1.0)},
+	}
+
+	for i, test := range testData {
+		t.Run(fmt.Sprintf("TestData%d", i), func(t *testing.T) {
+
+			opt := test.nonDefault
+			t.Logf("option %T", opt)
+
+			def := deepcopy.Copy(test.container).(excelize.PageMarginsOptionsPtr)
+			val1 := deepcopy.Copy(def).(excelize.PageMarginsOptionsPtr)
+			val2 := deepcopy.Copy(def).(excelize.PageMarginsOptionsPtr)
+
+			f := excelize.NewFile()
+			// Get the default value
+			assert.NoError(t, f.GetPageMargins(sheet, def), opt)
+			// Get again and check
+			assert.NoError(t, f.GetPageMargins(sheet, val1), opt)
+			if !assert.Equal(t, val1, def, opt) {
+				t.FailNow()
+			}
+			// Set the same value
+			assert.NoError(t, f.SetPageMargins(sheet, val1), opt)
+			// Get again and check
+			assert.NoError(t, f.GetPageMargins(sheet, val1), opt)
+			if !assert.Equal(t, val1, def, "%T: value should not have changed", opt) {
+				t.FailNow()
+			}
+			// Set a different value
+			assert.NoError(t, f.SetPageMargins(sheet, test.nonDefault), opt)
+			assert.NoError(t, f.GetPageMargins(sheet, val1), opt)
+			// Get again and compare
+			assert.NoError(t, f.GetPageMargins(sheet, val2), opt)
+			if !assert.Equal(t, val1, val2, "%T: value should not have changed", opt) {
+				t.FailNow()
+			}
+			// Value should not be the same as the default
+			if !assert.NotEqual(t, def, val1, "%T: value should have changed from default", opt) {
+				t.FailNow()
+			}
+			// Restore the default value
+			assert.NoError(t, f.SetPageMargins(sheet, def), opt)
+			assert.NoError(t, f.GetPageMargins(sheet, val1), opt)
+			if !assert.Equal(t, def, val1) {
+				t.FailNow()
+			}
+		})
+	}
+}
+
+func TestSetPageMargins(t *testing.T) {
+	f := excelize.NewFile()
+	// Test set page margins on not exists worksheet.
+	assert.EqualError(t, f.SetPageMargins("SheetN"), "sheet SheetN is not exist")
+}
+
+func TestGetPageMargins(t *testing.T) {
+	f := excelize.NewFile()
+	// Test get page margins on not exists worksheet.
+	assert.EqualError(t, f.GetPageMargins("SheetN"), "sheet SheetN is not exist")
+}
-- 
cgit v1.2.1