feat(insights): review fixes

2026-11-11 16:05:11 +03:00
parent 5228d43f15
commit db3a32c12e
16 changed files with 694 additions and 232 deletions
--- a/test/features/insights/presentation/charts/insights_chart_common_test.dart
+++ b/test/features/insights/presentation/charts/insights_chart_common_test.dart
@@ -0,0 +1,76 @@
+import 'package:eatvio_app/features/insights/presentation/charts/insights_chart_common.dart';
+import 'package:flutter_test/flutter_test.dart';
+import 'package:syncfusion_flutter_charts/charts.dart';
+
+void main() {
+  test('insightsSplineType — monotonic, чтобы кривая не уходила ниже точек', () {
+    expect(insightsSplineType, SplineType.monotonic);
+  });
+
+  group('buildPoints', () {
+    test('сохраняет индекс и дату даже при повторяющихся labels', () {
+      final points = buildPoints(
+        ['05.07', '05.07', '06.07', '07.07'],
+        [10, 20, 30, 40],
+      );
+
+      expect(points.map((p) => p.index).toList(), [0, 1, 2, 3]);
+      expect(points.map((p) => p.label).toList(), [
+        '05.07',
+        '05.07',
+        '06.07',
+        '07.07',
+      ]);
+      expect(points.map((p) => p.value).toList(), [10, 20, 30, 40]);
+    });
+
+    test('обрезает по короткому массиву', () {
+      expect(buildPoints(['a', 'b'], [1]), hasLength(1));
+    });
+  });
+
+  group('buildInsightsIndexAxis', () {
+    test('ось строится по числу точек, а не по уникальным строкам дат', () {
+      final labels = [
+        '05.07',
+        '06.07',
+        '07.07',
+        '08.07',
+        '09.07',
+        '10.07',
+        '11.07',
+      ];
+      final axis = buildInsightsIndexAxis(labels);
+
+      expect(axis.minimum, 0);
+      expect(axis.maximum, 6);
+      expect(axis.interval, 1);
+      expect(axis.axisLabelFormatter, isNotNull);
+    });
+
+    test('для длинного периода ставит интервал больше 1', () {
+      final labels = List.generate(30, (i) => '${i + 1}.07');
+      final axis = buildInsightsIndexAxis(labels);
+
+      expect(axis.maximum, 29);
+      expect(axis.interval, greaterThan(1));
+    });
+  });
+
+  group('paddedRange', () {
+    test('даёт локальный диапазон вокруг данных', () {
+      final range = paddedRange([70, 72], minPadding: 1);
+
+      expect(range.min, lessThan(70));
+      expect(range.max, greaterThan(72));
+      expect(range.min, greaterThan(0));
+    });
+
+    test('clampZero не уводит минимум ниже нуля', () {
+      final range = paddedRange([2, 8], minPadding: 5, clampZero: true);
+
+      expect(range.min, 0);
+      expect(range.max, greaterThan(8));
+    });
+  });
+}
--- a/test/features/insights/presentation/cubit/insights_cubit_server_test.dart
+++ b/test/features/insights/presentation/cubit/insights_cubit_server_test.dart
@@ -121,7 +121,7 @@ void main() {
      await cubit.close();
    });

