Bug 152841 - [model] IJavaProject.findType(name, monitor) doesn't find secondary type
Summary: [model] IJavaProject.findType(name, monitor) doesn't find secondary type
Status: VERIFIED FIXED
Alias: None
Product: JDT
Classification: Eclipse Project
Component: Core (show other bugs)
Version: 3.2   Edit
Hardware: PC Windows XP
: P3 normal (vote)
Target Milestone: 3.4 M2   Edit
Assignee: Frederic Fusier CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2006-08-04 09:22 EDT by Martin Aeschlimann CLA
Modified: 2007-09-18 04:48 EDT (History)
1 user (show)

See Also:

Attachments
Test case, needs to be in org.eclipse.jdt.ui.tests.refactoring (3.86 KB, text/x-java)
2006-08-04 09:25 EDT, Martin Aeschlimann CLA 		no flags Details
Simplified test case (3.33 KB, text/x-java)
2006-08-04 10:08 EDT, Martin Aeschlimann CLA 		no flags Details
Proposed patch (4.64 KB, patch)
2007-08-28 04:53 EDT, Frederic Fusier CLA 		no flags Details | Diff
Show Obsolete (2) View All 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 Martin Aeschlimann CLA 2006-08-04 09:22:19 EDT 
20060804

When I run the attached test case, the secondary type 'C' is not found. Setting a breakpoint before and step in will find it.

It seems that the index isn't complete and doesn't wait for it.
Comment 1 Martin Aeschlimann CLA 2006-08-04 09:25:28 EDT 
Created attachment 47395 [details]
Test case, needs to be in org.eclipse.jdt.ui.tests.refactoring

Copy this test case in the org.eclipse.jdt.ui.tests.refactoring plugin.
org.eclipse.jdt.ui.tests.refactoring requires org.eclipse.jdt.ui.tests
Comment 2 Martin Aeschlimann CLA 2006-08-04 10:08:49 EDT 
Created attachment 47399 [details]
Simplified test case

Simplified test case that has no references to refactoring, but just uses an AST (no need to be created in org.eclipse.jdt.ui.tests.refactoring).
Comment 3 Frederic Fusier CLA 2007-08-28 04:53:41 EDT 
Created attachment 77101 [details]
Proposed patch

Thanks for the test case, I rewrite it a little bit to put it in our model test suite...
Comment 4 Frederic Fusier CLA 2007-08-28 06:56:07 EDT 
Released for 3.4M2 in HEAD stream.
Comment 5 Maxime Daniel CLA 2007-09-18 04:48:27 EDT 
Verified for 3.4 M2 using build I20070917-0010.