divine-mobile: Skill for Claude Code

Flutter Dispose Timer Test Failure

Problem

Flutter widget tests fail with the error "A Timer is still pending even after the widget tree was disposed" when dispose() contains Future-based cleanup code.

Context / Trigger Conditions

• Test error: A Timer is still pending even after the widget tree was disposed
• Widget's dispose() method contains:
  • Future(() => ...)
  • Future.delayed(...)
  • Timer(...) or Timer.periodic(...)
  • Riverpod ref.read(provider.notifier).someMethod() wrapped in Future
• Tests pass individually but fail when run together
• Error appears at end of test, not during widget interaction

Solution

Step 1: Identify the problematic code

Look for patterns like this in your widget:

// BAD: Creates pending timer that test framework detects
@override
void dispose() {
  final notifier = _overlayNotifier;
  if (notifier != null) {
    Future(() => notifier.setSettingsOpen(false));  // <- Problem!
  }
  super.dispose();
}

Step 2: Move cleanup to deactivate()

Use deactivate() instead of dispose() and remove the Future wrapper:

// GOOD: Runs synchronously before widget is removed
@override
void deactivate() {
  _overlayNotifier?.setSettingsOpen(false);  // Direct call, no Future
  super.deactivate();
}

Step 3: Understand the lifecycle difference

• deactivate(): Called when widget is removed from tree, but State might be reinserted
• dispose(): Called when State will never build again, permanent cleanup

For most notification/provider cleanup, deactivate() is appropriate.

Verification

1. Run the specific test that was failing
2. Run the full test suite
3. Verify no "Timer is still pending" errors

Example

Before (causes test failure):

class _SettingsScreenState extends ConsumerState<SettingsScreen> {
  OverlayNotifier? _overlayNotifier;

  @override
  void dispose() {
    final notifier = _overlayNotifier;
    if (notifier != null) {
      // This Future creates a pending timer!
      Future(() => notifier.setSettingsOpen(false));
    }
    super.dispose();
  }
}