6
86%
main: 86%

Ran 24 Jun 2026 02:00PM UTC

Files 15

Run time 1s

Badge

Embed ▾

Committed 24 Jun 2026 01:56PM UTC coverage: 82.033%. Remained the same

Job # 28103817248.6

Build Type

push

github

Committed by

web-flow

Commit Message

refactor(storage_client): dedupe retry assertion and fetch options (#1462)

## What

Two pieces of repetition in `StorageFileApi` (`storage_file_api.dart`):

- The `assert(retryAttempts == null || retryAttempts >= 0, ...)` check
was duplicated across 6 methods → extracted into
`_assertValidRetryAttempts`.
- `FetchOptions(headers: headers)` was constructed in 13 places →
extracted into a `_fetchOptions` getter.

No behavior change. `FetchOptions` is immutable (all-final, `const`
constructor) and the `options` value is never mutated, so the getter is
semantically identical to the previous per-call construction.

## Verification

- `dart analyze lib` and `dcm analyze` pass with no issues.
- All storage_client unit tests pass. (The integration tests fail
locally only because they need a running storage server: `Connection
refused`.)

---
**Note:** This branch also removes two stale
`avoid-unnecessary-type-assertions` ignores in `postgrest_builder.dart`.
They are flagged as unused by the DCM version used in CI (a pre-existing
failure on `main`, present on every PR). Including it keeps this PR's
pipeline green; it drops out cleanly once another PR carrying the same
fix lands.

Coverage Stats

347 of 423 relevant lines covered (82.03%)

2.32 hits per line

supabase / supabase-flutter / 28103817248 / 6
86%
main: 86%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job supabase - 28103817248.6

supabase / supabase-flutter / 28103817248 / 6 86% main: 86%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job supabase - 28103817248.6

supabase / supabase-flutter / 28103817248 / 6
86%
main: 86%

README BADGES
x