Remove useless socket override in tests

lichess-org · Dec 26, 2024 · 6322ec8 · 6322ec8
1 parent a9baf8c
commit 6322ec8
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 28 deletions.
diff --git a/test/view/broadcast/broadcast_game_screen_test.dart b/test/view/broadcast/broadcast_game_screen_test.dart
@@ -29,20 +29,14 @@ final client = MockClient((request) {
 void main() {
   group('Broadcast game screen', () {
     testWidgets('Displays broadcast game screen', variant: kPlatformVariant, (tester) async {
-      final fakeSocket = FakeWebSocketChannel();
       final app = await makeTestProviderScopeApp(
         tester,
         home: const BroadcastGameScreen(
           tournamentId: _tournamentId,
           roundId: _roundId,
           gameId: _gameId,
         ),
-        overrides: [
-          lichessClientProvider.overrideWith((ref) => LichessClient(client, ref)),
-          webSocketChannelFactoryProvider.overrideWith((ref) {
-            return FakeWebSocketChannelFactory((_) => fakeSocket);
-          }),
-        ],
+        overrides: [lichessClientProvider.overrideWith((ref) => LichessClient(client, ref))],
       );
 
       await tester.pumpWidget(app);

diff --git a/test/view/broadcast/broadcast_round_screen_test.dart b/test/view/broadcast/broadcast_round_screen_test.dart
@@ -39,16 +39,10 @@ final client = MockClient((request) {
 void main() {
   group('Broadcast round screen', () {
     testWidgets('Displays broadcast round screen', variant: kPlatformVariant, (tester) async {
-      final fakeSocket = FakeWebSocketChannel();
       final app = await makeTestProviderScopeApp(
         tester,
         home: BroadcastRoundScreen(broadcast: _broadcast),
-        overrides: [
-          lichessClientProvider.overrideWith((ref) => LichessClient(client, ref)),
-          webSocketChannelFactoryProvider.overrideWith((ref) {
-            return FakeWebSocketChannelFactory((_) => fakeSocket);
-          }),
-        ],
+        overrides: [lichessClientProvider.overrideWith((ref) => LichessClient(client, ref))],
       );
 
       await tester.pumpWidget(app);
@@ -75,16 +69,10 @@ void main() {
       variant: TargetPlatformVariant.only(TargetPlatform.android),
       (tester) async {
         mockNetworkImagesFor(() async {
-          final fakeSocket = FakeWebSocketChannel();
           final app = await makeTestProviderScopeApp(
             tester,
             home: BroadcastRoundScreen(broadcast: _broadcast),
-            overrides: [
-              lichessClientProvider.overrideWith((ref) => LichessClient(client, ref)),
-              webSocketChannelFactoryProvider.overrideWith((ref) {
-                return FakeWebSocketChannelFactory((_) => fakeSocket);
-              }),
-            ],
+            overrides: [lichessClientProvider.overrideWith((ref) => LichessClient(client, ref))],
           );
 
           await tester.pumpWidget(app);
@@ -117,16 +105,10 @@ void main() {
       variant: TargetPlatformVariant.only(TargetPlatform.iOS),
       (tester) async {
         mockNetworkImagesFor(() async {
-          final fakeSocket = FakeWebSocketChannel();
           final app = await makeTestProviderScopeApp(
             tester,
             home: BroadcastRoundScreen(broadcast: _broadcast),
-            overrides: [
-              lichessClientProvider.overrideWith((ref) => LichessClient(client, ref)),
-              webSocketChannelFactoryProvider.overrideWith((ref) {
-                return FakeWebSocketChannelFactory((_) => fakeSocket);
-              }),
-            ],
+            overrides: [lichessClientProvider.overrideWith((ref) => LichessClient(client, ref))],
           );
 
           await tester.pumpWidget(app);