Merge pull request #18 from essentialkaos/develop

Version 1.4.3
essentialkaos · Jan 19, 2016 · 2e3643b · 2e3643b
2 parents 7527f47 + c1c0a91
commit 2e3643b
Show file tree

Hide file tree

Showing 3 changed files with 13 additions and 3 deletions.
diff --git a/changelog.md b/changelog.md
@@ -1,5 +1,9 @@
 ## Changelog
 
+#### v1.4.3
+
+* `[errutil]` Improved Add method
+
 #### v1.4.2
 
 * `[fsutil]` Added method `ProperPath` which return first proper path from given slice

diff --git a/errutil/errutil.go b/errutil/errutil.go
@@ -23,12 +23,16 @@ func NewErrors() *Errors {
 // ////////////////////////////////////////////////////////////////////////////////// //
 
 // Add adds new error to slice
-func (e *Errors) Add(err error) *Errors {
-	if err == nil {
+func (e *Errors) Add(errs ...error) *Errors {
+	if errs == nil || len(errs) == 0 {
 		return e
 	}
 
-	e.errors = append(e.errors, err)
+	for _, err := range errs {
+		if err != nil {
+			e.errors = append(e.errors, err)
+		}
+	}
 
 	return e
 }

diff --git a/errutil/errutil_test.go b/errutil/errutil_test.go
@@ -29,6 +29,8 @@ var _ = Suite(&ErrSuite{})
 func (s *ErrSuite) TestPositive(c *C) {
 	errs := NewErrors()
 
+	errs.Add()
+	errs.Add(nil)
 	errs.Add(errors.New("1"))
 	errs.Add(errors.New("2"))
 	errs.Add(errors.New("3"))