重构:宠物列表/详情/编辑页接入真实数据(T2-12,四态齐备)
CI / flutter-gates (push) Successful in 1m31s

- 档案 Tab 替换为真实宠物列表(05 号规范 P1):PetsController 四态
  驱动(loading/empty/error+重试/ready),空态插画 + 建档 CTA,
  下拉刷新,虚线添加卡;页面一律走 T2-11 数据层、不直连 ApiClient
- 新增 PetDetailPage(P2 档案信息部分):内存副本首屏 + getPet 刷新,
  40401 防枚举 → 不存在态返回列表并刷新;仅 owner 显示编辑入口
  (40300 语义);体重/疫苗/时间线区块留待 T2-13/14 接摘要与记录接口
- 新增 PetFormPage(建档/编辑):品种目录接口 + 自定义品种互斥
  (目录失败回落自定义 + 重试)、性别契约必填、生日+估算、芯片号;
  失焦+提交双校验、错误三层模型沿用登录纵切
- 40902 版本冲突:明确提示 + 自动拉取最新 version 重提路径(有测试);
  40903 芯片号冲突字段级报错(有测试)
- 埋点挂接:pet_create_started(表单首次输入、每次进入一次)/
  succeeded / failed 三事件接线;建宠表单路由名 pet_form、详情页
  pet_detail 进入既有 RouteObserver 采集;档案 Tab 页名 pet_archive
  改报 pet_list(字典 v2);编辑表单不带路由名(漏斗到达段不掺编辑)
- app.dart 装配:pet 服务 ApiClient(:8083)共享 TokenRefresher;
  登出 reset 控制器防跨账号泄漏;PetsController.loadBreeds 按物种缓存
- AppState 清理:pets 页原 demo 消费方移除,vaccines 及其模型/持久化
  删除;pet demo 仅存首页问候/创作页/主壳头像(后续工单收敛)
