mirror of
https://github.com/immich-app/immich.git
synced 2025-01-13 15:35:15 +02:00
chore(mobile): add tests for SearchPage
This commit is contained in:
parent
eb8f4898b3
commit
684d63c810
@ -464,6 +464,7 @@ class SearchPage extends HookConsumerWidget {
|
|||||||
Padding(
|
Padding(
|
||||||
padding: const EdgeInsets.only(right: 14.0),
|
padding: const EdgeInsets.only(right: 14.0),
|
||||||
child: IconButton(
|
child: IconButton(
|
||||||
|
key: const Key('contextual_search_button'),
|
||||||
icon: isContextualSearch.value
|
icon: isContextualSearch.value
|
||||||
? const Icon(Icons.abc_rounded)
|
? const Icon(Icons.abc_rounded)
|
||||||
: const Icon(Icons.image_search_rounded),
|
: const Icon(Icons.image_search_rounded),
|
||||||
@ -492,6 +493,7 @@ class SearchPage extends HookConsumerWidget {
|
|||||||
),
|
),
|
||||||
),
|
),
|
||||||
child: TextField(
|
child: TextField(
|
||||||
|
key: const Key('search_text_field'),
|
||||||
controller: textSearchController,
|
controller: textSearchController,
|
||||||
decoration: InputDecoration(
|
decoration: InputDecoration(
|
||||||
contentPadding: prefilter != null
|
contentPadding: prefilter != null
|
||||||
@ -547,6 +549,7 @@ class SearchPage extends HookConsumerWidget {
|
|||||||
child: SizedBox(
|
child: SizedBox(
|
||||||
height: 50,
|
height: 50,
|
||||||
child: ListView(
|
child: ListView(
|
||||||
|
key: const Key('search_filter_chip_list'),
|
||||||
shrinkWrap: true,
|
shrinkWrap: true,
|
||||||
scrollDirection: Axis.horizontal,
|
scrollDirection: Axis.horizontal,
|
||||||
padding: const EdgeInsets.symmetric(horizontal: 16),
|
padding: const EdgeInsets.symmetric(horizontal: 16),
|
||||||
@ -576,6 +579,7 @@ class SearchPage extends HookConsumerWidget {
|
|||||||
currentFilter: dateRangeCurrentFilterWidget.value,
|
currentFilter: dateRangeCurrentFilterWidget.value,
|
||||||
),
|
),
|
||||||
SearchFilterChip(
|
SearchFilterChip(
|
||||||
|
key: const Key('media_type_chip'),
|
||||||
icon: Icons.video_collection_outlined,
|
icon: Icons.video_collection_outlined,
|
||||||
onTap: showMediaTypePicker,
|
onTap: showMediaTypePicker,
|
||||||
label: 'search_filter_media_type'.tr(),
|
label: 'search_filter_media_type'.tr(),
|
||||||
|
@ -17,6 +17,7 @@ class MediaTypePicker extends HookWidget {
|
|||||||
shrinkWrap: true,
|
shrinkWrap: true,
|
||||||
children: [
|
children: [
|
||||||
RadioListTile(
|
RadioListTile(
|
||||||
|
key: const Key("search_filter_media_type_all"),
|
||||||
title: const Text("search_filter_media_type_all").tr(),
|
title: const Text("search_filter_media_type_all").tr(),
|
||||||
value: AssetType.other,
|
value: AssetType.other,
|
||||||
onChanged: (value) {
|
onChanged: (value) {
|
||||||
@ -26,6 +27,7 @@ class MediaTypePicker extends HookWidget {
|
|||||||
groupValue: selectedMediaType.value,
|
groupValue: selectedMediaType.value,
|
||||||
),
|
),
|
||||||
RadioListTile(
|
RadioListTile(
|
||||||
|
key: const Key("search_filter_media_type_image"),
|
||||||
title: const Text("search_filter_media_type_image").tr(),
|
title: const Text("search_filter_media_type_image").tr(),
|
||||||
value: AssetType.image,
|
value: AssetType.image,
|
||||||
onChanged: (value) {
|
onChanged: (value) {
|
||||||
@ -35,6 +37,7 @@ class MediaTypePicker extends HookWidget {
|
|||||||
groupValue: selectedMediaType.value,
|
groupValue: selectedMediaType.value,
|
||||||
),
|
),
|
||||||
RadioListTile(
|
RadioListTile(
|
||||||
|
key: const Key("search_filter_media_type_video"),
|
||||||
title: const Text("search_filter_media_type_video").tr(),
|
title: const Text("search_filter_media_type_video").tr(),
|
||||||
value: AssetType.video,
|
value: AssetType.video,
|
||||||
onChanged: (value) {
|
onChanged: (value) {
|
||||||
|
6
mobile/test/dto.mocks.dart
Normal file
6
mobile/test/dto.mocks.dart
Normal file
@ -0,0 +1,6 @@
|
|||||||
|
import 'package:mocktail/mocktail.dart';
|
||||||
|
import 'package:openapi/api.dart';
|
||||||
|
|
||||||
|
class MockSmartSearchDto extends Mock implements SmartSearchDto {}
|
||||||
|
|
||||||
|
class MockMetadataSearchDto extends Mock implements MetadataSearchDto {}
|
186
mobile/test/pages/search/search.page_test.dart
Normal file
186
mobile/test/pages/search/search.page_test.dart
Normal file
@ -0,0 +1,186 @@
|
|||||||
|
import 'package:flutter/material.dart';
|
||||||
|
import 'package:flutter_test/flutter_test.dart';
|
||||||
|
import 'package:hooks_riverpod/hooks_riverpod.dart';
|
||||||
|
import 'package:immich_mobile/entities/store.entity.dart';
|
||||||
|
import 'package:immich_mobile/pages/search/search.page.dart';
|
||||||
|
import 'package:immich_mobile/providers/api.provider.dart';
|
||||||
|
import 'package:immich_mobile/providers/db.provider.dart';
|
||||||
|
import 'package:immich_mobile/providers/search/paginated_search.provider.dart';
|
||||||
|
import 'package:immich_mobile/widgets/asset_grid/asset_grid_data_structure.dart';
|
||||||
|
import 'package:isar/isar.dart';
|
||||||
|
import 'package:mocktail/mocktail.dart';
|
||||||
|
import 'package:openapi/api.dart';
|
||||||
|
|
||||||
|
import '../../dto.mocks.dart';
|
||||||
|
import '../../service.mocks.dart';
|
||||||
|
import '../../test_utils.dart';
|
||||||
|
import '../../widget_tester_extensions.dart';
|
||||||
|
|
||||||
|
void main() {
|
||||||
|
late List<Override> overrides;
|
||||||
|
late Isar db;
|
||||||
|
late MockApiService mockApiService;
|
||||||
|
late MockSearchApi mockSearchApi;
|
||||||
|
|
||||||
|
setUpAll(() async {
|
||||||
|
TestUtils.init();
|
||||||
|
db = await TestUtils.initIsar();
|
||||||
|
Store.init(db);
|
||||||
|
mockApiService = MockApiService();
|
||||||
|
mockSearchApi = MockSearchApi();
|
||||||
|
when(() => mockApiService.searchApi).thenReturn(mockSearchApi);
|
||||||
|
registerFallbackValue(MockSmartSearchDto());
|
||||||
|
registerFallbackValue(MockMetadataSearchDto());
|
||||||
|
overrides = [
|
||||||
|
paginatedSearchRenderListProvider
|
||||||
|
.overrideWithValue(AsyncValue.data(RenderList.empty())),
|
||||||
|
dbProvider.overrideWithValue(db),
|
||||||
|
apiServiceProvider.overrideWithValue(mockApiService),
|
||||||
|
];
|
||||||
|
});
|
||||||
|
|
||||||
|
final emptyTextSearch = isA<MetadataSearchDto>()
|
||||||
|
.having((s) => s.originalFileName, 'originalFileName', null);
|
||||||
|
|
||||||
|
testWidgets('contextual search with/without text', (tester) async {
|
||||||
|
await tester.pumpConsumerWidget(
|
||||||
|
const SearchPage(),
|
||||||
|
overrides: overrides,
|
||||||
|
);
|
||||||
|
|
||||||
|
await tester.pumpAndSettle();
|
||||||
|
|
||||||
|
expect(
|
||||||
|
find.byIcon(Icons.abc_rounded),
|
||||||
|
findsOneWidget,
|
||||||
|
reason: 'Should have contextual search icon',
|
||||||
|
);
|
||||||
|
|
||||||
|
final searchField = find.byKey(const Key('search_text_field'));
|
||||||
|
expect(searchField, findsOneWidget);
|
||||||
|
|
||||||
|
await tester.enterText(searchField, 'test');
|
||||||
|
await tester.testTextInput.receiveAction(TextInputAction.search);
|
||||||
|
|
||||||
|
var captured = verify(
|
||||||
|
() => mockSearchApi.searchSmart(captureAny()),
|
||||||
|
).captured;
|
||||||
|
|
||||||
|
expect(
|
||||||
|
captured.first,
|
||||||
|
isA<SmartSearchDto>().having((s) => s.query, 'query', 'test'),
|
||||||
|
);
|
||||||
|
|
||||||
|
await tester.enterText(searchField, '');
|
||||||
|
await tester.testTextInput.receiveAction(TextInputAction.search);
|
||||||
|
|
||||||
|
captured = verify(() => mockSearchApi.searchAssets(captureAny())).captured;
|
||||||
|
expect(captured.first, emptyTextSearch);
|
||||||
|
});
|
||||||
|
|
||||||
|
testWidgets('not contextual search with/without text', (tester) async {
|
||||||
|
await tester.pumpConsumerWidget(
|
||||||
|
const SearchPage(),
|
||||||
|
overrides: overrides,
|
||||||
|
);
|
||||||
|
|
||||||
|
await tester.pumpAndSettle();
|
||||||
|
|
||||||
|
await tester.tap(find.byKey(const Key('contextual_search_button')));
|
||||||
|
|
||||||
|
await tester.pumpAndSettle();
|
||||||
|
|
||||||
|
expect(
|
||||||
|
find.byIcon(Icons.image_search_rounded),
|
||||||
|
findsOneWidget,
|
||||||
|
reason: 'Should not have contextual search icon',
|
||||||
|
);
|
||||||
|
|
||||||
|
final searchField = find.byKey(const Key('search_text_field'));
|
||||||
|
expect(searchField, findsOneWidget);
|
||||||
|
|
||||||
|
await tester.enterText(searchField, 'test');
|
||||||
|
await tester.testTextInput.receiveAction(TextInputAction.search);
|
||||||
|
|
||||||
|
var captured = verify(
|
||||||
|
() => mockSearchApi.searchAssets(captureAny()),
|
||||||
|
).captured;
|
||||||
|
|
||||||
|
expect(
|
||||||
|
captured.first,
|
||||||
|
isA<MetadataSearchDto>()
|
||||||
|
.having((s) => s.originalFileName, 'originalFileName', 'test'),
|
||||||
|
);
|
||||||
|
|
||||||
|
await tester.enterText(searchField, '');
|
||||||
|
await tester.testTextInput.receiveAction(TextInputAction.search);
|
||||||
|
|
||||||
|
captured = verify(() => mockSearchApi.searchAssets(captureAny())).captured;
|
||||||
|
expect(captured.first, emptyTextSearch);
|
||||||
|
});
|
||||||
|
|
||||||
|
testWidgets('contextual search with text combined with media type',
|
||||||
|
(tester) async {
|
||||||
|
await tester.pumpConsumerWidget(
|
||||||
|
const SearchPage(),
|
||||||
|
overrides: overrides,
|
||||||
|
);
|
||||||
|
|
||||||
|
await tester.pumpAndSettle();
|
||||||
|
|
||||||
|
expect(
|
||||||
|
find.byIcon(Icons.abc_rounded),
|
||||||
|
findsOneWidget,
|
||||||
|
reason: 'Should have contextual search icon',
|
||||||
|
);
|
||||||
|
|
||||||
|
final searchField = find.byKey(const Key('search_text_field'));
|
||||||
|
expect(searchField, findsOneWidget);
|
||||||
|
|
||||||
|
await tester.enterText(searchField, 'test');
|
||||||
|
await tester.testTextInput.receiveAction(TextInputAction.search);
|
||||||
|
|
||||||
|
var captured = verify(
|
||||||
|
() => mockSearchApi.searchSmart(captureAny()),
|
||||||
|
).captured;
|
||||||
|
|
||||||
|
expect(
|
||||||
|
captured.first,
|
||||||
|
isA<SmartSearchDto>().having((s) => s.query, 'query', 'test'),
|
||||||
|
);
|
||||||
|
|
||||||
|
await tester.dragUntilVisible(
|
||||||
|
find.byKey(const Key('media_type_chip')),
|
||||||
|
find.byKey(const Key('search_filter_chip_list')),
|
||||||
|
const Offset(-100, 0),
|
||||||
|
);
|
||||||
|
await tester.pumpAndSettle();
|
||||||
|
|
||||||
|
await tester.tap(find.byKey(const Key('media_type_chip')));
|
||||||
|
await tester.pumpAndSettle();
|
||||||
|
|
||||||
|
await tester.tap(find.byKey(const Key('search_filter_media_type_image')));
|
||||||
|
await tester.tap(find.byKey(const Key('search_filter_apply')));
|
||||||
|
await tester.pumpAndSettle();
|
||||||
|
|
||||||
|
captured = verify(() => mockSearchApi.searchSmart(captureAny())).captured;
|
||||||
|
|
||||||
|
expect(
|
||||||
|
captured.first,
|
||||||
|
isA<SmartSearchDto>()
|
||||||
|
.having((s) => s.query, 'query', 'test')
|
||||||
|
.having((s) => s.type, 'type', AssetTypeEnum.IMAGE),
|
||||||
|
);
|
||||||
|
|
||||||
|
await tester.enterText(searchField, '');
|
||||||
|
await tester.testTextInput.receiveAction(TextInputAction.search);
|
||||||
|
|
||||||
|
captured = verify(() => mockSearchApi.searchAssets(captureAny())).captured;
|
||||||
|
expect(
|
||||||
|
captured.first,
|
||||||
|
isA<MetadataSearchDto>()
|
||||||
|
.having((s) => s.originalFileName, 'originalFileName', null)
|
||||||
|
.having((s) => s.type, 'type', AssetTypeEnum.IMAGE),
|
||||||
|
);
|
||||||
|
});
|
||||||
|
}
|
@ -5,6 +5,7 @@ import 'package:immich_mobile/services/network.service.dart';
|
|||||||
import 'package:immich_mobile/services/sync.service.dart';
|
import 'package:immich_mobile/services/sync.service.dart';
|
||||||
import 'package:immich_mobile/services/user.service.dart';
|
import 'package:immich_mobile/services/user.service.dart';
|
||||||
import 'package:mocktail/mocktail.dart';
|
import 'package:mocktail/mocktail.dart';
|
||||||
|
import 'package:openapi/api.dart';
|
||||||
|
|
||||||
class MockApiService extends Mock implements ApiService {}
|
class MockApiService extends Mock implements ApiService {}
|
||||||
|
|
||||||
@ -17,3 +18,5 @@ class MockHashService extends Mock implements HashService {}
|
|||||||
class MockEntityService extends Mock implements EntityService {}
|
class MockEntityService extends Mock implements EntityService {}
|
||||||
|
|
||||||
class MockNetworkService extends Mock implements NetworkService {}
|
class MockNetworkService extends Mock implements NetworkService {}
|
||||||
|
|
||||||
|
class MockSearchApi extends Mock implements SearchApi {}
|
||||||
|
Loading…
Reference in New Issue
Block a user