Bug 378783 - [search] [test] testBug376673* are failing on test machines
Summary: [search] [test] testBug376673* are failing on test machines
Status: VERIFIED FIXED
Alias: None
Product: JDT
Classification: Eclipse Project
Component: Core (show other bugs)
Version: 3.8   Edit
Hardware: PC Windows 7
: P3 normal (vote)
Target Milestone: 3.8 RC1   Edit
Assignee: Satyam Kandula CLA
QA Contact:
URL:
Whiteboard:
Keywords: test
Depends on:
Blocks:
 
Reported: 2012-05-08 03:03 EDT by Satyam Kandula CLA
Modified: 2012-05-29 05:08 EDT (History)
5 users (show)

See Also:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Satyam Kandula CLA 2012-05-08 03:03:07 EDT 
All testBug376673* are failing. 
Look at http://download.eclipse.org/eclipse/downloads/drops4/S-4.2M7-201205031800/testresults/html/org.eclipse.jdt.core.tests.model_linux.gtk.x86_6.0.html.

The message is... 
############

Unexpected search results. ----------- Expected ------------ pkg/𠮟1.java pkg.𠮟1 [𠮟1] EXACT_MATCH ------------ but was ------------ --------- Difference is ---------- expected:<[pkg/𠮟1.java pkg.𠮟1 [𠮟1] EXACT_MATCH]> but was:<[]>
junit.framework.ComparisonFailure: Unexpected search results.
----------- Expected ------------
pkg/𠮟1.java pkg.𠮟1 [𠮟1] EXACT_MATCH
------------ but was ------------
###########

This is probably happening as these tests are using UTF-8 characters and somehow build is not really picking up the encoding.
Comment 1 Satyam Kandula CLA 2012-05-08 04:53:13 EDT 
The file has some unicode characters and I have set the file as UTF-8 encoding. This doesn't seem to be good enough.
I am trying out changing the BOM of the file also with the commit
0d335e6f3f14db13eb60c4dd03de7b6cbccf779d
Comment 2 Satyam Kandula CLA 2012-05-10 02:36:15 EDT 
(In reply to comment #1)
> The file has some unicode characters and I have set the file as UTF-8 encoding.
> This doesn't seem to be good enough.
> I am trying out changing the BOM of the file also with the commit
> 0d335e6f3f14db13eb60c4dd03de7b6cbccf779d
Changing the BOM didn't work. Infact there was an error.
Reverted the commit using commit 120defa76e249a7b50b5aa52206e16b815780186
Comment 3 Satyam Kandula CLA 2012-05-10 02:37:26 EDT 
Changed the string to use the unicode characters \uD842\uDF9F instead of the string.
Released the fix through the commit 635cdb5a7a6689c525de4e28f32c0630009bc2be.
Comment 4 Jay Arthanareeswaran CLA 2012-05-14 05:06:07 EDT 
Verified that the concerned tests are passing with build I20120511-1900.
Comment 5 Jay Arthanareeswaran CLA 2012-05-28 20:32:57 EDT 
The same tests are failing on Mac as can be seen here:
http://download.eclipse.org/eclipse/downloads/drops4/I20120527-1900/testresults/html/org.eclipse.jdt.core.tests.model_macosx.cocoa.x86_5.0.html
Comment 6 Srikanth Sankaran CLA 2012-05-29 00:26:45 EDT 
(In reply to comment #5)
> The same tests are failing on Mac as can be seen here:
> http://download.eclipse.org/eclipse/downloads/drops4/I20120527-1900/testresults/html/org.eclipse.jdt.core.tests.model_macosx.cocoa.x86_5.0.html

What is the impact of these failures ? Is this a test case issue or does it
indicate problems in the product ? If the latter case, what feature/functionality
gets affected by it and how serious is it ? Thanks.
Comment 7 Satyam Kandula CLA 2012-05-29 00:48:16 EDT 
(In reply to comment #6)
> (In reply to comment #5)
> > The same tests are failing on Mac as can be seen here:
> > http://download.eclipse.org/eclipse/downloads/drops4/I20120527-1900/testresults/html/org.eclipse.jdt.core.tests.model_macosx.cocoa.x86_5.0.html
> 
> What is the impact of these failures ? Is this a test case issue or does it
> indicate problems in the product ? If the latter case, what
> feature/functionality
> gets affected by it and how serious is it ? Thanks.
It is a test case issue and hence want to disable it for Mac only.
Comment 8 Srikanth Sankaran CLA 2012-05-29 00:50:12 EDT 
(In reply to comment #7)

> It is a test case issue and hence want to disable it for Mac only.

Thanks, sounds good.
Comment 9 Satyam Kandula CLA 2012-05-29 05:08:35 EDT 
(In reply to comment #8)
> (In reply to comment #7)
> 
> > It is a test case issue and hence want to disable it for Mac only.
> 
> Thanks, sounds good.
Disabled the test only on Mac using commit 3b08272ee847aa9d892ea0105ce6d34424762f9c