updated tests

VGVentures · Apr 15, 2024 · 0ca5f21 · 0ca5f21
1 parent e4493af
commit 0ca5f21
Show file tree

Hide file tree

Showing 7 changed files with 114 additions and 12 deletions.
diff --git a/lib/leaderboard/view/leaderboard_success.dart b/lib/leaderboard/view/leaderboard_success.dart
@@ -115,8 +115,7 @@ class CurrentPlayerNotTopRank extends StatelessWidget {
 
   @override
   Widget build(BuildContext context) {
-    final player =
-        context.select((PlayerBloc bloc) => bloc.state.leaderboardPlayer);
+    final player = context.select((PlayerBloc bloc) => bloc.state.player);
 
     final rank = context.select(
       (PlayerBloc bloc) => bloc.state.rank,

diff --git a/lib/player/bloc/player_bloc.dart b/lib/player/bloc/player_bloc.dart
@@ -25,7 +25,7 @@ class PlayerBloc extends Bloc<PlayerEvent, PlayerState> {
       onData: (data) {
         return PlayerState(
           status: PlayerStatus.playing,
-          leaderboardPlayer: data.$1,
+          player: data.$1,
           rank: data.$2,
         );
       },

diff --git a/lib/player/bloc/player_state.dart b/lib/player/bloc/player_state.dart
@@ -10,26 +10,26 @@ enum PlayerStatus {
 class PlayerState extends Equatable {
   const PlayerState({
     this.status = PlayerStatus.onboarding,
-    this.leaderboardPlayer = LeaderboardPlayer.empty,
+    this.player = LeaderboardPlayer.empty,
     this.rank = 0,
   });
 
   PlayerState copyWith({
     PlayerStatus? status,
-    LeaderboardPlayer? leaderboardPlayer,
+    LeaderboardPlayer? player,
     int? rank,
   }) {
     return PlayerState(
       status: status ?? this.status,
-      leaderboardPlayer: leaderboardPlayer ?? this.leaderboardPlayer,
+      player: player ?? this.player,
       rank: rank ?? this.rank,
     );
   }
 
   final PlayerStatus status;
-  final LeaderboardPlayer leaderboardPlayer;
+  final LeaderboardPlayer player;
   final int rank;
 
   @override
-  List<Object?> get props => [status, leaderboardPlayer, rank];
+  List<Object?> get props => [status, player, rank];
 }
diff --git a/test/leaderboard/view/leaderboard_success_test.dart b/test/leaderboard/view/leaderboard_success_test.dart
@@ -129,7 +129,7 @@ void main() {
         when(() => playerBloc.state).thenReturn(
           PlayerState(
             rank: 3,
-            leaderboardPlayer: currentPlayer,
+            player: currentPlayer,
           ),
         );
 
@@ -248,7 +248,7 @@ void main() {
 
             when(() => playerBloc.state).thenReturn(
               PlayerState(
-                leaderboardPlayer: player,
+                player: player,
                 rank: i,
               ),
             );
@@ -273,7 +273,7 @@ void main() {
 
           when(() => playerBloc.state).thenReturn(
             PlayerState(
-              leaderboardPlayer: player,
+              player: player,
               rank: 11,
             ),
           );

diff --git a/test/player/bloc/player_bloc_test.dart b/test/player/bloc/player_bloc_test.dart
@@ -43,7 +43,7 @@ void main() {
           PlayerState(
             status: PlayerStatus.playing,
             rank: 3,
-            leaderboardPlayer: player,
+            player: player,
           ),
         ],
       );

diff --git a/test/player/bloc/player_event_test.dart b/test/player/bloc/player_event_test.dart
@@ -0,0 +1,22 @@
+// ignore_for_file: prefer_const_constructors
+
+import 'package:flutter_test/flutter_test.dart';
+import 'package:io_crossword/player/bloc/player_bloc.dart';
+
+void main() {
+  group('$PlayerEvent', () {
+    group('$PlayerLoaded', () {
+      test('checks equality', () {
+        expect(
+          PlayerLoaded(userId: 'user-id'),
+          equals(PlayerLoaded(userId: 'user-id')),
+        );
+
+        expect(
+          PlayerLoaded(userId: 'user-id'),
+          isNot(equals(PlayerLoaded(userId: 'user-id-1'))),
+        );
+      });
+    });
+  });
+}
diff --git a/test/player/bloc/player_state_test.dart b/test/player/bloc/player_state_test.dart
@@ -0,0 +1,81 @@
+// ignore_for_file: prefer_const_constructors
+// ignore_for_file: prefer_const_literals_to_create_immutables
+
+import 'package:flutter_test/flutter_test.dart';
+import 'package:game_domain/game_domain.dart';
+import 'package:io_crossword/player/bloc/player_bloc.dart';
+
+void main() {
+  group('$PlayerState', () {
+    test('supports value comparisons', () {
+      expect(
+        PlayerState(rank: 4, status: PlayerStatus.failure),
+        equals(PlayerState(rank: 4, status: PlayerStatus.failure)),
+      );
+
+      expect(
+        PlayerState(status: PlayerStatus.failure),
+        isNot(equals(PlayerState())),
+      );
+
+      expect(
+        PlayerState(
+          player: LeaderboardPlayer(
+            userId: '1',
+            initials: 'AAA',
+            score: 100,
+            streak: 2,
+            mascot: Mascots.dash,
+          ),
+        ),
+        isNot(equals(PlayerState())),
+      );
+
+      expect(
+        PlayerState(rank: 4),
+        isNot(equals(PlayerState())),
+      );
+    });
+
+    group('copyWith', () {
+      test('updates status', () {
+        expect(
+          PlayerState().copyWith(status: PlayerStatus.playing),
+          equals(PlayerState(status: PlayerStatus.playing)),
+        );
+      });
+
+      test('updates status', () {
+        expect(
+          PlayerState().copyWith(rank: 5),
+          equals(PlayerState(rank: 5)),
+        );
+      });
+
+      test('updates players', () {
+        expect(
+          PlayerState().copyWith(
+            player: LeaderboardPlayer(
+              userId: '1',
+              initials: 'AAA',
+              score: 100,
+              streak: 2,
+              mascot: Mascots.dash,
+            ),
+          ),
+          equals(
+            PlayerState(
+              player: LeaderboardPlayer(
+                userId: '1',
+                initials: 'AAA',
+                score: 100,
+                streak: 2,
+                mascot: Mascots.dash,
+              ),
+            ),
+          ),
+        );
+      });
+    });
+  });
+}