supabase / supabase-flutter / 27768085173
85%

DEFAULT BRANCH: main
Ran
Jobs 8
Files 78
Run time 1min
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: 85.159% (-0.6%) from 85.744%
27768085173

push

github

web-flow 
test: remove useless tests (#1439)

## What

Removes tests that assert nothing meaningful, duplicate other tests, or
only exercise behavior that Dart itself guarantees. No production code
or genuinely useful coverage is touched.

406 deletions across 5 test files.

## Removed

**`gotrue/test/src/constants_test.dart`**
- `constants are consistent` — re-asserts the same constants in
different units (Dart `Duration` arithmetic).
- `v20240101 is UTC timestamp` — asserts `DateTime.parse('...Z').isUtc`,
a language guarantee.
- The `enum names match expected values` test for `OtpType` — asserts
`enum.name`, but `OtpType` serializes to the wire via `.snakeCase`, not
`.name`, so it only exercised Dart's `.name`. (The `OtpChannel` and
`SignOutScope` equivalents are **kept and reworded** as wire-value
tests, since those enums *are* serialized via `.name`.)
- The `Enum consistency` group — `toString`, `==`, and set/map usage of
enums are all Dart guarantees.
- The `Documentation and comments` group — re-tests `enum.name`
(duplicates the above).
- The `Real-world usage scenarios` group — tests a local switch helper,
arithmetic on hand-built lists, and `fromString` round-trips already
covered by the kept `GenerateLinkTypeExtended` tests.

Kept the constant value checks (they guard wire-protocol contracts), the
`jsName`/`fromString`/`snakeCase` mapping tests, the custom
`ApiVersions.name`/`timestamp` getters, and the enum-membership
(`values`/`contains`) checks.

**`gotrue/test/src/types/auth_exception_test.dart`**
- `all exception types implement Exception` and `all exception types
extend AuthException` — assert declared inheritance, fully redundant
with the per-subclass `extends AuthException` tests.

**`postgrest/test/transforms_test.dart`**
- The `maxAffected` group (5 `returnsNormally` smoke tests) — redundant
with the `maxAffected integration` group right below it, which actually
asserts the resulting `Prefer` headers.

**`supabase/test/client_test.dart`**
- `... (continued)

4074 of 4784 relevant lines covered (85.16%)

3.47 hits per line

Coverage Regressions

Lines Coverage ∆ File
19
0.0
 -86.36% packages/supabase/lib/src/supabase_query_schema.dart
9
88.34
 -5.52% packages/supabase/lib/src/supabase_client.dart
Jobs
ID Job ID Ran Files Coverage
5 supabase_flutter - 27768085173.5 9
77.0
 GitHub Action Run
6 gotrue - 27768085173.6 26
83.51
 GitHub Action Run
7 supabase - 27768085173.7 15
82.1
 GitHub Action Run
8 postgrest - 27768085173.8 9
83.76
 GitHub Action Run
8 yet_another_json_isolate - 27767998227.8 1
79.03
 GitHub Action Run
8 storage_client - 27763779899.8 5
93.01
 GitHub Action Run
7 functions_client - 27767998227.7 2
97.1
 GitHub Action Run
6 realtime_client - 27763779899.6 11
88.13
 GitHub Action Run
Source Files on build 27768085173