Merge pull request #26 from VGVentures/feat/flipper-placeholder-art

feat: adding placeholder art for the flippers

lib/game/components/flipper.dart

@@ -1,9 +1,9 @@
 import 'dart:async';
 import 'dart:math' as math;
 
+import 'package:flame/components.dart' show SpriteComponent;
 import 'package:flame/input.dart';
 import 'package:flame_forge2d/flame_forge2d.dart';
-import 'package:flutter/material.dart';
 import 'package:flutter/services.dart';
 import 'package:pinball/game/game.dart';
 
@@ -12,19 +12,15 @@ import 'package:pinball/game/game.dart';
 ///
 /// [Flipper] can be controlled by the player in an arc motion.
 /// {@endtemplate flipper}
-class Flipper extends BodyComponent with KeyboardHandler {
+class Flipper extends PositionBodyComponent with KeyboardHandler {
   /// {@macro flipper}
   Flipper._({
     required Vector2 position,
     required this.side,
     required List<LogicalKeyboardKey> keys,
   })  : _position = position,
-        _keys = keys {
+        _keys = keys,
-    // TODO(alestiago): Use sprite instead of color when provided.
+        super(size: Vector2(width, height));
-    paint = Paint()
-      ..color = const Color(0xFF00FF00)
-      ..style = PaintingStyle.fill;
-  }
 
   /// A left positioned [Flipper].
   Flipper.left({
@@ -50,6 +46,11 @@ class Flipper extends BodyComponent with KeyboardHandler {
           ],
         );
 
+  /// Asset location of the sprite that renders with the [Flipper].
+  ///
+  /// Sprite is preloaded by [PinballGameAssetsX].
+  static const spritePath = 'components/flipper.png';
+
   /// The width of the [Flipper].
   static const width = 12.0;
 
@@ -75,6 +76,20 @@ class Flipper extends BodyComponent with KeyboardHandler {
   /// [onKeyEvent] method listens to when one of these keys is pressed.
   final List<LogicalKeyboardKey> _keys;
 
+  @override
+  Future<void> onLoad() async {
+    await super.onLoad();
+    final sprite = await gameRef.loadSprite(spritePath);
+    positionComponent = SpriteComponent(
+      sprite: sprite,
+      size: size,
+    );
+
+    if (side == BoardSide.right) {
+      positionComponent?.flipHorizontally();
+    }
+  }
+
   /// Applies downward linear velocity to the [Flipper], moving it to its
   /// resting position.
   void _moveDown() {

lib/game/game_assets.dart

@@ -6,6 +6,7 @@ extension PinballGameAssetsX on PinballGame {
   Future<void> preLoadAssets() async {
     await Future.wait([
       images.load(Ball.spritePath),
+      images.load(Flipper.spritePath),
     ]);
   }
 }

test/game/components/anchor_test.dart

@@ -15,6 +15,7 @@ void main() {
       'loads correctly',
       (game) async {
         final anchor = Anchor(position: Vector2.zero());
+        await game.ready();
         await game.ensureAdd(anchor);
 
         expect(game.contains(anchor), isTrue);
@@ -25,6 +26,7 @@ void main() {
       flameTester.test(
         'positions correctly',
         (game) async {
+          await game.ready();
           final position = Vector2.all(10);
           final anchor = Anchor(position: position);
           await game.ensureAdd(anchor);
@@ -37,6 +39,7 @@ void main() {
       flameTester.test(
         'is static',
         (game) async {
+          await game.ready();
           final anchor = Anchor(position: Vector2.zero());
           await game.ensureAdd(anchor);
 

test/game/components/pathway_test.dart

@@ -18,6 +18,7 @@ void main() {
         flameTester.test(
           'has transparent color by default when no color is specified',
           (game) async {
+            await game.ready();
             final pathway = Pathway.straight(
               position: Vector2.zero(),
               start: Vector2(10, 10),
@@ -38,6 +39,7 @@ void main() {
         flameTester.test(
           'has a color when is specified',
           (game) async {
+            await game.ready();
             const defaultColor = Colors.blue;
 
             final pathway = Pathway.straight(
@@ -59,6 +61,7 @@ void main() {
       flameTester.test(
         'loads correctly',
         (game) async {
+          await game.ready();
           final pathway = Pathway.straight(
             position: Vector2.zero(),
             start: Vector2(10, 10),
@@ -75,6 +78,7 @@ void main() {
         flameTester.test(
           'positions correctly',
           (game) async {
+            await game.ready();
             final position = Vector2.all(10);
             final pathway = Pathway.straight(
               position: position,
@@ -92,6 +96,7 @@ void main() {
         flameTester.test(
           'is static',
           (game) async {
+            await game.ready();
             final pathway = Pathway.straight(
               position: Vector2.zero(),
               start: Vector2(10, 10),
@@ -109,6 +114,7 @@ void main() {
         flameTester.test(
           'has only one ChainShape when singleWall is true',
           (game) async {
+            await game.ready();
             final pathway = Pathway.straight(
               position: Vector2.zero(),
               start: Vector2(10, 10),
@@ -128,6 +134,7 @@ void main() {
         flameTester.test(
           'has two ChainShape when singleWall is false (default)',
           (game) async {
+            await game.ready();
             final pathway = Pathway.straight(
               position: Vector2.zero(),
               start: Vector2(10, 10),
@@ -150,6 +157,7 @@ void main() {
       flameTester.test(
         'loads correctly',
         (game) async {
+          await game.ready();
           final pathway = Pathway.arc(
             position: Vector2.zero(),
             width: width,
@@ -166,6 +174,7 @@ void main() {
         flameTester.test(
           'positions correctly',
           (game) async {
+            await game.ready();
             final position = Vector2.all(10);
             final pathway = Pathway.arc(
               position: position,
@@ -183,6 +192,7 @@ void main() {
         flameTester.test(
           'is static',
           (game) async {
+            await game.ready();
             final pathway = Pathway.arc(
               position: Vector2.zero(),
               width: width,
@@ -208,6 +218,7 @@ void main() {
       flameTester.test(
         'loads correctly',
         (game) async {
+          await game.ready();
           final pathway = Pathway.bezierCurve(
             position: Vector2.zero(),
             controlPoints: controlPoints,
@@ -223,6 +234,7 @@ void main() {
         flameTester.test(
           'positions correctly',
           (game) async {
+            await game.ready();
             final position = Vector2.all(10);
             final pathway = Pathway.bezierCurve(
               position: position,
@@ -239,6 +251,7 @@ void main() {
         flameTester.test(
           'is static',
           (game) async {
+            await game.ready();
             final pathway = Pathway.bezierCurve(
               position: Vector2.zero(),
               controlPoints: controlPoints,

test/game/components/plunger_test.dart

@@ -16,6 +16,7 @@ void main() {
     flameTester.test(
       'loads correctly',
       (game) async {
+        await game.ready();
         final plunger = Plunger(position: Vector2.zero());
         await game.ensureAdd(plunger);
 

test/game/components/wall_test.dart

@@ -37,6 +37,7 @@ void main() {
     flameTester.test(
       'loads correctly',
       (game) async {
+        await game.ready();
         final wall = Wall(
           start: Vector2.zero(),
           end: Vector2(100, 0),