supabase / supabase-flutter / 38905844892

92%

Build Type

push

github

Committed by web-flow

Commit Message

fix(auth): check persistSession up front and give examples a storage (#1972)

Follow-up to #1971 for
[SDK-2259](https://linear.app/supabase/issue/SDK-2259).

## Changes

- `SupabaseClient` now also catches `persistSession: true` without an
`asyncStorage` up front, with its own message pointing at `authOptions`
or `persistSession: false`. Before, that case fell through to the
`AuthClient` message, which doesn't mention `SupabaseClient`, after the
default HTTP client had already been created.
- Removes the JSON codec dispose before the throw. `YAJsonIsolate`
doesn't spawn a long-lived isolate, so there was nothing to clean up.
- The `supabase` web example and the second client in the realtime room
integration test now pass a `MemoryAuthAsyncStorage`. Since #1971, both
throw when they are created.
- Adds tests for the persisted-session error and for the `AuthException`
from `exchangeCodeForSession` on an implicit-flow client without a
storage.
- Updates the `SupabaseClient` dartdoc and the MIGRATION.md section to
match.

Coverage Stats

6 of 6 new or added lines in 1 file covered. (100.0%)

36 existing lines in 2 files now uncovered.

9661 of 10487 relevant lines covered (92.12%)

5.48 hits per line