feat(gestion): suppression en cascade exercice/programme/séance-modèle (ticket #37)
Ajoute la suppression avec gestion des dépendances en cascade sur les écrans bibliothèque d'exercices, programme et séance-modèle, avec les use cases et le bootstrap applicatif associés. flutter analyze propre, 58/58 tests verts, build APK debug validé. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
This commit is contained in:
@ -156,6 +156,165 @@ void main() {
|
||||
);
|
||||
});
|
||||
|
||||
test('delete exercise removes it from referencing programs', () async {
|
||||
final exerciseRepository = _FakeExerciseRepository()
|
||||
..exercise = Exercise(
|
||||
metadata: _metadata('exercise-1'),
|
||||
name: 'Squat',
|
||||
hasTimeMeasure: false,
|
||||
hasRepsMeasure: true,
|
||||
hasScoreMeasure: false,
|
||||
defaultTargetReps: 10,
|
||||
);
|
||||
final programRepository = _FakeProgramRepository()
|
||||
..programs.add(
|
||||
Program(
|
||||
metadata: _metadata('program-1'),
|
||||
name: 'Jambes',
|
||||
defaultRestSeconds: 60,
|
||||
exercises: [
|
||||
_programExercise(
|
||||
id: 'program-exercise-1',
|
||||
programId: 'program-1',
|
||||
sourceExerciseId: 'exercise-1',
|
||||
position: 0,
|
||||
),
|
||||
_programExercise(
|
||||
id: 'program-exercise-2',
|
||||
programId: 'program-1',
|
||||
sourceExerciseId: 'exercise-2',
|
||||
position: 1,
|
||||
),
|
||||
],
|
||||
),
|
||||
);
|
||||
final useCase = _exerciseUseCase(
|
||||
exerciseRepository,
|
||||
programRepository: programRepository,
|
||||
);
|
||||
|
||||
final deleted = await useCase.delete('exercise-1');
|
||||
|
||||
expect(deleted.metadata.deletedAt, isNotNull);
|
||||
expect(exerciseRepository.listActive(), completion(isEmpty));
|
||||
expect(programRepository.programs.single.exercises, hasLength(1));
|
||||
expect(
|
||||
programRepository.programs.single.exercises.single.sourceExerciseId,
|
||||
'exercise-2',
|
||||
);
|
||||
expect(programRepository.programs.single.exercises.single.position, 0);
|
||||
});
|
||||
|
||||
test(
|
||||
'delete program removes it from referencing workout templates',
|
||||
() async {
|
||||
final programRepository = _FakeProgramRepository()
|
||||
..programs.add(
|
||||
Program(
|
||||
metadata: _metadata('program-1'),
|
||||
name: 'Jambes',
|
||||
defaultRestSeconds: 60,
|
||||
),
|
||||
);
|
||||
final templateRepository = _FakeWorkoutTemplateRepository()
|
||||
..templates.add(
|
||||
WorkoutTemplate(
|
||||
metadata: _metadata('template-1'),
|
||||
name: 'Séance',
|
||||
programs: [
|
||||
_templateProgram(
|
||||
id: 'template-program-1',
|
||||
workoutTemplateId: 'template-1',
|
||||
sourceProgramId: 'program-1',
|
||||
position: 0,
|
||||
),
|
||||
_templateProgram(
|
||||
id: 'template-program-2',
|
||||
workoutTemplateId: 'template-1',
|
||||
sourceProgramId: 'program-2',
|
||||
position: 1,
|
||||
),
|
||||
],
|
||||
overrides: [
|
||||
WorkoutTemplateExerciseOverride(
|
||||
metadata: _metadata('override-1'),
|
||||
workoutTemplateProgramId: 'template-program-1',
|
||||
snapshotProgramExerciseId: 'exercise-snapshot-1',
|
||||
targetRepsOverride: 8,
|
||||
),
|
||||
],
|
||||
),
|
||||
);
|
||||
final useCase = ProgramUseCases(
|
||||
programRepository: programRepository,
|
||||
exerciseRepository: _FakeExerciseRepository(),
|
||||
templateRepository: templateRepository,
|
||||
clock: _FakeClock(DateTime.utc(2026, 7, 17, 12)),
|
||||
ids: _FakeIds(),
|
||||
originDeviceId: 'device-1',
|
||||
);
|
||||
|
||||
final deleted = await useCase.delete('program-1');
|
||||
|
||||
expect(deleted.metadata.deletedAt, isNotNull);
|
||||
expect(programRepository.listActive(), completion(isEmpty));
|
||||
final template = templateRepository.templates.single;
|
||||
expect(template.programs, hasLength(1));
|
||||
expect(template.programs.single.sourceProgramId, 'program-2');
|
||||
expect(template.programs.single.position, 0);
|
||||
expect(template.overrides, isEmpty);
|
||||
},
|
||||
);
|
||||
|
||||
test('delete workout template hides it from active list', () async {
|
||||
final templateRepository = _FakeWorkoutTemplateRepository()
|
||||
..templates.add(
|
||||
WorkoutTemplate(metadata: _metadata('template-1'), name: 'Séance'),
|
||||
);
|
||||
final useCase = WorkoutTemplateUseCases(
|
||||
templateRepository: templateRepository,
|
||||
programRepository: _FakeProgramRepository(),
|
||||
clock: _FakeClock(DateTime.utc(2026, 7, 17, 12)),
|
||||
ids: _FakeIds(),
|
||||
originDeviceId: 'device-1',
|
||||
);
|
||||
|
||||
final deleted = await useCase.delete('template-1');
|
||||
|
||||
expect(deleted.metadata.deletedAt, isNotNull);
|
||||
expect(templateRepository.listActive(), completion(isEmpty));
|
||||
});
|
||||
|
||||
test('history stays unchanged after deleting its source template', () async {
|
||||
final templateRepository = _FakeWorkoutTemplateRepository()
|
||||
..templates.add(
|
||||
WorkoutTemplate(metadata: _metadata('template-1'), name: 'Séance'),
|
||||
);
|
||||
final history = WorkoutHistory(
|
||||
metadata: _metadata('history-1'),
|
||||
sourceWorkoutTemplateId: 'template-1',
|
||||
nameSnapshot: 'Séance historique',
|
||||
startedAt: DateTime.utc(2026, 7, 17, 10),
|
||||
endedAt: DateTime.utc(2026, 7, 17, 11),
|
||||
totalActiveMs: 3600000,
|
||||
completed: true,
|
||||
historySnapshotJson: '{"name":"Séance historique"}',
|
||||
);
|
||||
final useCase = WorkoutTemplateUseCases(
|
||||
templateRepository: templateRepository,
|
||||
programRepository: _FakeProgramRepository(),
|
||||
clock: _FakeClock(DateTime.utc(2026, 7, 17, 12)),
|
||||
ids: _FakeIds(),
|
||||
originDeviceId: 'device-1',
|
||||
);
|
||||
|
||||
await useCase.delete('template-1');
|
||||
|
||||
expect(history.sourceWorkoutTemplateId, 'template-1');
|
||||
expect(history.nameSnapshot, 'Séance historique');
|
||||
expect(history.historySnapshotJson, '{"name":"Séance historique"}');
|
||||
});
|
||||
|
||||
test('Workout template use case rejects active measure overrides', () async {
|
||||
final useCase = WorkoutTemplateUseCases(
|
||||
templateRepository: _FakeWorkoutTemplateRepository(),
|
||||
@ -612,15 +771,59 @@ final class _FakeIds implements IdGenerator {
|
||||
}
|
||||
}
|
||||
|
||||
ExerciseUseCases _exerciseUseCase(_FakeExerciseRepository repository) {
|
||||
ExerciseUseCases _exerciseUseCase(
|
||||
_FakeExerciseRepository repository, {
|
||||
_FakeProgramRepository? programRepository,
|
||||
}) {
|
||||
return ExerciseUseCases(
|
||||
repository: repository,
|
||||
programRepository: programRepository ?? _FakeProgramRepository(),
|
||||
clock: _FakeClock(DateTime.utc(2026, 7, 17, 12)),
|
||||
ids: _FakeIds(),
|
||||
originDeviceId: 'device-1',
|
||||
);
|
||||
}
|
||||
|
||||
ProgramExercise _programExercise({
|
||||
required String id,
|
||||
required String programId,
|
||||
required String sourceExerciseId,
|
||||
required int position,
|
||||
}) {
|
||||
return ProgramExercise(
|
||||
metadata: _metadata(id),
|
||||
programId: programId,
|
||||
sourceExerciseId: sourceExerciseId,
|
||||
position: position,
|
||||
exerciseNameSnapshot: 'Exercise $id',
|
||||
availableTimeSnapshot: false,
|
||||
availableRepsSnapshot: true,
|
||||
availableScoreSnapshot: false,
|
||||
setsCount: 3,
|
||||
timeEnabled: false,
|
||||
repsEnabled: true,
|
||||
scoreEnabled: false,
|
||||
targetReps: 10,
|
||||
);
|
||||
}
|
||||
|
||||
WorkoutTemplateProgram _templateProgram({
|
||||
required String id,
|
||||
required String workoutTemplateId,
|
||||
required String sourceProgramId,
|
||||
required int position,
|
||||
}) {
|
||||
return WorkoutTemplateProgram(
|
||||
metadata: _metadata(id),
|
||||
workoutTemplateId: workoutTemplateId,
|
||||
sourceProgramId: sourceProgramId,
|
||||
position: position,
|
||||
programNameSnapshot: 'Program $id',
|
||||
defaultRestSecondsSnapshot: 60,
|
||||
programSnapshotJson: '{"exercises":[]}',
|
||||
);
|
||||
}
|
||||
|
||||
final class _FakeExerciseRepository implements ExerciseRepository {
|
||||
Exercise? exercise;
|
||||
final saved = <Exercise>[];
|
||||
@ -635,7 +838,9 @@ final class _FakeExerciseRepository implements ExerciseRepository {
|
||||
|
||||
@override
|
||||
Future<List<Exercise>> listActive() async =>
|
||||
exercise == null ? const [] : [exercise!];
|
||||
exercise == null || exercise!.metadata.deletedAt != null
|
||||
? const []
|
||||
: [exercise!];
|
||||
|
||||
@override
|
||||
Future<void> save(Exercise exercise) async {
|
||||
@ -646,20 +851,38 @@ final class _FakeExerciseRepository implements ExerciseRepository {
|
||||
|
||||
final class _FakeWorkoutTemplateRepository
|
||||
implements WorkoutTemplateRepository {
|
||||
@override
|
||||
Future<WorkoutTemplate?> findById(String id) async => null;
|
||||
final templates = <WorkoutTemplate>[];
|
||||
|
||||
@override
|
||||
Future<List<WorkoutTemplate>> listActive() async => const [];
|
||||
Future<WorkoutTemplate?> findById(String id) async {
|
||||
for (final template in templates) {
|
||||
if (template.metadata.id == id && template.metadata.deletedAt == null) {
|
||||
return template;
|
||||
}
|
||||
}
|
||||
return null;
|
||||
}
|
||||
|
||||
@override
|
||||
Future<List<WorkoutTemplate>> listActive() async {
|
||||
return templates
|
||||
.where((template) => template.metadata.deletedAt == null)
|
||||
.toList();
|
||||
}
|
||||
|
||||
@override
|
||||
Future<void> replaceComposition(
|
||||
WorkoutTemplate template,
|
||||
DateTime deletedAt,
|
||||
) async {}
|
||||
) async {
|
||||
await save(template);
|
||||
}
|
||||
|
||||
@override
|
||||
Future<void> save(WorkoutTemplate template) async {}
|
||||
Future<void> save(WorkoutTemplate template) async {
|
||||
templates.removeWhere((saved) => saved.metadata.id == template.metadata.id);
|
||||
templates.add(template);
|
||||
}
|
||||
|
||||
@override
|
||||
Future<void> saveOverride(WorkoutTemplateExerciseOverride override) async {}
|
||||
@ -669,20 +892,54 @@ final class _FakeWorkoutTemplateRepository
|
||||
}
|
||||
|
||||
final class _FakeProgramRepository implements ProgramRepository {
|
||||
@override
|
||||
Future<Program?> findById(String id) async => null;
|
||||
final programs = <Program>[];
|
||||
|
||||
@override
|
||||
Future<List<Program>> listActive() async => const [];
|
||||
Future<Program?> findById(String id) async {
|
||||
for (final program in programs) {
|
||||
if (program.metadata.id == id && program.metadata.deletedAt == null) {
|
||||
return program;
|
||||
}
|
||||
}
|
||||
return null;
|
||||
}
|
||||
|
||||
@override
|
||||
Future<void> replaceExercises(Program program, DateTime deletedAt) async {}
|
||||
Future<List<Program>> listActive() async {
|
||||
return programs
|
||||
.where((program) => program.metadata.deletedAt == null)
|
||||
.toList();
|
||||
}
|
||||
|
||||
@override
|
||||
Future<void> save(Program program) async {}
|
||||
Future<void> replaceExercises(Program program, DateTime deletedAt) async {
|
||||
await save(program);
|
||||
}
|
||||
|
||||
@override
|
||||
Future<void> saveExercise(ProgramExercise exercise) async {}
|
||||
Future<void> save(Program program) async {
|
||||
programs.removeWhere((saved) => saved.metadata.id == program.metadata.id);
|
||||
programs.add(program);
|
||||
}
|
||||
|
||||
@override
|
||||
Future<void> saveExercise(ProgramExercise exercise) async {
|
||||
final index = programs.indexWhere(
|
||||
(program) => program.metadata.id == exercise.programId,
|
||||
);
|
||||
if (index == -1) {
|
||||
return;
|
||||
}
|
||||
final program = programs[index];
|
||||
programs[index] = program.copyWith(
|
||||
exercises: [
|
||||
...program.exercises.where(
|
||||
(saved) => saved.metadata.id != exercise.metadata.id,
|
||||
),
|
||||
exercise,
|
||||
],
|
||||
);
|
||||
}
|
||||
}
|
||||
|
||||
ActiveWorkoutSessionUseCases _activeUseCase(
|
||||
|
||||
Reference in New Issue
Block a user