supabase / supabase-swift / 31011736834

84%

Build Type

push

github

Committed by web-flow

Commit Message

refactor(auth): deprecate customizable JSON encoder/decoder (#1165)

* refactor(auth): deprecate customizable JSON encoder/decoder

There is no real use case for customizing Auth's JSON encoding/decoding
per-instance. Deprecate `Configuration.encoder`/`.decoder` and the
init overloads that accept them, in favor of new inits that always use
the internal default. Behavior is unchanged for now (deprecated inits
still honor a custom encoder/decoder) -- this only starts the removal
path for a future major version.

Internal call sites now read a non-deprecated `resolvedEncoder`/
`resolvedDecoder` pair instead of the deprecated public properties, to
keep the SDK's own code warning-free.

* fix(auth): restore single-codec deprecated init overloads

The new encoder/decoder-accepting init required both arguments, so a
caller previously passing only encoder: (or only decoder:) on the
fuller-signature init (with logger/redirectToURL/etc.) no longer
compiled. Add encoder-only and decoder-only deprecated overloads,
alongside the existing both-codecs one, each defaulting the omitted
codec. Add compile coverage for all four single-codec call forms.

* fix(ci): add "initializers" to spell-check dictionary

Plural form was missing from all bundled cspell dictionaries, failing
CI on Sources/Auth/AuthClientConfiguration.swift.

* fix(ci): disable parallel test execution in xcodebuild test runs

withMainSerialExecutor (ConcurrencyExtras) hooks a process-wide task-enqueue
global, not just the calling suite's own tasks. Swift Testing's default
parallel execution runs unrelated suites concurrently in the same process, so
while any withMainSerialExecutor-gated suite (AuthClientTests, SessionManagerTests,
RealtimeTests, RealtimeIntegrationTests) holds that hook, every other
concurrently-running suite's tasks get silently forced onto the same serial
queue too. Under CI's loaded iOS Simulator this backs up enough to blow
through hardcoded test timeouts (e.g. AuthClientTests.ass... (continued)

Coverage Stats

87 of 93 new or added lines in 11 files covered. (93.55%)

27 existing lines in 4 files now uncovered.

8452 of 10080 relevant lines covered (83.85%)

40.22 hits per line