Project

General

Profile

Actions

Patch #32054

closed

Add test for 4 byte characters (emoji) support

Added by Marius BĂLTEANU over 4 years ago. Updated about 3 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
Code cleanup/refactoring
Target version:
Start date:
Due date:
% Done:

0%

Estimated time:

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

Related to Redmine - Feature #31921: Changes to properly support 4 byte characters (emoji) when database is MySQLClosedGo MAEDA

Actions
Actions #1

Updated by Marius BĂLTEANU over 4 years ago

  • Related to Feature #31921: Changes to properly support 4 byte characters (emoji) when database is MySQL added
Actions #2

Updated by Marius BĂLTEANU over 4 years ago

  • Target version changed from 4.1.0 to 4.2.0
Actions #3

Updated by Toshi MARUYAMA almost 4 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

Actions #4

Updated by Go MAEDA about 3 years ago

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.

Actions #5

Updated by Go MAEDA about 3 years ago

  • Status changed from New to Closed
  • Assignee set to Go MAEDA

Committed the patch. Thank you.

Actions

Also available in: Atom PDF