Patch #32054
Add test for 4 byte characters (emoji) support
| Status: | Closed | Start date: | ||
|---|---|---|---|---|
| Priority: | Normal | Due date: | ||
| Assignee: |  Go MAEDA | % Done: | 0% | |
| Category: | Code cleanup/refactoring | |||
| Target version: | 4.2.0 |
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.
test_for_emoji_support.patch 
(832 Bytes)
test_for_emoji_support-v2.patch
(1.23 KB)
Related issues
Associated revisions
Add test for 4 byte characters (emoji) support (#32054).
Patch by Marius BALTEANU.
History
#1
Updated by Marius BALTEANU about 2 years ago
- Related to Feature #31921: Changes to properly support 4 byte characters (emoji) when database is MySQL added
#2
Updated by Marius BALTEANU almost 2 years ago
- Target version changed from 4.1.0 to 4.2.0
#3
Updated by Toshi MARUYAMA over 1 year 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
#4
Updated by Go MAEDA 7 months ago
- File 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.