Page MenuHomePhabricator

Remove UTF8 BMP unit test and replace it with a general UTF8 test
ClosedPublic

Authored by epriestley on Oct 2 2014, 3:31 PM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, Mar 28, 7:37 AM
Unknown Object (File)
Thu, Mar 28, 7:00 AM
Unknown Object (File)
Tue, Mar 26, 3:36 AM
Unknown Object (File)
Mon, Mar 25, 6:50 PM
Unknown Object (File)
Mon, Mar 25, 6:49 PM
Unknown Object (File)
Mon, Mar 25, 6:49 PM
Unknown Object (File)
Thu, Mar 21, 9:53 PM
Unknown Object (File)
Thu, Mar 21, 8:04 PM
Subscribers

Details

Summary

Ref T1191. After utf8mb4 conversion, these tests no longer pass because MySQL allows emoji and gclefs and such.

We could keep these tests running by keeping a ut8f_bin table around somewhere, but we have no other use cases for it and it does not seem worth the added complexity. All these BMP-only codepaths are on the way out.

Update the %s / %B test to make sure it's rejecting invalid byte sequences, which are still not permitted.

Test Plan

Tests now pass.

Diff Detail

Repository
rP Phabricator
Branch
utf3
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 2719
Build 2723: [Placeholder Plan] Wait for 30 Seconds

Event Timeline

epriestley retitled this revision from to Remove UTF8 BMP unit test and replace it with a general UTF8 test.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: btrahan.
btrahan edited edge metadata.
This revision is now accepted and ready to land.Oct 2 2014, 4:38 PM
This revision was automatically updated to reflect the committed changes.