From 7f29c2fd88f271ac59f1c10b90942fec57b35ae2 Mon Sep 17 00:00:00 2001 From: Toshi MARUYAMA Date: Fri, 16 Jan 2015 04:57:10 +0000 Subject: [PATCH] IssuesControllerTest: make encoding tests independent of ordering (#18856) Contributed by Antonio Terceiro. git-svn-id: http://svn.redmine.org/redmine/trunk@13885 e93f8b46-1217-0410-a6f0-8f06a7374b81 --- test/functional/issues_controller_test.rb | 16 ++++++++++------ 1 file changed, 10 insertions(+), 6 deletions(-) diff --git a/test/functional/issues_controller_test.rb b/test/functional/issues_controller_test.rb index 49fff0adb..04ebec0df 100644 --- a/test/functional/issues_controller_test.rb +++ b/test/functional/issues_controller_test.rb @@ -496,9 +496,11 @@ class IssuesControllerTest < ActionController::TestCase :format => 'csv' assert_equal 'text/csv; header=present', @response.content_type lines = @response.body.chomp.split("\n") + header = lines[0] + issue_line = lines.find {|l| l =~ /^#{issue.id},/} s1 = "\xaa\xac\xbaA".force_encoding('Big5') - assert_include s1, lines[0] - assert_include str_big5, lines[1] + assert_include s1, header + assert_include str_big5, issue_line end end @@ -515,9 +517,11 @@ class IssuesControllerTest < ActionController::TestCase :set_filter => 1 assert_equal 'text/csv; header=present', @response.content_type lines = @response.body.chomp.split("\n") + header = lines[0] + issue_line = lines.find {|l| l =~ /^#{issue.id},/} s1 = "\xaa\xac\xbaA".force_encoding('Big5') # status - assert lines[0].include?(s1) - s2 = lines[1].split(",")[2] + assert header.include?(s1) + s2 = issue_line.split(",")[2] s3 = "\xa5H?".force_encoding('Big5') # subject assert_equal s3, s2 end @@ -536,7 +540,7 @@ class IssuesControllerTest < ActionController::TestCase :set_filter => 1 assert_equal 'text/csv; header=present', @response.content_type lines = @response.body.chomp.split("\n") - assert_equal "#{issue.id},1234.50,#{str1}", lines[1] + assert_include "#{issue.id},1234.50,#{str1}", lines end end @@ -553,7 +557,7 @@ class IssuesControllerTest < ActionController::TestCase :set_filter => 1 assert_equal 'text/csv; header=present', @response.content_type lines = @response.body.chomp.split("\n") - assert_equal "#{issue.id};1234,50;#{str1}", lines[1] + assert_include "#{issue.id};1234,50;#{str1}", lines end end -- 2.39.5