From 520aa679f34bafbc00626151075b0b123eceb516 Mon Sep 17 00:00:00 2001 From: xuri Date: Sun, 18 Oct 2020 00:01:33 +0800 Subject: Fix #706, #713 improve AddPicture performance, fix missing worksheet when rename with same names --- sheet_test.go | 27 +++++++++++++++++++++++++-- 1 file changed, 25 insertions(+), 2 deletions(-) (limited to 'sheet_test.go') diff --git a/sheet_test.go b/sheet_test.go index 890a4e5..56f5f46 100644 --- a/sheet_test.go +++ b/sheet_test.go @@ -300,7 +300,8 @@ func TestRemovePageBreak(t *testing.T) { } func TestGetSheetName(t *testing.T) { - f, _ := OpenFile(filepath.Join("test", "Book1.xlsx")) + f, err := OpenFile(filepath.Join("test", "Book1.xlsx")) + assert.NoError(t, err) assert.Equal(t, "Sheet1", f.GetSheetName(0)) assert.Equal(t, "Sheet2", f.GetSheetName(1)) assert.Equal(t, "", f.GetSheetName(-1)) @@ -312,10 +313,32 @@ func TestGetSheetMap(t *testing.T) { 1: "Sheet1", 2: "Sheet2", } - f, _ := OpenFile(filepath.Join("test", "Book1.xlsx")) + f, err := OpenFile(filepath.Join("test", "Book1.xlsx")) + assert.NoError(t, err) sheetMap := f.GetSheetMap() for idx, name := range sheetMap { assert.Equal(t, expectedMap[idx], name) } assert.Equal(t, len(sheetMap), 2) } + +func TestSetActiveSheet(t *testing.T) { + f := NewFile() + f.WorkBook.BookViews = nil + f.SetActiveSheet(1) + f.WorkBook.BookViews = &xlsxBookViews{WorkBookView: []xlsxWorkBookView{}} + f.Sheet["xl/worksheets/sheet1.xml"].SheetViews = &xlsxSheetViews{SheetView: []xlsxSheetView{}} + f.SetActiveSheet(1) + f.Sheet["xl/worksheets/sheet1.xml"].SheetViews = nil + f.SetActiveSheet(1) + f = NewFile() + f.SetActiveSheet(-1) + assert.Equal(t, f.GetActiveSheetIndex(), 0) +} + +func TestSetSheetName(t *testing.T) { + f := NewFile() + // Test set workksheet with the same name. + f.SetSheetName("Sheet1", "Sheet1") + assert.Equal(t, "Sheet1", f.GetSheetName(0)) +} -- cgit v1.2.1