Adam GastineauandGitHub
8dcfd36fa5
* chore(mobile): move Drift into mobile/data * Move into more structured subdirectories * Fix changed merge paths
78 lines
2.6 KiB
Dart
78 lines
2.6 KiB
Dart
import 'package:flutter_test/flutter_test.dart';
|
|
import 'package:immich_mobile/data/db/main/dao/person.dart';
|
|
|
|
import '../repository_context.dart';
|
|
|
|
void main() {
|
|
late MediumRepositoryContext ctx;
|
|
late PeopleRepository sut;
|
|
|
|
setUp(() {
|
|
ctx = MediumRepositoryContext();
|
|
sut = PeopleRepository(ctx.db);
|
|
});
|
|
|
|
tearDown(() async {
|
|
await ctx.dispose();
|
|
});
|
|
|
|
group('getAssetPeople', () {
|
|
test('does not duplicate a person with multiple face records on the same asset', () async {
|
|
// Regression check for #20585: a join on asset_face_entity returned one row
|
|
// per face, so a person appeared twice in the asset details panel when the
|
|
// same face was on the asset more than once (e.g., metadata import + ML)
|
|
final user = await ctx.newUser();
|
|
final asset = await ctx.newRemoteAsset(ownerId: user.id);
|
|
|
|
final person = await ctx.newPerson(ownerId: user.id);
|
|
await ctx.newFace(assetId: asset.id, personId: person.id);
|
|
await ctx.newFace(assetId: asset.id, personId: person.id);
|
|
|
|
final people = await sut.getAssetPeople(asset.id);
|
|
|
|
expect(people, hasLength(1));
|
|
expect(people.single.id, person.id);
|
|
});
|
|
|
|
test('returns all distinct people of an asset', () async {
|
|
final user = await ctx.newUser();
|
|
final asset = await ctx.newRemoteAsset(ownerId: user.id);
|
|
|
|
final person1 = await ctx.newPerson(ownerId: user.id);
|
|
final person2 = await ctx.newPerson(ownerId: user.id);
|
|
await ctx.newFace(assetId: asset.id, personId: person1.id);
|
|
await ctx.newFace(assetId: asset.id, personId: person2.id);
|
|
|
|
final people = await sut.getAssetPeople(asset.id);
|
|
|
|
expect(people, hasLength(2));
|
|
expect(people.map((person) => person.id), containsAll([person1.id, person2.id]));
|
|
});
|
|
|
|
test('does not return hidden people', () async {
|
|
final user = await ctx.newUser();
|
|
final asset = await ctx.newRemoteAsset(ownerId: user.id);
|
|
|
|
final hidden = await ctx.newPerson(ownerId: user.id, isHidden: true);
|
|
await ctx.newFace(assetId: asset.id, personId: hidden.id);
|
|
|
|
final people = await sut.getAssetPeople(asset.id);
|
|
|
|
expect(people, isEmpty);
|
|
});
|
|
|
|
test('does not return people from other assets', () async {
|
|
final user = await ctx.newUser();
|
|
final asset = await ctx.newRemoteAsset(ownerId: user.id);
|
|
final otherAsset = await ctx.newRemoteAsset(ownerId: user.id);
|
|
|
|
final person = await ctx.newPerson(ownerId: user.id);
|
|
await ctx.newFace(assetId: otherAsset.id, personId: person.id);
|
|
|
|
final people = await sut.getAssetPeople(asset.id);
|
|
|
|
expect(people, isEmpty);
|
|
});
|
|
});
|
|
}
|