refactor: defined isFullyProgressed method

lib/game/components/flutter_forest/behaviors/flutter_forest_bonus_behavior.dart

@@ -31,7 +31,7 @@ class FlutterForestBonusBehavior extends Component
             bumper.bloc.onReset();
           }
 
-          if (signpost.bloc.state.isFullyLit()) {
+          if (signpost.bloc.isFullyProgressed()) {
             gameRef
                 .read<GameBloc>()
                 .add(const BonusActivated(GameBonus.dashNest));

packages/pinball_components/lib/src/components/signpost/cubit/signpost_cubit.dart

@@ -13,4 +13,6 @@ class SignpostCubit extends Cubit<SignpostState> {
       SignpostState.values[(index + 1) % SignpostState.values.length],
     );
   }
+
+  bool isFullyProgressed() => state == SignpostState.active3;
 }

packages/pinball_components/lib/src/components/signpost/cubit/signpost_state.dart

@@ -15,7 +15,3 @@ enum SignpostState {
   /// Signpost with all signs of lit up dashes.
   active3,
 }
-
-extension SignpostX on SignpostState {
-  bool isFullyLit() => this == SignpostState.active3;
-}

packages/pinball_components/lib/src/components/signpost/signpost.dart

@@ -48,6 +48,12 @@ class Signpost extends BodyComponent with InitialPosition {
   // ignore: public_member_api_docs
   final SignpostCubit bloc;
 
+  @override
+  void onRemove() {
+    bloc.close();
+    super.onRemove();
+  }
+
   @override
   Body createBody() {
     final shape = CircleShape()..radius = 0.25;

packages/pinball_components/test/src/components/signpost/cubit/signpost_cubit_test.dart

@@ -0,0 +1,39 @@
+import 'package:bloc_test/bloc_test.dart';
+import 'package:flutter_test/flutter_test.dart';
+import 'package:pinball_components/pinball_components.dart';
+
+void main() {
+  group('SignpostCubit', () {
+    blocTest<SignpostCubit, SignpostState>(
+      'onProgressed progresses',
+      build: SignpostCubit.new,
+      act: (bloc) {
+        bloc
+          ..onProgressed()
+          ..onProgressed()
+          ..onProgressed()
+          ..onProgressed();
+      },
+      expect: () => [
+        SignpostState.active1,
+        SignpostState.active2,
+        SignpostState.active3,
+        SignpostState.inactive,
+      ],
+    );
+
+    test('isFullyProgressed when on active3', () {
+      final bloc = SignpostCubit();
+      expect(bloc.isFullyProgressed(), isFalse);
+
+      bloc.onProgressed();
+      expect(bloc.isFullyProgressed(), isFalse);
+
+      bloc.onProgressed();
+      expect(bloc.isFullyProgressed(), isFalse);
+
+      bloc.onProgressed();
+      expect(bloc.isFullyProgressed(), isTrue);
+    });
+  });
+}

packages/pinball_components/test/src/components/signpost/signpost_test.dart

@@ -1,12 +1,16 @@
 // ignore_for_file: cascade_invocations
 
+import 'package:bloc_test/bloc_test.dart';
 import 'package:flame/components.dart';
 import 'package:flame_test/flame_test.dart';
 import 'package:flutter_test/flutter_test.dart';
+import 'package:mocktail/mocktail.dart';
 import 'package:pinball_components/pinball_components.dart';
 
 import '../../../helpers/helpers.dart';
 
+class _MockSignpostCubit extends Mock implements SignpostCubit {}
+
 void main() {
   TestWidgetsFlutterBinding.ensureInitialized();
   final assets = [
@@ -140,5 +144,22 @@ void main() {
       await game.ensureAdd(signpost);
       expect(signpost.children, contains(component));
     });
+
+    flameTester.test('closes bloc when removed', (game) async {
+      final bloc = _MockSignpostCubit();
+      whenListen(
+        bloc,
+        const Stream<SignpostCubit>.empty(),
+        initialState: SignpostState.inactive,
+      );
+      when(bloc.close).thenAnswer((_) async {});
+      final component = Signpost.test(bloc: bloc);
+
+      await game.ensureAdd(component);
+      game.remove(component);
+      await game.ready();
+
+      verify(bloc.close).called(1);
+    });
   });
 }