DEV/flutter_flutter
1
0
Fork 0
mirror of https://github.com/flutter/flutter.git synced 2026-02-20 02:29:02 +08:00
Code Issues Packages Projects Releases Wiki Activity

Add more tests for CompositionAwareMixin (flutter/engine#44717)

Browse Source 
Adds missing tests for `CompositionAwareMixin`. Check https://github.com/flutter/engine/pull/44139#discussion_r1288733128 and https://github.com/flutter/engine/pull/44139#discussion_r1288009016

[C++, Objective-C, Java style guides]: https://github.com/flutter/engine/blob/main/CONTRIBUTING.md#style
This commit is contained in:
Amir Panahandeh 2023-08-16 03:38:42 +03:30 committed by GitHub
parent 7bf371d161
commit e9f54512c8
1 changed files with 80 additions and 9 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

89
engine/src/flutter/lib/web_ui/test/engine/composition_test.dart
View File

@ -106,14 +106,81 @@ Future<void> testMain() async {
});
group('determine composition state', () {
test('should return new composition state - compositing middle of text', () {
const int baseOffset = 100;
test('should return editing state if extentOffset is null', () {
final EditingState editingState = EditingState(text: 'Test');
final _MockWithCompositionAwareMixin mockWithCompositionAwareMixin =
_MockWithCompositionAwareMixin();
mockWithCompositionAwareMixin.composingText = 'Test';
expect(
mockWithCompositionAwareMixin.determineCompositionState(editingState),
editingState,
);
});
test('should return editing state if composingText is null', () {
final EditingState editingState = EditingState(
text: 'Test',
baseOffset: 0,
extentOffset: 4,
);
final _MockWithCompositionAwareMixin mockWithCompositionAwareMixin =
_MockWithCompositionAwareMixin();
expect(
mockWithCompositionAwareMixin.determineCompositionState(editingState),
editingState,
);
});
test('should return editing state if text is null', () {
final EditingState editingState = EditingState(
baseOffset: 0,
extentOffset: 0,
);
final _MockWithCompositionAwareMixin mockWithCompositionAwareMixin =
_MockWithCompositionAwareMixin();
mockWithCompositionAwareMixin.composingText = 'Test';
expect(
mockWithCompositionAwareMixin.determineCompositionState(editingState),
editingState,
);
});
test(
'should return editing state if extentOffset is smaller than composingText length',
() {
const String composingText = 'composeMe';
final EditingState editingState = EditingState(
extentOffset: baseOffset,
text: 'testing',
text: 'Test',
baseOffset: 0,
extentOffset: 4,
);
final _MockWithCompositionAwareMixin mockWithCompositionAwareMixin =
_MockWithCompositionAwareMixin();
mockWithCompositionAwareMixin.composingText = composingText;
expect(
mockWithCompositionAwareMixin.determineCompositionState(editingState),
editingState,
);
});
test('should return new composition state - compositing middle of text',
() {
const int baseOffset = 7;
const String composingText = 'Test';
final EditingState editingState = EditingState(
text: 'Testing',
baseOffset: baseOffset,
extentOffset: baseOffset,
);
final _MockWithCompositionAwareMixin mockWithCompositionAwareMixin =
@ -123,13 +190,17 @@ Future<void> testMain() async {
const int expectedComposingBase = baseOffset - composingText.length;
expect(
mockWithCompositionAwareMixin.determineCompositionState(editingState),
editingState.copyWith(
composingBaseOffset: expectedComposingBase,
composingExtentOffset: expectedComposingBase + composingText.length));
mockWithCompositionAwareMixin.determineCompositionState(editingState),
editingState.copyWith(
composingBaseOffset: expectedComposingBase,
composingExtentOffset: expectedComposingBase + composingText.length,
),
);
});
test('should return new composition state - compositing from beginning of text', () {
test(
'should return new composition state - compositing from beginning of text',
() {
const String composingText = '今日は';
final EditingState editingState = EditingState(