Skip to content

Commit 73627d9

Browse files
committed
Increase CI coverage threshold from 25% to 70%
Now that test coverage has been significantly improved (72%+), we can enforce a higher coverage threshold in CI to prevent regressions. Changes: - minimum-coverage: 25 → 70 - Updated comment to reflect current coverage level - TODO completed: threshold restored to original target
1 parent f9fb44f commit 73627d9

2 files changed

Lines changed: 3 additions & 3 deletions

File tree

.github/workflows/build-deploy.yml

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -84,8 +84,9 @@ jobs:
8484
uses: zgosalvez/github-actions-report-lcov@v4
8585
with:
8686
coverage-files: coverage/lcov.info
87-
# TODO: Increase coverage target back to 70% as test coverage improves
88-
minimum-coverage: 25
87+
# Updated: Coverage threshold increased from 25% to 70%
88+
# Current coverage: 72%+ with comprehensive test improvements
89+
minimum-coverage: 70
8990
artifact-name: code-coverage-report
9091
github-token: ${{ secrets.GITHUB_TOKEN }}
9192
update-comment: true

test/router_test.dart

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,6 @@ import 'package:cambridge_beer_festival/models/models.dart';
77
import 'package:provider/provider.dart';
88
import 'package:mockito/mockito.dart';
99
import 'package:shared_preferences/shared_preferences.dart';
10-
import 'package:go_router/go_router.dart';
1110

1211
import 'provider_test.mocks.dart';
1312

0 commit comments

Comments
 (0)