-
-
Notifications
You must be signed in to change notification settings - Fork 564
Open
0 / 30 of 3 issues completedOpen
0 / 30 of 3 issues completed
Copy link
Labels
dartPull requests that update Dart codePull requests that update Dart codedependenciesPull requests that update a dependency filePull requests that update a dependency filegood first issueGood for newcomersGood for newcomersrefactorRefactor the code to improve its qualityRefactor the code to improve its qualitysecuritySecurity fixSecurity fixtestTesting applicationTesting application
Description
Problem Statement
Missing golden tests for critical widgets.
Files to Modify / Create (static targets)
- test/goldens/**
- test/goldens/golden_test.dart
Files to Modify / Create (detected for this PR)
-
lib/widgets/add_members_bottom_sheet.dart -
lib/widgets/add_pledge_dialogue_box.dart -
lib/widgets/agenda_item_tile.dart -
lib/widgets/caption_text_widget.dart -
lib/widgets/comment_interactions.dart -
lib/widgets/custom_alert_dialog.dart -
lib/widgets/custom_alert_dialog_with_checkbox.dart -
lib/widgets/custom_avatar.dart -
lib/widgets/custom_drawer.dart -
lib/widgets/custom_list_tile.dart
Proposed Solution
Implement within MVVM + Provider + GetIt; avoid large rewrites. Keep changes scoped to the listed files.
Implementation Approach
- Update only the files listed above (<= 10 in this PR).
- Add/adjust tests accordingly; keep CI green.
- If more files are needed, open follow-up PRs.
Acceptance Criteria
- All listed files updated
- Tests added/updated (>= 80% for changed units)
- CI passing
- Docs updated if applicable
Starter Code
Starter Code
// test/goldens/golden_test.dart
import 'package:flutter_test/flutter_test.dart';
import 'package:flutter/material.dart';
Widget themed(Widget c,ThemeMode m)=>MaterialApp(themeMode:m,theme:ThemeData.light(),darkTheme:ThemeData.dark(),home:c);
void main(){
testWidgets('MyWidget golden', (t) async{
await t.pumpWidget(themed(const Placeholder(), ThemeMode.light));
await expectLater(find.byType(Placeholder), matchesGoldenFile('mywidget_light.png'));
});
}Dependencies
- Blocked by: None
- Blocks: MobileFix - Testing: Golden tests for critical widgets (Part 2 of 4) #3214
Phase
P2 - High Impact
Sub-issues
Metadata
Metadata
Assignees
Labels
dartPull requests that update Dart codePull requests that update Dart codedependenciesPull requests that update a dependency filePull requests that update a dependency filegood first issueGood for newcomersGood for newcomersrefactorRefactor the code to improve its qualityRefactor the code to improve its qualitysecuritySecurity fixSecurity fixtestTesting applicationTesting application