Page MenuHomePhabricator

Add test coverage that our definition of BMP agrees with MySQL
ClosedPublic

Authored by epriestley on Feb 23 2014, 8:38 PM.
Tags
None
Referenced Files
F13186414: D8314.diff
Sat, May 11, 3:37 AM
F13181056: D8314.id19781.diff
Thu, May 9, 7:58 AM
F13181054: D8314.id19766.diff
Thu, May 9, 7:58 AM
F13181053: D8314.id19765.diff
Thu, May 9, 7:58 AM
F13181038: D8314.id.diff
Thu, May 9, 7:43 AM
F13181035: D8314.diff
Thu, May 9, 7:37 AM
F13176584: D8314.id19766.diff
Wed, May 8, 3:19 PM
Unknown Object (File)
Mon, May 6, 1:02 AM
Subscribers

Details

Summary

Ref T1191. Test that MySQL's rules match those of phutil_is_utf8_with_only_bmp_characters():

  • Build a string with every character that we consider to be a BMP character.
  • Write it into MySQL.
  • Read it back out.
  • Make sure MySQL didn't truncate it.
Test Plan

Ran unit test. This test runs pretty quickly (50ms), the string with every character isn't all that enormous.

Diff Detail

Repository
rP Phabricator
Branch
pmbp1
Lint
Lint Passed
Unit
Tests Passed

Event Timeline

epriestley updated this revision to Unknown Object (????).Feb 23 2014, 8:40 PM
  • Adjust the character ranges to match phutil_is_utf8_with_only_bmp_characters().