Do not rely on file encoding for non-latin1 character in utf8 test

basho / cuttlefish

never lose your childlike sense of wonder baby cuttlefish, promise me?

Apache License 2.0

205 stars 124 forks source link

Do not rely on file encoding for non-latin1 character in utf8 test #199

Closed macintux closed 9 years ago

macintux commented 9 years ago

Recent work to incorporate a Unicode-friendly version of neotoma has revealed a minor test issue that results in inconsistencies when testing on R16 vs 17/18. Fix this so we can update our dependencies.

In short, under R16 with the latest tagged neotoma this test fails to generate a unicode error as expected, because the non-latin1 character is split into two integers that fall within the latin1 space.

jadeallenx commented 9 years ago

LGTM

jadeallenx commented 9 years ago

:+1: 1d6982b

macintux commented 9 years ago

@borshop merge