commit fbe9ff33e6692e64d6cd7f839a387920eb53f5b7
author Jason Simmons <jason-simmons@users.noreply.github.com> Tue Jan 31 14:17:43 2023 -0800
committer GitHub <noreply@github.com> Tue Jan 31 22:17:43 2023 +0000
tree a84d14eee276c3d7f371ef95f20f21d35bdd64d7
parent 60c1f293dc87f0a07459a173a39082d0455d65b2

Disable an inaccurate test assertion that will be fixed by an engine roll (#119653)

https://skia-review.googlesource.com/c/skia/+/619838 fixes a bug in
SkParagraph.  One of the selectable region tests relies on the old
behavior.

After that Skia change rolls into the framework, the update to the test
in https://github.com/flutter/flutter/pull/118902 should be landed.

packages/flutter/test/widgets/selectable_region_test.dart

diff --git a/packages/flutter/test/widgets/selectable_region_test.dart b/packages/flutter/test/widgets/selectable_region_test.dart
index 240df70..4e834f8 100644
--- a/packages/flutter/test/widgets/selectable_region_test.dart
+++ b/packages/flutter/test/widgets/selectable_region_test.dart
@@ -842,7 +842,8 @@
       // Add a little offset to cross the boundary between paragraph 2 and 3.
       await gesture.moveTo(textOffsetToPosition(paragraph3, 6) + const Offset(0, 1));
       expect(paragraph1.selections[0], const TextSelection(baseOffset: 2, extentOffset: 12));
-      expect(paragraph2.selections[0], const TextSelection(baseOffset: 0, extentOffset: 8));
+      // TODO(jsimmons): reenable this with updated expectations after https://skia-review.googlesource.com/c/skia/+/619838 lands
+      //expect(paragraph2.selections[0], const TextSelection(baseOffset: 0, extentOffset: 8));
       expect(paragraph3.selections[0], const TextSelection(baseOffset: 0, extentOffset: 6));
 
       await gesture.up();