feat(app): 剧场页九宫格与极简我的页

test/discover/discover_logic_test.dart

@@ -1,255 +1,32 @@
+// Author: fruit.com.cn
+
 import 'dart:async';
 
 import 'package:flutter/material.dart';
 import 'package:flutter_test/flutter_test.dart';
-import 'package:fruitshorts/core/data/models/drama.dart';
-import 'package:fruitshorts/l10n/app_localizations.dart';
 import 'package:fruitshorts/pages/discover/discover_logic.dart';
-
-import '../support/fake_watch_history.dart';
-
-Drama _drama(
-  String id, {
-  List<int> numbers = const [1, 2, 3],
-  String language = '',
-}) => Drama(
-  language: language,
-  id: id,
-  title: id,
-  synopsis: 'synopsis',
-  posterUrl: '/files/poster.png',
-  episodes: [
-    for (final number in numbers)
-      Episode(
-        id: '$id-$number',
-        number: number,
-        title: 'Episode $number',
-        videoUrl: 'https://example.test/clip.mp4',
-        isFree: true,
-      ),
-  ],
-);
+import 'package:fruitshorts/pages/discover/discover_types.dart';
 
 void main() {
   TestWidgetsFlutterBinding.ensureInitialized();
 
-  test(
-    'should_resolve_resume_position_of_episode_one_when_constructed',
-    () async {
-      final history = FakeWatchHistory()..saved = const Duration(seconds: 4);
-      final logic = DiscoverLogic(history: history, drama: _drama('signal'));
+  test('should_report_query_only_for_non_whitespace_text', () {
+    final logic = DiscoverLogic();
 
-      expect(await logic.position, const Duration(seconds: 4));
-      expect(history.reads, ['signal']);
-      expect(logic.searchOpen, isFalse);
-      logic.dispose();
-    },
-  );
+    expect(logic.hasSearchQuery, isFalse);
 
-  test('should_reread_resume_position_when_drama_is_swapped', () async {
-    final history = FakeWatchHistory();
-    final logic = DiscoverLogic(history: history, drama: _drama('signal'));
+    logic.search.text = '   ';
+    expect(logic.hasSearchQuery, isFalse);
 
-    logic.drama = _drama('city');
-    logic.refreshPosition();
-    await logic.position;
-
-    expect(history.reads, ['signal', 'city']);
+    logic.search.text = 'remote';
+    expect(logic.hasSearchQuery, isTrue);
     logic.dispose();
   });
 
-  test(
-    'should_resume_from_last_watch_when_list_card_has_no_episodes',
-    () async {
-      final history = FakeWatchHistory()
-        ..saved = const Duration(seconds: 9)
-        ..lastEpisode = 'signal-2';
-      final logic = DiscoverLogic(
-        history: history,
-        drama: _drama('signal', numbers: const []),
-      );
-
-      expect(await logic.position, const Duration(seconds: 9));
-      expect(history.reads, ['signal']);
-      logic.dispose();
-    },
-  );
-
-  test('should_filter_visible_dramas_when_language_is_selected', () {
-    final logic = DiscoverLogic(
-      history: FakeWatchHistory(),
-      drama: _drama('s'),
-    );
-    final dramas = [
-      _drama('zh', language: 'zh-CN'),
-      _drama('id', language: 'id'),
-      _drama('unknown'),
-    ];
-
-    expect(logic.visibleDramas(dramas), hasLength(3));
-
-    logic.selectLanguage('id');
-    expect(logic.visibleDramas(dramas).map((item) => item.id), ['id']);
-
-    logic.selectLanguage(null);
-    expect(logic.visibleDramas(dramas), hasLength(3));
-    logic.dispose();
-  });
-
-  test('should_list_distinct_languages_when_feed_has_cards', () {
-    final logic = DiscoverLogic(
-      history: FakeWatchHistory(),
-      drama: _drama('s'),
-    );
-    final dramas = [
-      _drama('zh', language: 'zh-CN'),
-      _drama('id', language: 'id'),
-      _drama('id2', language: 'id'),
-      _drama('unknown'),
-    ];
-
-    expect(logic.languagesOf(dramas), ['zh-CN', 'id']);
-    logic.dispose();
-  });
-  test('should_have_no_resume_position_when_server_has_no_record', () async {
-    final history = FakeWatchHistory();
-    final logic = DiscoverLogic(
-      history: history,
-      drama: _drama('signal', numbers: const []),
-    );
-
-    expect(await logic.position, isNull);
-    // 续看位置改为按剧问服务端：即使卡片没有剧集也会读一次
-    expect(history.reads, ['signal']);
-    logic.dispose();
-  });
-
-  test('should_show_search_when_flag_is_on_or_text_is_typed', () {
-    final logic = DiscoverLogic(
-      history: FakeWatchHistory(),
-      drama: _drama('s'),
-    );
-
-    expect(logic.showSearch, isFalse);
-
-    logic.search.text = 'vengeance';
-    logic.onSearchChanged();
-    expect(logic.showSearch, isTrue);
-
-    logic.search.clear();
-    expect(logic.showSearch, isFalse);
-
-    logic.searchOpen = true;
-    expect(logic.showSearch, isTrue);
-    logic.dispose();
-  });
-
-  test('should_match_by_id_and_tolerate_missing_id_when_looking_up_drama', () {
-    final logic = DiscoverLogic(
-      history: FakeWatchHistory(),
-      drama: _drama('s'),
-    );
-    final catalog = [_drama('signal'), _drama('city')];
-
-    expect(logic.dramaFor('city', catalog)?.id, 'city');
-    expect(logic.dramaFor('missing', catalog), isNull);
-    expect(logic.dramaFor(null, catalog), isNull);
-    expect(logic.dramaFor('city', const []), isNull);
-    logic.dispose();
-  });
-
-  test('should_adopt_drama_only_when_page_is_active', () {
-    final history = FakeWatchHistory();
-    final logic = DiscoverLogic(history: history, drama: _drama('signal'));
-
-    void sync(
-      Drama next, {
-      required bool active,
-      required bool wasActive,
-      required bool dramaChanged,
-      int searchRequest = 0,
-      int previousSearchRequest = 0,
-    }) => logic.syncDrama(
-      active: active,
-      wasActive: wasActive,
-      nextDrama: next,
-      dramaChanged: dramaChanged,
-      searchRequest: searchRequest,
-      previousSearchRequest: previousSearchRequest,
-      isActive: () => active,
-    );
-
-    // An inactive page keeps showing its own drama.
-    sync(_drama('city'), active: false, wasActive: true, dramaChanged: true);
-    expect(logic.drama.id, 'signal');
-
-    // Becoming active adopts the incoming drama.
-    sync(_drama('city'), active: true, wasActive: false, dramaChanged: false);
-    expect(logic.drama.id, 'city');
-
-    // Staying active with a changed drama re-reads history.
-    history.reads.clear();
-    sync(
-      _drama('vengeance'),
-      active: true,
-      wasActive: true,
-      dramaChanged: true,
-    );
-    expect(logic.drama.id, 'vengeance');
-    expect(history.reads, ['vengeance']);
-
-    // Staying active with no change leaves both drama and history alone.
-    history.reads.clear();
-    sync(_drama('other'), active: true, wasActive: true, dramaChanged: false);
-    expect(logic.drama.id, 'vengeance');
-    expect(history.reads, isEmpty);
-    logic.dispose();
-  });
-
-  testWidgets(
-    'should_open_and_focus_field_when_search_is_requested_and_page_is_active',
-    (tester) async {
-      final logic = DiscoverLogic(
-        history: FakeWatchHistory(),
-        drama: _drama('s'),
-      );
-      await tester.pumpWidget(
-        MaterialApp(
-          home: Scaffold(
-            body: TextField(
-              controller: logic.search,
-              focusNode: logic.searchFocus,
-            ),
-          ),
-        ),
-      );
-
-      logic.syncDrama(
-        active: true,
-        wasActive: true,
-        nextDrama: logic.drama,
-        dramaChanged: false,
-        searchRequest: 1,
-        previousSearchRequest: 0,
-        isActive: () => true,
-      );
-
-      expect(logic.searchOpen, isTrue);
-      await tester.pumpAndSettle();
-      expect(logic.searchFocus.hasFocus, isTrue);
-
-      await tester.pumpWidget(const SizedBox.shrink());
-      logic.dispose();
-    },
-  );
-
-  testWidgets('should_keep_search_field_closed_when_page_is_hidden', (
+  testWidgets('should_focus_search_only_for_new_request_on_active_page', (
     tester,
   ) async {
-    final logic = DiscoverLogic(
-      history: FakeWatchHistory(),
-      drama: _drama('s'),
-    );
+    final logic = DiscoverLogic();
     await tester.pumpWidget(
       MaterialApp(
         home: Scaffold(
@@ -261,33 +38,33 @@ void main() {
       ),
     );
 
-    // The search request branch is gated on `active`, so a page that is not on
-    // screen must neither reveal the field nor pull focus away from the visible tab.
-    logic.syncDrama(
+    logic.syncSearch(
       active: false,
-      wasActive: false,
-      nextDrama: logic.drama,
-      dramaChanged: false,
-      searchRequest: 1,
-      previousSearchRequest: 0,
+      request: 1,
+      previousRequest: 0,
       isActive: () => false,
     );
-
-    expect(logic.searchOpen, isFalse);
-    await tester.pumpAndSettle();
+    await tester.pump();
     expect(logic.searchFocus.hasFocus, isFalse);
 
+    logic.syncSearch(
+      active: true,
+      request: 2,
+      previousRequest: 1,
+      isActive: () => true,
+    );
+    await tester.pump();
+    await tester.pump();
+    expect(logic.searchFocus.hasFocus, isTrue);
+
     await tester.pumpWidget(const SizedBox.shrink());
     logic.dispose();
   });
 
-  testWidgets('should_return_choice_and_forward_it_when_act_runs_embedded', (
+  testWidgets('should_forward_action_when_embedded_or_pop_when_standalone', (
     tester,
   ) async {
-    final logic = DiscoverLogic(
-      history: FakeWatchHistory(),
-      drama: _drama('s'),
-    );
+    final logic = DiscoverLogic();
     final forwarded = <DiscoverPageAction>[];
     final popped = Completer<DiscoverPageAction?>();
     late BuildContext pageContext;
@@ -311,7 +88,6 @@ void main() {
         ),
       ),
     );
-
     await tester.tap(find.text('open'));
     await tester.pumpAndSettle();
 
@@ -322,83 +98,15 @@ void main() {
       onAction: forwarded.add,
     );
     expect(forwarded, [DiscoverPageAction.home]);
-    expect(Navigator.of(pageContext).canPop(), isTrue);
 
-    logic.act(pageContext, DiscoverPageAction.watchlist, embedded: false);
+    logic.act(pageContext, DiscoverPageAction.my, embedded: false);
     await tester.pumpAndSettle();
-
-    await expectLater(popped.future, completion(DiscoverPageAction.watchlist));
-    logic.dispose();
-  });
-
-  testWidgets('should_show_artwork_notice_when_card_has_no_playable_media', (
-    tester,
-  ) async {
-    final logic = DiscoverLogic(
-      history: FakeWatchHistory(),
-      drama: _drama('s'),
-    );
-    late BuildContext context;
-
-    await tester.pumpWidget(
-      MaterialApp(
-        locale: const Locale('en'),
-        localizationsDelegates: AppLocalizations.localizationsDelegates,
-        supportedLocales: AppLocalizations.supportedLocales,
-        home: Builder(
-          builder: (inner) {
-            context = inner;
-            return const Scaffold(body: SizedBox.shrink());
-          },
-        ),
-      ),
-    );
-
-    logic.openCard(context, null, embedded: false);
-    await tester.pumpAndSettle();
-
-    final l10n = AppLocalizations.of(context)!;
-    expect(find.text(l10n.artworkOnlyNotice), findsOneWidget);
-    logic.dispose();
-  });
-
-  testWidgets('should_forward_chosen_drama_when_card_is_embedded', (
-    tester,
-  ) async {
-    final logic = DiscoverLogic(
-      history: FakeWatchHistory(),
-      drama: _drama('s'),
-    );
-    late BuildContext context;
-    Drama? selected;
-
-    await tester.pumpWidget(
-      MaterialApp(
-        home: Builder(
-          builder: (inner) {
-            context = inner;
-            return const SizedBox.shrink();
-          },
-        ),
-      ),
-    );
-
-    logic.openCard(
-      context,
-      _drama('city'),
-      embedded: true,
-      onDramaSelected: (drama) => selected = drama,
-    );
-
-    expect(selected?.id, 'city');
+    await expectLater(popped.future, completion(DiscoverPageAction.my));
     logic.dispose();
   });
 
   test('should_do_nothing_when_notifying_after_dispose', () {
-    final logic = DiscoverLogic(
-      history: FakeWatchHistory(),
-      drama: _drama('s'),
-    );
+    final logic = DiscoverLogic();
     var notifications = 0;
     logic.addListener(() => notifications++);