Add tap more info on release event detail screen

VocaDB · Mar 19, 2024 · 8910168 · 8910168
1 parent 8813458
commit 8910168
Show file tree

Hide file tree

Showing 5 changed files with 76 additions and 21 deletions.
diff --git a/...leaseEvents/presentation/release_event_detail_screen/release_event_detail_button_bar.dart b/...leaseEvents/presentation/release_event_detail_screen/release_event_detail_button_bar.dart
@@ -1,8 +1,12 @@
 import 'package:flutter/material.dart';
+import 'package:flutter_riverpod/flutter_riverpod.dart';
 import 'package:vocadb_app/src/constants/app_sizes.dart';
+import 'package:vocadb_app/src/features/releaseEvents/domain/release_event.dart';
+import 'package:vocadb_app/src/utils/url_launcher.dart';
 
 class ReleaseEventDetailButtonBar extends StatelessWidget {
-  const ReleaseEventDetailButtonBar({super.key});
+  final ReleaseEvent releaseEvent;
+  const ReleaseEventDetailButtonBar({super.key, required this.releaseEvent});
 
   static const addBtnKey = Key('add-btn-key');
   static const shareBtnKey = Key('share-btn-key');
@@ -19,8 +23,8 @@ class ReleaseEventDetailButtonBar extends StatelessWidget {
           style: ButtonStyle(
             foregroundColor: MaterialStateProperty.all<Color>(Colors.white),
           ),
-          child: Column(
-            children: const [
+          child: const Column(
+            children:  [
               Icon(Icons.favorite),
               gapH4,
               Text('Like'),
@@ -33,28 +37,32 @@ class ReleaseEventDetailButtonBar extends StatelessWidget {
           style: ButtonStyle(
             foregroundColor: MaterialStateProperty.all<Color>(Colors.white),
           ),
-          child: Column(
-            children: const [
+          child: const Column(
+            children: [
               Icon(Icons.share),
               gapH4,
               Text('Share'),
             ],
           ),
         ),
-        TextButton(
-          key: infoBtnKey,
-          onPressed: () => {},
-          style: ButtonStyle(
-            foregroundColor: MaterialStateProperty.all<Color>(Colors.white),
-          ),
-          child: Column(
-            children: const [
-              Icon(Icons.info),
-              gapH4,
-              Text('Info'),
-            ],
-          ),
-        ),
+        Consumer(builder: (context, ref, _) {
+          return TextButton(
+            key: infoBtnKey,
+            onPressed: () {
+              ref.read(urlLauncherProvider).launchReleaseEventMoreInfo(releaseEvent.id);
+            },
+            style: ButtonStyle(
+              foregroundColor: MaterialStateProperty.all<Color>(Colors.white),
+            ),
+            child: const Column(
+              children: [
+                Icon(Icons.info),
+                gapH4,
+                Text('Info'),
+              ],
+            ),
+          );
+        }),
       ],
     );
   }

diff --git a/...s/releaseEvents/presentation/release_event_detail_screen/release_event_detail_screen.dart b/...s/releaseEvents/presentation/release_event_detail_screen/release_event_detail_screen.dart
@@ -56,7 +56,7 @@ class ReleaseEventDetailScreen extends StatelessWidget {
                     ),
                     SliverList(
                       delegate: SliverChildListDelegate([
-                        const ReleaseEventDetailButtonBar(),
+                        ReleaseEventDetailButtonBar(releaseEvent: releaseEvent,),
                         Column(
                           mainAxisAlignment: MainAxisAlignment.start,
                           crossAxisAlignment: CrossAxisAlignment.start,

diff --git a/lib/src/utils/url_launcher.dart b/lib/src/utils/url_launcher.dart
@@ -19,6 +19,9 @@ class UrlLauncher {
 
   launchTagMoreInfo(int id) =>
     launchUrlString('$host/T/$id');
+
+  launchReleaseEventMoreInfo(int id) =>
+    launchUrlString('$host/E/$id');
 }
 
 

diff --git a/...easeEvents/presentation/release_event_detail_screen/release_event_detail_screen_test.dart b/...easeEvents/presentation/release_event_detail_screen/release_event_detail_screen_test.dart
@@ -71,4 +71,36 @@ void main() {
     await r.expectAlbumsListVisible(false);
     await r.expectWebLinksVisible(false);
   });
+
+  testWidgets('tap more info on release event detail screen', (tester) async {
+    final r = ReleaseEventRobot(tester);
+    final releaseEventRepository = MockReleaseEventRepository();
+    final urlLauncher = MockUrlLauncher();
+
+    when(() => releaseEventRepository.fetchReleaseEventByID(any()))
+        .thenAnswer((_) => Future.value(kFakeReleaseEventDetail));
+
+    when(() => releaseEventRepository.fetchAlbums(any()))
+        .thenAnswer((_) => Future.value(kFakeAlbumsList));
+
+    when(() => releaseEventRepository.fetchPublishedSongs(any()))
+        .thenAnswer((_) => Future.value(kFakeSongsList));
+
+    await r.pumpReleaseEventDetailScreen(
+      releaseEventRepository: releaseEventRepository,
+      urlLauncher: urlLauncher
+    );
+
+    await tester.pump();
+
+    await r.expectErrorMessageWidgetNotVisible();
+
+    await r.expectAddButtonVisible();
+    await r.expectShareButtonVisible();
+    await r.expectInfoButtonVisible();
+
+    await r.tapMoreInfo();
+
+    verify(() => urlLauncher.launchReleaseEventMoreInfo(kFakeReleaseEventDetail.id)).called(1);
+  });
 }
diff --git a/test/src/features/releaseEvents/release_event_robot.dart b/test/src/features/releaseEvents/release_event_robot.dart
@@ -10,6 +10,7 @@ import 'package:vocadb_app/src/features/releaseEvents/presentation/release_event
 import 'package:vocadb_app/src/features/songs/presentation/songs_list/songs_list_view.dart';
 import 'package:vocadb_app/src/features/tags/presentation/tag_widgets/tag_usage_group_view.dart';
 import 'package:vocadb_app/src/features/weblinks/presentation/web_link_group_view.dart';
+import 'package:vocadb_app/src/utils/url_launcher.dart';
 
 class ReleaseEventRobot {
   final WidgetTester tester;
@@ -18,13 +19,17 @@ class ReleaseEventRobot {
 
   Future<void> pumpReleaseEventDetailScreen({
     ReleaseEventRepository? releaseEventRepository,
+    UrlLauncher? urlLauncher
   }) async {
     await tester.pumpWidget(
       ProviderScope(
         overrides: [
           if (releaseEventRepository != null)
             releaseEventRepositoryProvider
-                .overrideWithValue(releaseEventRepository)
+                .overrideWithValue(releaseEventRepository),
+          if (urlLauncher != null)
+            urlLauncherProvider
+                .overrideWithValue(urlLauncher)
         ],
         child: const MaterialApp(
           home: ReleaseEventDetailScreen(
@@ -91,6 +96,13 @@ class ReleaseEventRobot {
     expect(finder, findsOneWidget);
   }
 
+  Future<void> tapMoreInfo() async {
+    final finder = find.byKey(ReleaseEventDetailButtonBar.infoBtnKey);
+    expect(finder, findsOneWidget);
+    await tester.tap(finder);
+    await tester.pump();
+  }
+
   Future<void> expectTagNameIs(String name) async {
     final finder = find.widgetWithText(ListTile, name);
     expect(finder, findsOneWidget);