Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

Bug 324037

Summary:	Improve IUBundleContainerTests and check for cases where profile is stale
Product:	[Eclipse Project] PDE	Reporter:	Curtis Windatt <curtis.windatt.public>
Component:	UI	Assignee:	Curtis Windatt <curtis.windatt.public>
Status:	RESOLVED FIXED	QA Contact:
Severity:	normal
Priority:	P3	CC:	ankur_sharma, darin.eclipse
Version:	3.7
Target Milestone:	3.7 M2
Hardware:	PC
OS:	Linux
Whiteboard:
Bug Depends on:	276326
Bug Blocks:

Description Curtis Windatt

2010-08-30 17:34:20 EDT

Follow up to bug 276326

1) We should add a couple new test cases to check that the profile resolution gets used.

2) The tests should delete the profile in a try/finally otherwise one failure can lead to others (the later tests look up information in the profile)

3) While the tests pass and my manual testing passed, there still may be ways to get a stale profile.  One possibility would be when switching from slicing to planning, while the planner would actually fail, we never get to that point because the profile contains the previously sliced IUs.

Comment 1 Curtis Windatt

2010-08-31 17:21:45 EDT

1) Still working on this, I have a test that will modify the profile during the test so the if the resolveWithProfile is running correctly it will get different results.

2) Fixed in HEAD.

3) Changing slicing vs planning forces the profile to be recreated because the properties are immutable.  I think we are ok here.

Comment 2 Curtis Windatt

2010-09-01 11:56:32 EDT

Fixed in HEAD, see IUBundleContainerTests.  I also had to add some new dependencies to the test bundle to modify the profile during the test.