Skip to content

Conversation

Copilot
Copy link
Contributor

@Copilot Copilot AI commented Aug 25, 2025

Updated the "should display app bar with correct title" test in daily_challenge_view_test.dart to use the existing createTestWidget() helper function instead of manually creating a MaterialApp instance.

Before:

await tester.pumpWidget(
  const MaterialApp(
    home: DailyChallengeView(),
  ),
);

After:

await tester.pumpWidget(createTestWidget());

This change:

  • Makes the code consistent with other tests in the same file that already use createTestWidget()
  • Eliminates code duplication by reusing the existing helper function
  • Maintains exactly the same functionality since createTestWidget() returns an identical MaterialApp widget

Fixes #1655.


✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Copy link
Contributor Author

Copilot AI commented Aug 25, 2025

@huyenltnguyen 👋 This repository doesn't have Copilot instructions. With Copilot instructions, I can understand the repository better, work faster and produce higher quality PRs.

I can generate a .github/copilot-instructions.md file for you automatically. Click here to open a pre-filled issue and assign it to me. I'll write the instructions, and then tag you for review.

@Copilot Copilot AI changed the title [WIP] Update daily challenge view test to use createTestWidget Update daily challenge view test to use createTestWidget Aug 25, 2025
@Copilot Copilot AI requested a review from huyenltnguyen August 25, 2025 17:19
Copilot finished work on behalf of huyenltnguyen August 25, 2025 17:19
@huyenltnguyen huyenltnguyen changed the title Update daily challenge view test to use createTestWidget refactor: update daily challenge view test to use createTestWidget Aug 25, 2025
@huyenltnguyen huyenltnguyen marked this pull request as ready for review August 25, 2025 17:21
@huyenltnguyen huyenltnguyen requested a review from a team as a code owner August 25, 2025 17:21
@Sembauke Sembauke merged commit 6131169 into main Aug 26, 2025
5 of 6 checks passed
@Sembauke Sembauke deleted the copilot/fix-1655 branch August 26, 2025 07:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Update daily challenge view test to use createTestWidget
3 participants