Merge pull request #9 from stephens2424/master

ignoring trailing whitespace. fixes #8

Author: fatih

tags.go

@@ -42,6 +42,8 @@ type Tag struct {
 func Parse(tag string) (*Tags, error) {
 	var tags []*Tag
 
+	hasTag := tag != ""
+
 	// NOTE(arslan) following code is from reflect and vet package with some
 	// modifications to collect all necessary information and extend it with
 	// usable methods
@@ -53,7 +55,7 @@ func Parse(tag string) (*Tags, error) {
 		}
 		tag = tag[i:]
 		if tag == "" {
-			return nil, nil
+			break
 		}
 
 		// Scan to colon. A space, a quote or a control character is a syntax
@@ -113,6 +115,10 @@ func Parse(tag string) (*Tags, error) {
 		})
 	}
 
+	if hasTag && len(tags) == 0 {
+		return nil, nil
+	}
+
 	return &Tags{
 		tags: tags,
 	}, nil

tags_test.go

@@ -3,6 +3,7 @@ package structtag
 import (
 	"reflect"
 	"sort"
+	"strings"
 	"testing"
 )
 
@@ -162,6 +163,16 @@ func TestParse(t *testing.T) {
 				},
 			},
 		},
+		{
+			name: "tag with trailing space",
+			tag:  `json:"foo" `,
+			exp: []*Tag{
+				{
+					Key:  "json",
+					Name: "foo",
+				},
+			},
+		},
 	}
 
 	for _, ts := range test {
@@ -182,8 +193,9 @@ func TestParse(t *testing.T) {
 				t.Errorf("parse\n\twant: %#v\n\tgot : %#v", ts.exp, got)
 			}
 
-			if ts.tag != tags.String() {
-				t.Errorf("parse string\n\twant: %#v\n\tgot : %#v", ts.tag, tags.String())
+			trimmedInput := strings.TrimSpace(ts.tag)
+			if trimmedInput != tags.String() {
+				t.Errorf("parse string\n\twant: %#v\n\tgot : %#v", trimmedInput, tags.String())
 			}
 		})
 	}