supabase / gotrue / 7227105859

65%

Build Type

push

github

Committed by web-flow

Commit Message

fix: handle oauth email check separately (#1348)

## What kind of change does this PR introduce?
* Instead of returning an error in every provider if the email doesn't
exist, we check if the email exists in
`internalExternalProviderCallback` instead
* Setting the `Email` and `EmailVerified` claims in the `Claims` struct
is redundant since we are tracking it separately in `Emails`.
 
```go
type UserProvidedData struct {
	Emails   []Email
	Metadata *Claims
}
```

* Fix buggy oauth tests that appear to pass but they were actually
returning the wrong value / wrong expected error
* This PR lays the foundation for removing the dependency on the email
returned by the oauth provider

Run Details

244 of 331 new or added lines in 19 files covered. (73.72%)

21 existing lines in 8 files now uncovered.

7927 of 12159 relevant lines covered (65.19%)

55.24 hits per line