Merge pull request #6 from klaidliadon/v2.0.0

Timezone fixes

Author: mcuadros

objects.go

@@ -76,7 +76,15 @@ func (s *Signature) Decode(b []byte) {
 			if c == ' ' || end {
 				t, err := strconv.ParseInt(string(b[from:i]), 10, 64)
 				if err == nil {
-					s.When = time.Unix(t, 0)
+					loc := time.UTC
+					ts := time.Unix(t, 0)
+					if len(b[i:]) >= 6 {
+						tl, err := time.Parse(" -0700", string(b[i:i+6]))
+						if err == nil {
+							loc = tl.Location()
+						}
+					}
+					s.When = ts.In(loc)
 				}
 				end = true
 			}

objects_test.go

@@ -1,6 +1,7 @@
 package git
 
 import (
+	"fmt"
 	"io/ioutil"
 	"time"
 
@@ -42,7 +43,7 @@ func (s *ObjectsSuite) TestNewCommit(c *C) {
 
 	c.Assert(commit.Author.Email, Equals, "[email protected]")
 	c.Assert(commit.Author.Name, Equals, "Máximo Cuadros")
-	c.Assert(commit.Author.When.Unix(), Equals, int64(1427802434))
+	c.Assert(commit.Author.When.Format(time.RFC3339), Equals, "2015-03-31T13:47:14+02:00")
 	c.Assert(commit.Committer.Email, Equals, "[email protected]")
 	c.Assert(commit.Message, Equals, "Merge pull request #1 from dripolles/feature\n\nCreating changelog\n")
 }
@@ -92,17 +93,22 @@ func (s *ObjectsSuite) TestParseSignature(c *C) {
 		`Foo Bar <[email protected]> 1257894000 +0100`: {
 			Name:  "Foo Bar",
 			Email: "[email protected]",
-			When:  time.Unix(1257894000, 0),
+			When:  MustParseTime("2009-11-11 00:00:00 +0100"),
+		},
+		`Foo Bar <[email protected]> 1257894000 -0700`: {
+			Name:  "Foo Bar",
+			Email: "[email protected]",
+			When:  MustParseTime("2009-11-10 16:00:00 -0700"),
 		},
 		`Foo Bar <> 1257894000 +0100`: {
 			Name:  "Foo Bar",
 			Email: "",
-			When:  time.Unix(1257894000, 0),
+			When:  MustParseTime("2009-11-11 00:00:00 +0100"),
 		},
 		` <> 1257894000`: {
 			Name:  "",
 			Email: "",
-			When:  time.Unix(1257894000, 0),
+			When:  MustParseTime("2009-11-10 23:00:00 +0000"),
 		},
 		`Foo Bar <[email protected]>`: {
 			Name:  "Foo Bar",
@@ -122,11 +128,17 @@ func (s *ObjectsSuite) TestParseSignature(c *C) {
 	}
 
 	for raw, exp := range cases {
+		fmt.Println("> testing", raw)
 		got := &Signature{}
 		got.Decode([]byte(raw))
 
 		c.Assert(got.Name, Equals, exp.Name)
 		c.Assert(got.Email, Equals, exp.Email)
-		c.Assert(got.When.Unix(), Equals, exp.When.Unix())
+		c.Assert(got.When.Format(time.RFC3339), Equals, exp.When.Format(time.RFC3339))
 	}
 }
+
+func MustParseTime(value string) time.Time {
+	t, _ := time.Parse("2006-01-02 15:04:05 -0700", value)
+	return t
+}