Preston_PLB
/
oauth2
forked from Mirrors/oauth2
1
0
Fork 0
Code Pull Requests Activity

oauth2: fix test to work with Go tip also 

json.UnmarshalError has a new field in Go 1.5. Adjust tests to cope.

Change-Id: I6733b2e14513794676e7329a828001f3f8c6c342
Reviewed-on: https://go-review.googlesource.com/8341
Reviewed-by: Burcu Dogan <jbd@google.com>
This commit is contained in:
Brad Fitzpatrick 2015-04-01 17:31:48 +02:00
parent 3046bc76d6
commit c58fcf0ffc
1 changed files with 19 additions and 15 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

28
oauth2_test.go
View File

@ -184,9 +184,9 @@ func TestExchangeRequest_JSONResponse_Expiry(t *testing.T) {
{fmt.Sprintf(`"expires_in": %d`, seconds), nil}, {fmt.Sprintf(`"expires_in": %d`, seconds), nil},
{fmt.Sprintf(`"expires_in": "%d"`, seconds), nil}, // PayPal case {fmt.Sprintf(`"expires_in": "%d"`, seconds), nil}, // PayPal case
{fmt.Sprintf(`"expires": %d`, seconds), nil}, // Facebook case {fmt.Sprintf(`"expires": %d`, seconds), nil}, // Facebook case
{`"expires": false`, &json.UnmarshalTypeError{"bool", jsonNumberType}}, // wrong type {`"expires": false`, &json.UnmarshalTypeError{Value: "bool", Type: jsonNumberType}}, // wrong type
{`"expires": {}`, &json.UnmarshalTypeError{"object", jsonNumberType}}, // wrong type {`"expires": {}`, &json.UnmarshalTypeError{Value: "object", Type: jsonNumberType}}, // wrong type
{`"expires": "zzz"`, &strconv.NumError{"ParseInt", "zzz", strconv.ErrSyntax}}, // wrong value {`"expires": "zzz"`, &strconv.NumError{Func: "ParseInt", Num: "zzz", Err: strconv.ErrSyntax}}, // wrong value
} { } {
testExchangeRequest_JSONResponse_expiry(t, c.expires, c.expect) testExchangeRequest_JSONResponse_expiry(t, c.expires, c.expect)
} }
@ -202,16 +202,20 @@ func testExchangeRequest_JSONResponse_expiry(t *testing.T, exp string, expect er
t1 := time.Now().Add(day) t1 := time.Now().Add(day)
tok, err := conf.Exchange(NoContext, "exchange-code") tok, err := conf.Exchange(NoContext, "exchange-code")
t2 := time.Now().Add(day) t2 := time.Now().Add(day)
if err == nil && expect != nil { // Do a fmt.Sprint comparison so either side can be
t.Errorf("Incorrect state, conf.Exchange() should return an error: %v", expect) // nil. fmt.Sprint just stringifies them to "<nil>", and no
} else if err != nil { // non-nil expected error ever stringifies as "<nil>", so this
if reflect.DeepEqual(err, expect) { // isn't terribly disgusting. We do this because Go 1.4 and
t.Logf("Expected error: %v", err) // Go 1.5 return a different deep value for
return // json.UnmarshalTypeError. In Go 1.5, the
} else { // json.UnmarshalTypeError contains a new field with a new
t.Error(err) // non-zero value. Rather than ignore it here with reflect or
// add new files and +build tags, just look at the strings.
if fmt.Sprint(err) != fmt.Sprint(expect) {
t.Errorf("Error = %v; want %v", err, expect)
} }
if err != nil {
return
} }
if !tok.Valid() { if !tok.Valid() {
t.Fatalf("Token invalid. Got: %#v", tok) t.Fatalf("Token invalid. Got: %#v", tok)