diff --git a/packages/flutter/lib/src/animation/animations.dart b/packages/flutter/lib/src/animation/animations.dart index 3b4d92f235b..924eb9c2e44 100644 --- a/packages/flutter/lib/src/animation/animations.dart +++ b/packages/flutter/lib/src/animation/animations.dart @@ -543,7 +543,7 @@ class TrainHoppingAnimation extends Animation void _statusChangeHandler(AnimationStatus status) { assert(_currentTrain != null); if (status != _lastStatus) { - notifyListeners(); + notifyStatusListeners(status); _lastStatus = status; } assert(_lastStatus != null); diff --git a/packages/flutter/test/animation/animations_test.dart b/packages/flutter/test/animation/animations_test.dart index f2d063b23b1..d6486c86c72 100644 --- a/packages/flutter/test/animation/animations_test.dart +++ b/packages/flutter/test/animation/animations_test.dart @@ -99,7 +99,9 @@ void main() { test('TrainHoppingAnimation', () { final AnimationController currentTrain = AnimationController(vsync: const TestVSync()); + addTearDown(currentTrain.dispose); final AnimationController nextTrain = AnimationController(vsync: const TestVSync()); + addTearDown(nextTrain.dispose); currentTrain.value = 0.5; nextTrain.value = 0.75; bool didSwitchTrains = false; @@ -133,6 +135,32 @@ void main() { ); }); + // Regression test for https://github.com/flutter/flutter/issues/178336. + test('TrainHoppingAnimation notifies status listeners', () { + final AnimationController controller = AnimationController( + duration: const Duration(milliseconds: 100), + vsync: const TestVSync(), + ); + addTearDown(controller.dispose); + + final TrainHoppingAnimation animation = TrainHoppingAnimation( + Tween(begin: 1.0, end: -1.0).animate(controller), + Tween(begin: -1.0, end: 1.0).animate(controller), + ); + addTearDown(animation.dispose); + + final List statusLog = []; + animation.addStatusListener(statusLog.add); + expect(statusLog, isEmpty); + + controller.forward(); + expect(statusLog, equals([AnimationStatus.forward])); + statusLog.clear(); + + controller.reverse(); + expect(statusLog, equals([AnimationStatus.dismissed])); + }); + test('AnimationMean control test', () { final AnimationController left = AnimationController(value: 0.5, vsync: const TestVSync()); final AnimationController right = AnimationController(vsync: const TestVSync());