Setting email for OpenShift connector

sabre1041 · sabre1041 · commit 76bb453ff329 · 2020-02-21T16:53:46.000+01:00
diff --git a/connector/openshift/openshift.go b/connector/openshift/openshift.go
@@ -177,6 +177,7 @@ func (c *openshiftConnector) HandleCallback(s connector.Scopes, r *http.Request)
 		UserID:            user.UID,
 		Username:          user.Name,
 		PreferredUsername: user.Name,
+		Email:             user.Name,
 		Groups:            user.Groups,
 	}
 
diff --git a/connector/openshift/openshift_test.go b/connector/openshift/openshift_test.go
@@ -180,6 +180,7 @@ func TestCallbackIdentity(t *testing.T) {
 	expectEquals(t, identity.UserID, "12345")
 	expectEquals(t, identity.Username, "jdoe")
 	expectEquals(t, identity.PreferredUsername, "jdoe")
+	expectEquals(t, identity.Email, "jdoe")
 	expectEquals(t, len(identity.Groups), 1)
 	expectEquals(t, identity.Groups[0], "users")
 }

Original file line number	Diff line number	Diff line change
`@@ -177,6 +177,7 @@ func (c openshiftConnector) HandleCallback(s connector.Scopes, r http.Request)`
`177`	`177`	`UserID: user.UID,`
`178`	`178`	`Username: user.Name,`
`179`	`179`	`PreferredUsername: user.Name,`
	`180`	`+ Email: user.Name,`
`180`	`181`	`Groups: user.Groups,`
`181`	`182`	`}`
`182`	`183`
Original file line number	Diff line number	Diff line change
`@@ -180,6 +180,7 @@ func TestCallbackIdentity(t *testing.T) {`
`180`	`180`	`expectEquals(t, identity.UserID, "12345")`
`181`	`181`	`expectEquals(t, identity.Username, "jdoe")`
`182`	`182`	`expectEquals(t, identity.PreferredUsername, "jdoe")`
	`183`	`+ expectEquals(t, identity.Email, "jdoe")`
`183`	`184`	`expectEquals(t, len(identity.Groups), 1)`
`184`	`185`	`expectEquals(t, identity.Groups[0], "users")`
`185`	`186`	`}`