- 头像按 ADR-010 本地占位,不做上传
- 测试 145 → 177 全绿(列表/详情/表单四态与冲突路径 widget 测试、
  控制器 loadBreeds/reset、展示纯函数);analyze 0;format 无 diff

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
This commit is contained in:
2026-09-08 12:15:32 +08:00
parent c0a8a56e91
commit 97a1f46e3f
17 changed files with 2249 additions and 852 deletions
+63 -40
View File
@@ -3,40 +3,9 @@ import 'package:patbond_flutter/core/network/api_exception.dart';
import 'package:patbond_flutter/features/pets/pet_exceptions.dart';
import 'package:patbond_flutter/features/pets/pet_models.dart';
import 'package:patbond_flutter/features/pets/pets_controller.dart';
import 'package:patbond_flutter/features/pets/pets_repository.dart';
import '../../helpers/pet_test_helpers.dart';
/// 假仓库:各方法可注入行为,未注入的方法抛 UnimplementedError。
class FakePetsRepository implements PetsRepository {
Future<List<Pet>> Function()? listPetsHandler;
Future<Pet> Function(CreatePetRequest)? createPetHandler;
Future<Pet> Function(String)? getPetHandler;
Future<Pet> Function(String, UpdatePetRequest)? updatePetHandler;
@override
Future<List<Pet>> listPets() =>
listPetsHandler?.call() ?? Future.value(const []);
@override
Future<Pet> createPet(CreatePetRequest request) => createPetHandler!(request);
@override
Future<Pet> getPet(String petId) => getPetHandler!(petId);
@override
Future<Pet> updatePet(String petId, UpdatePetRequest request) =>
updatePetHandler!(petId, request);
@override
dynamic noSuchMethod(Invocation invocation) =>
throw UnimplementedError('${invocation.memberName}');
}
Pet pet(String id, {String name = '豆豆', int version = 1}) => Pet.fromJson(
samplePetJson(overrides: {'id': id, 'name': name, 'version': version}),
);
void main() {
late FakePetsRepository repository;
late PetsController controller;
@@ -53,7 +22,7 @@ void main() {
});
test('refresh 成功:loading → ready,列表就位', () async {
repository.listPetsHandler = () async => [pet('p-1'), pet('p-2')];
repository.listPetsHandler = () async => [buildPet('p-1'), buildPet('p-2')];
final phases = <PetsLoadPhase>[];
controller.addListener(() => phases.add(controller.phase));
@@ -83,7 +52,7 @@ void main() {
expect(controller.phase, PetsLoadPhase.error);
expect(controller.lastError, isA<ApiNetworkException>());
repository.listPetsHandler = () async => [pet('p-1')];
repository.listPetsHandler = () async => [buildPet('p-1')];
await controller.refresh();
expect(controller.phase, PetsLoadPhase.ready);
@@ -92,9 +61,10 @@ void main() {
});
test('createPet:成功后插入列表头', () async {
repository.listPetsHandler = () async => [pet('p-1')];
repository.listPetsHandler = () async => [buildPet('p-1')];
await controller.refresh();
repository.createPetHandler = (request) async => pet('p-2', name: '咪咪');
repository.createPetHandler = (request) async =>
buildPet('p-2', name: '咪咪');
final created = await controller.createPet(
const CreatePetRequest(
@@ -110,7 +80,7 @@ void main() {
});
test('createPet 失败(40903):类型化异常外抛,列表不变', () async {
repository.listPetsHandler = () async => [pet('p-1')];
repository.listPetsHandler = () async => [buildPet('p-1')];
await controller.refresh();
repository.createPetHandler = (request) async =>
throw const MicrochipTakenException(message: '芯片号已被登记');
@@ -130,10 +100,10 @@ void main() {
});
test('updatePet:成功后同步列表内存副本', () async {
repository.listPetsHandler = () async => [pet('p-1', version: 1)];
repository.listPetsHandler = () async => [buildPet('p-1', version: 1)];
await controller.refresh();
repository.updatePetHandler = (petId, request) async =>
pet('p-1', name: '豆豆二世', version: 2);
buildPet('p-1', name: '豆豆二世', version: 2);
await controller.updatePet(
'p-1',
@@ -155,13 +125,66 @@ void main() {
});
test('getPet:详情结果回写列表副本', () async {
repository.listPetsHandler = () async => [pet('p-1', version: 1)];
repository.listPetsHandler = () async => [buildPet('p-1', version: 1)];
await controller.refresh();
repository.getPetHandler = (petId) async => pet('p-1', version: 5);
repository.getPetHandler = (petId) async => buildPet('p-1', version: 5);
final detail = await controller.getPet('p-1');
expect(detail.version, 5);
expect(controller.pets.single.version, 5);
});
test('loadBreeds:按物种缓存,仓库只打一次', () async {
var calls = 0;
repository.listBreedsHandler = (species) async {
calls++;
return [Breed.fromJson(sampleBreedJson())];
};
final first = await controller.loadBreeds(PetSpecies.dog);
final second = await controller.loadBreeds(PetSpecies.dog);
expect(first.single.displayName, '柴犬');
expect(identical(first, second), isTrue);
expect(calls, 1);
await controller.loadBreeds(PetSpecies.cat);
expect(calls, 2);
});
test('loadBreeds 失败:异常外抛且不缓存,重试可恢复', () async {
var calls = 0;
repository.listBreedsHandler = (species) async {
calls++;
if (calls == 1) throw const ApiNetworkException('断网');
return [Breed.fromJson(sampleBreedJson())];
};
await expectLater(
controller.loadBreeds(PetSpecies.dog),
throwsA(isA<ApiNetworkException>()),
);
final breeds = await controller.loadBreeds(PetSpecies.dog);
expect(breeds, hasLength(1));
});
test('reset:登出清空回 initial(含品种缓存)', () async {
repository.listPetsHandler = () async => [buildPet('p-1')];
var breedCalls = 0;
repository.listBreedsHandler = (species) async {
breedCalls++;
return [Breed.fromJson(sampleBreedJson())];
};
await controller.refresh();
await controller.loadBreeds(PetSpecies.dog);
controller.reset();
expect(controller.phase, PetsLoadPhase.initial);
expect(controller.pets, isEmpty);
expect(controller.lastError, isNull);
await controller.loadBreeds(PetSpecies.dog);
expect(breedCalls, 2);
});
}