From dcc1951b4775bdba32c6f2f3b98f358c03c6725b Mon Sep 17 00:00:00 2001
From: arturplaczek <arturp1989@gmail.com>
Date: Wed, 4 May 2022 12:34:07 +0200
Subject: [PATCH] chore: remove pinballGame from PlayButtonOverlay

---
 .../view/widgets/play_button_overlay.dart     | 17 ++-----
 .../widgets/play_button_overlay_test.dart     | 49 ++++++-------------
 2 files changed, 20 insertions(+), 46 deletions(-)

diff --git a/lib/game/view/widgets/play_button_overlay.dart b/lib/game/view/widgets/play_button_overlay.dart
index 1d4a10fb..7a954c77 100644
--- a/lib/game/view/widgets/play_button_overlay.dart
+++ b/lib/game/view/widgets/play_button_overlay.dart
@@ -1,7 +1,7 @@
 import 'package:flutter/material.dart';
-import 'package:pinball/game/pinball_game.dart';
+import 'package:flutter_bloc/flutter_bloc.dart';
 import 'package:pinball/l10n/l10n.dart';
-import 'package:pinball/select_character/select_character.dart';
+import 'package:pinball/start_game/start_game.dart';
 import 'package:pinball_ui/pinball_ui.dart';
 
 /// {@template play_button_overlay}
@@ -9,13 +9,7 @@ import 'package:pinball_ui/pinball_ui.dart';
 /// {@endtemplate}
 class PlayButtonOverlay extends StatelessWidget {
   /// {@macro play_button_overlay}
-  const PlayButtonOverlay({
-    Key? key,
-    required PinballGame game,
-  })  : _game = game,
-        super(key: key);
-
-  final PinballGame _game;
+  const PlayButtonOverlay({Key? key}) : super(key: key);
 
   @override
   Widget build(BuildContext context) {
@@ -23,9 +17,8 @@ class PlayButtonOverlay extends StatelessWidget {
 
     return PinballButton(
       text: l10n.play,
-      onTap: () async {
-        _game.gameFlowController.start();
-        await showCharacterSelectionDialog(context);
+      onTap: () {
+        context.read<StartGameBloc>().add(const PlayTapped());
       },
     );
   }
diff --git a/test/game/view/widgets/play_button_overlay_test.dart b/test/game/view/widgets/play_button_overlay_test.dart
index 843592c3..7fc606d1 100644
--- a/test/game/view/widgets/play_button_overlay_test.dart
+++ b/test/game/view/widgets/play_button_overlay_test.dart
@@ -2,64 +2,45 @@ import 'package:bloc_test/bloc_test.dart';
 import 'package:flutter_test/flutter_test.dart';
 import 'package:mocktail/mocktail.dart';
 import 'package:pinball/game/game.dart';
-import 'package:pinball/select_character/select_character.dart';
+import 'package:pinball/start_game/bloc/start_game_bloc.dart';
 
 import '../../../helpers/helpers.dart';
 
-class _MockPinballGame extends Mock implements PinballGame {}
-
-class _MockGameFlowController extends Mock implements GameFlowController {}
-
-class _MockCharacterThemeCubit extends Mock implements CharacterThemeCubit {}
+class _MockStartGameBloc extends Mock implements StartGameBloc {}
 
 void main() {
   group('PlayButtonOverlay', () {
-    late PinballGame game;
-    late GameFlowController gameFlowController;
-    late CharacterThemeCubit characterThemeCubit;
+    late StartGameBloc startGameBloc;
 
     setUp(() async {
       await mockFlameImages();
 
-      game = _MockPinballGame();
-      gameFlowController = _MockGameFlowController();
-      characterThemeCubit = _MockCharacterThemeCubit();
+      startGameBloc = _MockStartGameBloc();
 
       whenListen(
-        characterThemeCubit,
-        const Stream<CharacterThemeState>.empty(),
-        initialState: const CharacterThemeState.initial(),
+        startGameBloc,
+        Stream.value(const StartGameState.initial()),
+        initialState: const StartGameState.initial(),
       );
-      when(() => characterThemeCubit.state)
-          .thenReturn(const CharacterThemeState.initial());
-      when(() => game.gameFlowController).thenReturn(gameFlowController);
-      when(gameFlowController.start).thenAnswer((_) {});
     });
 
     testWidgets('renders correctly', (tester) async {
-      await tester.pumpApp(PlayButtonOverlay(game: game));
-      expect(find.text('Play'), findsOneWidget);
-    });
+      await tester.pumpApp(const PlayButtonOverlay());
 
-    testWidgets('calls gameFlowController.start when tapped', (tester) async {
-      await tester.pumpApp(
-        PlayButtonOverlay(game: game),
-        characterThemeCubit: characterThemeCubit,
-      );
-      await tester.tap(find.text('Play'));
-      await tester.pump();
-      verify(gameFlowController.start).called(1);
+      expect(find.text('Play'), findsOneWidget);
     });
 
-    testWidgets('displays CharacterSelectionDialog when tapped',
+    testWidgets('adds PlayTapped event to StartGameBloc when taped',
         (tester) async {
       await tester.pumpApp(
-        PlayButtonOverlay(game: game),
-        characterThemeCubit: characterThemeCubit,
+        const PlayButtonOverlay(),
+        startGameBloc: startGameBloc,
       );
+
       await tester.tap(find.text('Play'));
       await tester.pump();
-      expect(find.byType(CharacterSelectionDialog), findsOneWidget);
+
+      verify(() => startGameBloc.add(const PlayTapped())).called(1);
     });
   });
 }