From a94fb6767c326ab9ca87f8eccec6c15efaf127f6 Mon Sep 17 00:00:00 2001
From: arturplaczek <arturp1989@gmail.com>
Date: Tue, 26 Apr 2022 13:33:57 +0200
Subject: [PATCH] feat: update PinballGamePage

---
 lib/game/view/pinball_game_page.dart       | 19 ++++++---
 test/game/view/pinball_game_page_test.dart | 46 ++++++++++++++++++++--
 2 files changed, 56 insertions(+), 9 deletions(-)

diff --git a/lib/game/view/pinball_game_page.dart b/lib/game/view/pinball_game_page.dart
index 2a81f4e0..0fb2dd73 100644
--- a/lib/game/view/pinball_game_page.dart
+++ b/lib/game/view/pinball_game_page.dart
@@ -43,10 +43,9 @@ class PinballGamePage extends StatelessWidget {
     final loadables = [
       ...game.preLoadAssets(),
       pinballAudio.load(),
+      ...BonusAnimation.loadAssets(),
     ];
 
-    BonusAnimation.loadAssets();
-
     return MultiBlocProvider(
       providers: [
         BlocProvider(create: (_) => StartGameBloc(game: game)),
@@ -115,6 +114,11 @@ class PinballGameLoadedView extends StatelessWidget {
 
   @override
   Widget build(BuildContext context) {
+    final leftMargin = MediaQuery.of(context).size.width * 0.3;
+    final isPlaying = context.select(
+      (StartGameBloc bloc) => bloc.state.status == StartGameStatus.play,
+    );
+
     return Stack(
       children: [
         Positioned.fill(
@@ -133,10 +137,13 @@ class PinballGameLoadedView extends StatelessWidget {
             },
           ),
         ),
-        const Positioned(
-          top: 8,
-          left: 8,
-          child: GameHud(),
+        Positioned(
+          top: 16,
+          left: leftMargin,
+          child: Visibility(
+            visible: isPlaying,
+            child: const GameHud(),
+          ),
         ),
       ],
     );
diff --git a/test/game/view/pinball_game_page_test.dart b/test/game/view/pinball_game_page_test.dart
index bf6391d9..191d3676 100644
--- a/test/game/view/pinball_game_page_test.dart
+++ b/test/game/view/pinball_game_page_test.dart
@@ -6,6 +6,7 @@ import 'package:flutter/material.dart';
 import 'package:flutter_test/flutter_test.dart';
 import 'package:pinball/game/game.dart';
 import 'package:pinball/select_character/select_character.dart';
+import 'package:pinball/start_game/start_game.dart';
 
 import '../../helpers/helpers.dart';
 
@@ -79,6 +80,7 @@ void main() {
         'renders PinballGameLoadedView after resources have been loaded',
         (tester) async {
       final assetsManagerCubit = MockAssetsManagerCubit();
+      final startGameBloc = MockStartGameBloc();
 
       final loadedAssetsState = AssetsManagerState(
         loadables: [Future<void>.value()],
@@ -89,6 +91,11 @@ void main() {
         Stream.value(loadedAssetsState),
         initialState: loadedAssetsState,
       );
+      whenListen(
+        startGameBloc,
+        Stream.value(StartGameState.initial()),
+        initialState: StartGameState.initial(),
+      );
 
       await tester.pumpApp(
         PinballGameView(
@@ -97,6 +104,7 @@ void main() {
         assetsManagerCubit: assetsManagerCubit,
         characterThemeCubit: characterThemeCubit,
         gameBloc: gameBloc,
+        startGameBloc: startGameBloc,
       );
 
       await tester.pump();
@@ -160,27 +168,59 @@ void main() {
   });
 
   group('PinballGameView', () {
+    final gameBloc = MockGameBloc();
+    final startGameBloc = MockStartGameBloc();
+
     setUp(() async {
       await Future.wait<void>(game.preLoadAssets());
-    });
 
-    testWidgets('renders game and a hud', (tester) async {
-      final gameBloc = MockGameBloc();
       whenListen(
         gameBloc,
         Stream.value(const GameState.initial()),
         initialState: const GameState.initial(),
       );
 
+      whenListen(
+        startGameBloc,
+        Stream.value(StartGameState.initial()),
+        initialState: StartGameState.initial(),
+      );
+    });
+
+    testWidgets('renders game', (tester) async {
       await tester.pumpApp(
         PinballGameView(game: game),
         gameBloc: gameBloc,
+        startGameBloc: startGameBloc,
       );
 
       expect(
         find.byWidgetPredicate((w) => w is GameWidget<PinballGame>),
         findsOneWidget,
       );
+      expect(
+        find.byType(GameHud),
+        findsNothing,
+      );
+    });
+
+    testWidgets('renders a hud on play state', (tester) async {
+      final startGameState = StartGameState.initial().copyWith(
+        status: StartGameStatus.play,
+      );
+
+      whenListen(
+        startGameBloc,
+        Stream.value(startGameState),
+        initialState: startGameState,
+      );
+
+      await tester.pumpApp(
+        PinballGameView(game: game),
+        gameBloc: gameBloc,
+        startGameBloc: startGameBloc,
+      );
+
       expect(
         find.byType(GameHud),
         findsOneWidget,