-    test('ошибка возвращает прежний период и оставляет данные', () async {
+    test('ошибка оставляет выбранный период и старые данные', () async {
      final cubit = buildCubit();
      await cubit.load();
      final before = loaded(cubit).payload;
@@ -129,7 +129,7 @@ void main() {

      await cubit.setDailyPeriod(DailyPeriod.week);

-      expect(loaded(cubit).dailyPeriod, DailyPeriod.twoWeeks);
+      expect(loaded(cubit).dailyPeriod, DailyPeriod.week);
      expect(loaded(cubit).payload, before);
      expect(loaded(cubit).isRefreshing, isFalse);
      await cubit.close();
@@ -179,6 +179,65 @@ void main() {
      await cubit.close();
    });

+    test(
+      'быстрые «назад» запрашивают разные дни (оптимистичная дата)',
+      () async {
+        repository.fetchIntraDelay = const Duration(milliseconds: 30);
+        repository.intraByDate['2026-11-07'] = yesterday.copyWith(
+          date: '2026-11-07',
+          dateLabel: '7 ноября',
+        );
+        final cubit = buildCubit();
+        await cubit.load();
+
+        final first = cubit.showPreviousDay();
+        final second = cubit.showPreviousDay();
+        await Future.wait([first, second]);
+
+        expect(repository.intraRequests.map((e) => e.date).toList(), [
+          '2026-11-08',
+          '2026-11-07',
+        ]);
+        expect(loaded(cubit).payload.intra!.date, '2026-11-07');
+        await cubit.close();
+      },
+    );
+
+    test('полный ответ не затирает более свежий intra и UI-выбор', () async {
+      repository.fetchInsightsDelay = const Duration(milliseconds: 40);
+      final cubit = buildCubit();
+      await cubit.load();
+      repository.fetchInsightsDelay = const Duration(milliseconds: 80);
+      repository.serverRequests.clear();
+
+      final periodChange = cubit.setBodyPeriod(BodyPeriod.year);
+      await Future<void>.delayed(const Duration(milliseconds: 10));
+      cubit.setPanel(InsightsPanel.measures);
+      cubit.toggleExpanded();
+      await cubit.setIntraDate(DateTime(2026, 11, 8));
+      await periodChange;
+
+      final state = loaded(cubit);
+      expect(state.bodyPeriod, BodyPeriod.year);
+      expect(state.payload.weight, repository.payload.weight);
+      expect(state.payload.intra, yesterday);
+      expect(state.intraDate, '2026-11-08');
+      expect(state.panel, InsightsPanel.measures);
+      expect(state.isExpanded, isTrue);
+      expect(state.isRefreshing, isFalse);
+      await cubit.close();
+    });
+
+    test('load с initialIntraDate запрашивает этот день', () async {
+      final cubit = buildCubit();
+
+      await cubit.load(intraDate: '2026-11-08');
+
+      expect(repository.serverRequests.single.intraDate, '2026-11-08');
+      expect(loaded(cubit).intraDate, '2026-11-08');
+      await cubit.close();
+    });
+
    test('showPreviousDay и showNextDay листают по дню', () async {
      final cubit = buildCubit();
      await cubit.load();
--- a/test/features/insights/presentation/cubit/insights_cubit_test.dart
+++ b/test/features/insights/presentation/cubit/insights_cubit_test.dart
@@ -200,7 +200,7 @@ void main() {
      await cubit.close();
    });

-    test('ошибка перезапроса возвращает прежний период и данные', () async {
+    test('ошибка перезапроса оставляет выбранный период и старые данные', () async {
      final cubit = buildCubit();
      await cubit.load();
      when(
@@ -213,7 +213,7 @@ void main() {

      await cubit.setBodyPeriod(BodyPeriod.all);

-      expect(loaded(cubit).bodyPeriod, BodyPeriod.threeMonths);
+      expect(loaded(cubit).bodyPeriod, BodyPeriod.all);
      expect(loaded(cubit).payload, _payload);
      expect(loaded(cubit).isRefreshing, isFalse);
      await cubit.close();
--- a/test/features/insights/presentation/widgets/insights_kbju_test.dart
+++ b/test/features/insights/presentation/widgets/insights_kbju_test.dart
@@ -97,7 +97,10 @@ void main() {
      await tester.pumpAndSettle();

      expect(repository.serverRequests.single.dailyPeriod, DailyPeriod.month);
-      expect(repository.serverRequests.single.bodyPeriod, BodyPeriod.threeMonths);
+      expect(
+        repository.serverRequests.single.bodyPeriod,
+        BodyPeriod.threeMonths,
+      );
    });

    testWidgets('без daily показывает заглушку', (tester) async {
@@ -150,32 +153,32 @@ void main() {
      }
    });

-    testWidgets('на сегодняшнем дне «следующий день» отключён, «предыдущий» нет', (
-      tester,
-    ) async {
-      await pumpKbjuCard(
-        tester,
-        FakeInsightsRepository(),
-        'insights_tab_intra',
-      );
+    testWidgets(
+      'на сегодняшнем дне «следующий день» отключён, «предыдущий» нет',
+      (tester) async {
+        await pumpKbjuCard(
+          tester,
+          FakeInsightsRepository(),
+          'insights_tab_intra',
+        );

-      IconButton buttonIn(String key) => tester.widget<IconButton>(
-        find.descendant(
-          of: find.byKey(Key(key)),
-          matching: find.byType(IconButton),
-        ),
-      );
+        IconButton buttonIn(String key) => tester.widget<IconButton>(
+          find.descendant(
+            of: find.byKey(Key(key)),
+            matching: find.byType(IconButton),
+          ),
+        );

-      expect(buttonIn('insights_intra_next').onPressed, isNull);
-      expect(buttonIn('insights_intra_prev').onPressed, isNotNull);
-    });
+        expect(buttonIn('insights_intra_next').onPressed, isNull);
+        expect(buttonIn('insights_intra_prev').onPressed, isNotNull);
+      },
+    );

    testWidgets('«предыдущий день» запрашивает только intra и меняет дату', (
      tester,
    ) async {
      final repository = FakeInsightsRepository();
-      repository.intraByDate['2026-11-08'] = loadInsightsPayloadFixture()
-          .intra!
+      repository.intraByDate['2026-11-08'] = loadInsightsPayloadFixture().intra!
          .copyWith(date: '2026-11-08', dateLabel: '8 ноября', isToday: false);
      await pumpKbjuCard(tester, repository, 'insights_tab_intra');
      repository.serverRequests.clear();
@@ -312,6 +315,34 @@ void main() {
      expect(find.byType(IntraLegend), findsOneWidget);
    });

+    testWidgets('день только со сном рисует затемнение, без заглушки', (
+      tester,
+    ) async {
+      final intra = loadInsightsPayloadFixture().intra!.copyWith(
+        bars: const [],
+        satiety: const [],
+        mood: const [],
+      );
+      await pumpKbjuCard(
+        tester,
+        FakeInsightsRepository(
+          payload: loadInsightsPayloadFixture().copyWith(intra: intra),
+        ),
+        'insights_tab_intra',
+      );
+
+      expect(find.byKey(const Key('insights_intra_chart')), findsOneWidget);
+      expect(find.text('Недостаточно данных'), findsNothing);
+      final xAxis =
+          tester
+                  .widget<SfCartesianChart>(
+                    find.byKey(const Key('insights_intra_chart')),
+                  )
+                  .primaryXAxis
+              as NumericAxis;
+      expect(xAxis.plotBands, isNotEmpty);
+    });
+
    testWidgets('пустой день показывает заглушку', (tester) async {
      await pumpKbjuCard(
        tester,