@@ -97,7 +97,7 @@ func TestGitHubProviderOverrides(t *testing.T) {
9797}
9898
9999func TestGitHubProviderGetEmailAddress(t *testing.T) {
100− b := testGitHubBackend([]string{`[ {"email": "michael.bland@gsa.gov", "primary": true} ]`})
100+ b := testGitHubBackend([]string{`[ {"email": "michael.bland@gsa.gov", "verified": true, "primary": true} ]`})
101101 defer b.Close()
102102
103103 bURL, _ := url.Parse(b.URL)
@@ -109,10 +109,23 @@ func TestGitHubProviderGetEmailAddress(t *testing.T) {
109109 assert.Equal(t, "michael.bland@gsa.gov", email)
110110}
111111
112+func TestGitHubProviderGetEmailAddressNotVerified(t *testing.T) {
113+ b := testGitHubBackend([]string{`[ {"email": "michael.bland@gsa.gov", "verified": false, "primary": true} ]`})
114+ defer b.Close()
115+
116+ bURL, _ := url.Parse(b.URL)
117+ p := testGitHubProvider(bURL.Host)
118+
119+ session := &SessionState{AccessToken: "imaginary_access_token"}
120+ email, err := p.GetEmailAddress(session)
121+ assert.Equal(t, nil, err)
122+ assert.Empty(t, "", email)
123+}
124+
112125func TestGitHubProviderGetEmailAddressWithOrg(t *testing.T) {
113126 b := testGitHubBackend([]string{
114− `[ {"email": "michael.bland@gsa.gov", "primary": true, "login":"testorg"} ]`,
115− `[ {"email": "michael.bland1@gsa.gov", "primary": true, "login":"testorg1"} ]`,
127+ `[ {"email": "michael.bland@gsa.gov", "primary": true, "verified": true, "login":"testorg"} ]`,
128+ `[ {"email": "michael.bland1@gsa.gov", "primary": true, "verified": true, "login":"testorg1"} ]`,
116129 `[ ]`,
117130 })
118131 defer b.Close()