Added tests to cover the new options

Also updated/refactored the table test a little so they always output which test is the failing test.
hkant26 · Jan 24, 2017 · 96b9d08 · 96b9d08
1 parent d552393
commit 96b9d08
Show file tree

Hide file tree

Showing 2 changed files with 355 additions and 226 deletions.
diff --git a/dataframe/dataframe.go b/dataframe/dataframe.go
@@ -236,7 +236,7 @@ func (df DataFrame) print(
 			notShownArr = append(notShownArr, notShowing[i:len(notShowing)])
 		}
 		for k, ns := range notShownArr {
-			notShown += fmt.Sprintf("%s", strings.Join(ns, ", "))
+			notShown += strings.Join(ns, ", ")
 			if k != len(notShownArr)-1 {
 				notShown += ","
 			}
@@ -708,7 +708,7 @@ func HasHeader(b bool) LoadOption {
 }
 
 // Names sets the names option for loadOptions.
-func Names(names []string) LoadOption {
+func Names(names ...string) LoadOption {
 	return func(c *loadOptions) {
 		c.names = names
 	}