supabase / supabase-flutter / 29102073868 / 7
87%
main: 87%

DEFAULT BRANCH: main
Ran
Files 1
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: 77.778%. Remained the same
29102073868.7

push

github

web-flow 
test: remove abbreviations and code smells across package tests (#1584)

## What

Cleans up the test suites across every package. No production (`lib/`)
changes.

- **Abbreviations removed** in test-local identifiers: `res`/`getRes`/…
→ `response`/`result`, `req` → `request`, `params` → `parameters`, `msg`
→ `message`, `e`/`err` → `error`, `conn` → `connection`, `db` →
`database`, `at` → `accessToken`, `sig` → `signature`, `sub` →
`subscription`, and similar. Public API names, named arguments, JSON/map
keys and conventional acronyms (id, url, jwt, otp, pkce, ws, …) are left
untouched.
- **Weak assertions strengthened** with proper matchers: `isNotNull`,
`isEmpty`, `isTrue`/`isFalse`, `hasLength`, `contains`, `isA<T>()`,
`endsWith`, `isNot(contains(...))`. Redundant `.toString()`/casts and a
pointless cascade removed.
- **Idiomatic throw assertions**: converted `try { await op } catch (e)
{ expect(e, isA<X>()) }` and closure-wrapped `expectLater(() => future,
throwsA(...))` to `expectLater(future, throwsA(...))` (69 sites).
Genuinely synchronous `expect(() => syncCall(), throwsX)` and mockito
`when`/`verify` closures were left as-is.
- **Stale phone-auth tests fixed**: the two gotrue "should fail because
Twilio is not setup" tests were silently passing as no-ops (a
`try/catch` swallowed the non-exception). The test stack deliberately
enables phone auth, so they now assert the real success behavior.
- **Useless/duplicate tests removed (7)**:
- gotrue: four identical `same()` reference-equality tautologies
(`session`, `auth_exception`, `user`, `user_identity`).
- realtime: `socket_test` "calls callback" (incremented a counter
unconditionally, asserted nothing real).
- postgrest: "bulk insert with one row uses default value" (exact
duplicate).
- storage: "getPublicUrl leaves the URL unchanged when download is null"
(byte-for-byte duplicate).

## Testing

All package suites pass against a local Supabase stack: gotrue 446,
realtime 182, postgrest 177, stora... (continued)

49 of 63 relevant lines covered (77.78%)

1.27 hits per line

Source Files on job yet_another_json_isolate - 29102073868.7