From 0a0a36bafd160752c2af914ab3bdb1d28665ebe3 Mon Sep 17 00:00:00 2001
From: Ri Xu <xuri.me@gmail.com>
Date: Fri, 2 Sep 2016 10:28:29 +0800
Subject: Update testing file more case added and reading cell value usage
 example in readme added.

---
 README.md           |  22 +++++++++++++++++++++-
 excelize_test.go    |  19 +++++++++++++++----
 file.go             |   3 +--
 test/Workbook1.xlsx | Bin 19085 -> 18199 bytes
 4 files changed, 37 insertions(+), 7 deletions(-)

diff --git a/README.md b/README.md
index 351cd61..3614490 100644
--- a/README.md
+++ b/README.md
@@ -69,10 +69,30 @@ func main() {
     xlsx = excelize.SetCellInt(xlsx, "Sheet3", "A23", 10)
     xlsx = excelize.SetCellStr(xlsx, "Sheet3", "b230", "World")
     xlsx = excelize.SetActiveSheet(xlsx, 2)
+    err = excelize.Save(xlsx, "~/Workbook.xlsx")
+        if err != nil {
+        fmt.Println(err)
+    }
+}
+```
+
+### Reading XLSX files
+
+```
+package main
+
+import (
+    "fmt"
+    "github.com/Luxurioust/excelize"
+)
+
+func main() {
+    xlsx, err := excelize.OpenFile("~/Workbook.xlsx")
     if err != nil {
         fmt.Println(err)
     }
-    err = excelize.Save(xlsx, "~/Workbook.xlsx")
+    cell := excelize.GetCellValue(file, "Sheet2", "D11")
+    fmt.Println(cell)
 }
 ```
 
diff --git a/excelize_test.go b/excelize_test.go
index 69f7964..456455e 100644
--- a/excelize_test.go
+++ b/excelize_test.go
@@ -9,7 +9,7 @@ func TestExcelize(t *testing.T) {
 	// Test update a XLSX file
 	file, err := OpenFile("./test/Workbook1.xlsx")
 	if err != nil {
-		t.Error(err)
+		t.Log(err)
 	}
 	file = SetCellInt(file, "SHEET2", "B2", 100)
 	file = SetCellStr(file, "SHEET2", "C11", "Knowns")
@@ -18,12 +18,21 @@ func TestExcelize(t *testing.T) {
 	file = SetCellStr(file, "SHEET3", "b230", "10")
 	file = SetActiveSheet(file, 2)
 	if err != nil {
-		t.Error(err)
+		t.Log(err)
 	}
 	for i := 1; i <= 300; i++ {
 		file = SetCellStr(file, "SHEET3", "c"+strconv.Itoa(i), strconv.Itoa(i))
 	}
 	err = Save(file, "./test/Workbook_2.xlsx")
+	if err != nil {
+		t.Log(err)
+	}
+
+	// Test save to not exist directory
+	err = Save(file, "")
+	if err != nil {
+		t.Log(err)
+	}
 
 	// Test create a XLSX file
 	file2 := CreateFile()
@@ -31,18 +40,20 @@ func TestExcelize(t *testing.T) {
 	file2 = NewSheet(file2, 3, "TestSheet3")
 	file2 = SetCellInt(file2, "Sheet2", "A23", 10)
 	file2 = SetCellStr(file2, "SHEET1", "B20", "10")
+	file2 = SetActiveSheet(file2, 0)
 	err = Save(file2, "./test/Workbook_3.xlsx")
 	if err != nil {
-		t.Error(err)
+		t.Log(err)
 	}
 
 	// Test read cell value
 	file, err = OpenFile("./test/Workbook1.xlsx")
 	if err != nil {
-		t.Error(err)
+		t.Log(err)
 	}
 	GetCellValue(file, "Sheet2", "a-1")
 	GetCellValue(file, "Sheet2", "a5")
+	GetCellValue(file, "Sheet2", "C11")
 	GetCellValue(file, "Sheet2", "D11")
 	GetCellValue(file, "Sheet2", "D12")
 	GetCellValue(file, "Sheet2", "E12")
diff --git a/file.go b/file.go
index bdefc3b..d7123d0 100644
--- a/file.go
+++ b/file.go
@@ -3,7 +3,6 @@ package excelize
 import (
 	"archive/zip"
 	"bytes"
-	"fmt"
 	"os"
 )
 
@@ -34,7 +33,7 @@ func Save(files []FileList, name string) error {
 	for _, file := range files {
 		f, err := w.Create(file.Key)
 		if err != nil {
-			fmt.Println(err)
+			return err
 		}
 		_, err = f.Write([]byte(file.Value))
 		if err != nil {
diff --git a/test/Workbook1.xlsx b/test/Workbook1.xlsx
index cbdeda9..54810bc 100644
Binary files a/test/Workbook1.xlsx and b/test/Workbook1.xlsx differ
-- 
cgit v1.2.1