Patch #32054
closed
Add test for 4 byte characters (emoji) support
Description
In #31921 was added proper support for 4 byte characters (emoji) for new installations with MySQL as database.
I proposed to have a test for this case, but we're worried that the test will broke the CI server.
The scope of this ticket is to fix the CI server if the MySQL is not already configured properly by recreating the database with utf8mb4 encoding.
Files
Related issues
Updated by Marius BĂLTEANU almost 6 years ago
- Related to Feature #31921: Changes to properly support 4 byte characters (emoji) when database is MySQL added
Updated by Marius BĂLTEANU almost 6 years ago
- Target version changed from 4.1.0 to 4.2.0
Updated by Toshi MARUYAMA about 5 years ago
I have already added Emoji test (r19699).
git_urf8 test repository is not extracted due to this.
https://www.redmine.org/builds/logs/build_trunk_mysql_ruby-2.6_1044.html
+ PATH=/var/lib/jenkins/ruby/ruby-2.6/bin:/usr/local/bin:/usr/bin:/bin:/usr/local/games: /usr/games SCMS=subversion,git,mercurial,filesystem,bazaar,cvs bundle exec rake ci:about ci:setup db:environment:set RAILS_ENV=test
Updated by Go MAEDA over 4 years ago
- File test_for_emoji_support-v2.patch test_for_emoji_support-v2.patch added
- Category set to Code cleanup/refactoring
- Assignee deleted (
Jean-Philippe Lang)
Marius BALTEANU wrote:
I proposed to have a test for this case, but we're worried that the test will broke the CI server.
I think we can avoid errors occurring on the CI server by adding the following check at the beginning of the test.
skip if Redmine::Database.mysql? && !is_mysql_utf8mb4
I will commit test_for_emoji_support-v2.patch soon.
Updated by Go MAEDA over 4 years ago
- Status changed from New to Closed
- Assignee set to Go MAEDA
Committed the patch. Thank you.