Bug 195526

Summary: org.eclipse.jdt.core.tests.compiler.regression.GenericTypeTest.test0744 depends on non-specified package-private class HashMap.Entry implementation
Product: [Eclipse Project] JDT Reporter: Nina Rinskaya <nina.rinskaya>
Component: CoreAssignee: Olivier Thomann <Olivier_Thomann>
Status: VERIFIED FIXED QA Contact:
Severity: normal    
Priority: P3 CC: david_audel, philippe_mulet
Version: 3.3   
Target Milestone: 3.3.1   
Hardware: PC   
OS: Windows XP   
Whiteboard:
Attachments:
Description Flags
Proposed fix none

Description Nina Rinskaya CLA 2007-07-05 09:44:48 EDT 
Eclipse Automated test org.eclipse.jdt.core.tests.compiler.regression.GenericTypeTest.test0744 fails on Apache Harmony with "The constructor HashMap.Entry<K2,V2>(int, null, null, null) is undefined" unexpected output because Apache Harmony implementation of java.util.HashMap.Entry (which has package-private access and is not described in J2SE specification) constructor differs from Sun's implementation.
Comment 1 Olivier Thomann CLA 2007-07-05 10:46:26 EDT 
Trivial to fix.
Setting target as 3.3.1.
Comment 2 Olivier Thomann CLA 2007-07-05 10:47:51 EDT 
Created attachment 73124 [details]
Proposed fix

This reports the same error, but doesn't rely on the HashMap.Entry implementation.
Comment 3 Olivier Thomann CLA 2007-07-05 11:09:23 EDT 
Released for 3.4M1.
Comment 4 Olivier Thomann CLA 2007-07-05 11:12:34 EDT 
Released for 3.3.1.
Comment 5 Nina Rinskaya CLA 2007-07-06 02:34:17 EDT 
The issue is not reproducible on Harmony with these changes, thanks.
Comment 6 Olivier Thomann CLA 2007-07-06 09:38:40 EDT 
You might want to use the version from branch R3_3_maintenance.
Comment 7 Nina Rinskaya CLA 2007-07-11 00:01:43 EDT 
I actually don't have the Eclipse Automated sources from CVS, just the sources packed into binary bundle (eclipse-Automated-Tests-3.2.zip/eclipse-Automated-Tests-3.3.zip), so I just copy-pasted the lines that were changed from http://dev.eclipse.org/viewcvs/index.cgi/org.eclipse.jdt.core.tests.compiler/src/org/eclipse/jdt/core/tests/compiler/regression/GenericTypeTest.java?view=markup, hope that's ok.
Comment 8 Frederic Fusier CLA 2007-08-07 06:41:35 EDT 
Code verified for 3.4M1 using build I20070806-1800.
Comment 9 Eric Jodet CLA 2007-09-03 10:29:09 EDT 
Code verified for 3.3.1 using build M20070831-2000