supabase / supabase-flutter / 27767998227 / 7
85%
main: 85%

DEFAULT BRANCH: main
Ran
Files 2
Run time 0s
Badge
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst
coverage: 97.101%. Remained the same
27767998227.7

push

github

web-flow 
chore(supabase_lints): enable avoid-misused-test-matchers (#1424)

Stacked on #1423.

## What

Enables the `avoid-misused-test-matchers` DCM rule by fixing all 31
flagged sites. Two patterns:

### `throwsA` on an already-evaluated builder (7 sites, retry_test)
`expectLater(client.from('users').select(), throwsA(...))` passed the
already-built `PostgrestBuilder` to `throwsA`. Wrapped the targets in
closures (`() => client.from('users').select()`) so `throwsA` defers the
call. Behaviorally equivalent (the builder is lazy), and the correct
idiom.

### Vacuous `expect(x, isNotNull)` on non-nullable values (24 sites)
The target types are non-nullable, so the assertion always passed. Each
was either:
- **Replaced** with a meaningful check: `isNotEmpty` (error message,
user id, token, claims), `greaterThanOrEqualTo(0)` (count queries),
`same(Supabase.instance.client)` (singleton identity).
- **Removed** when the following line already asserted the value (e.g.
`expect(res['type'], 'FeatureCollection')`), or when it was a
post-`initialize` smoke test whose value is "init did not throw".

## Verification
- `dcm analyze packages` -> no issues found.
- `dart analyze` clean across affected packages (lib + test).
- `flutter analyze --fatal-warnings` clean on supabase_flutter.
- Tests: postgrest retry_test 12 pass, supabase_flutter 49 pass.
Integration suites that need a local server are unaffected (assertion
swaps compile and are equivalent).

67 of 69 relevant lines covered (97.1%)

1.59 hits per line

Source Files on job functions_client - 27767998227.7