evergreen.lowercase needs test

[Liam Whalen]

Test Writing Day - 2015/08/14

evergreen.lowercase is a potential target for a test.

[Dan Scott (denials)]

Suggestion: include some examples of Unicode (e.g. É -> é)?

[Yamil (ysuarez)]

Josh,

Found this file that you can use to get ideas of how to thoroughly test with unicode examples....

Open-ILS/src/sql/Pg/t/naco_normalize.pg

http://git.evergreen-ils.org/?p=Evergreen.git;a=blob;f=Open-ILS/src/sql/Pg/t/naco_normalize.pg;h=0e59107e99886d79e7349b9acb700a8efb8899c9;hb=HEAD

God luck,
Yamil

[Josh Stompro (u-launchpad-stompro-org)]

Branch at
working user/stompro/lp1483500_pgtap_evergreen_lowercase

http://git.evergreen-ils.org/?p=working/Evergreen.git;a=shortlog;h=refs/heads/user/stompro/lp1483500_pgtap_evergreen_lowercase

I think I might have found a bug in evergreen.lowercase when handling Turkish characters. There are two i's in the Turkish character set, İ-> i, I -> ı, Capitol I with a dot, and without a dot, and lowercase versions of each. But select evergreen.lowercase('İI'); returns " i̇i" when it should return "iı". But I would like someone that has more UNICODE experience to confirm it.

This site talks about the double I problems.
http://www.i18nguy.com/unicode/turkish-i18n.html#problem

I also can now think of an instance where I used the built in Postgresq lower() function that might break with unicode characters. I'll go back and fix that.

Josh

[Yamil (ysuarez)]

The pgTAP test passed

pg_prove -U evergreen lowercase.pg
lowercase.pg .. ok
All tests successful.
Files=1, Tests=7, 0 wallclock secs ( 0.02 usr 0.01 sys + 0.02 cusr 0.04 csys = 0.09 CPU)
Result: PASS

here is my sign off branch:
http://git.evergreen-ils.org/?p=working/Evergreen.git;a=shortlog;h=refs/heads/user/ysuarez/lp1483500_pgtap_evergreen_lowercase_signoff

[Kathy Lussier (klussier)]

Looks good to me. Merged to master.

Thank you Josh and Yamil!