Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 336775 - Loading a removal ChangeDescription throws an IllegalValueException
Summary: Loading a removal ChangeDescription throws an IllegalValueException
Status: RESOLVED WORKSFORME
Alias: None
Product: EMF
Classification: Modeling
Component: Core (show other bugs)
Version: 2.6.0   Edit
Hardware: PC Windows 7
: P3 critical (vote)
Target Milestone: ---   Edit
Assignee: Ed Merks CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-02-09 19:00 EST by Cedric Vidal CLA
Modified: 2011-02-10 04:31 EST (History)
0 users

See Also:

Attachments
Testcase (5.31 KB, text/x-java)
2011-02-09 19:02 EST, Cedric Vidal CLA 		no flags Details
Testcase works under EMF 2.7 (5.37 KB, text/x-java)
2011-02-10 04:31 EST, Cedric Vidal CLA 		no flags Details
Show Obsolete (1) 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 Cedric Vidal CLA 2011-02-09 19:00:56 EST 
Build Identifier: Eclipse I20100608-0911 with EMF 2.6.0.v20100614-1136

Loading a ChangeDescription describing the removal of an EObject in a bidirectional one to many reference throws an IllegalValueException.


Reproducible: Always
Comment 1 Cedric Vidal CLA 2011-02-09 19:02:56 EST 
Created attachment 188638 [details]
Testcase
Comment 2 Ed Merks CLA 2011-02-10 03:58:43 EST 
It doesn't do that for me with the 2.7 code base.   It gets to 		assertNotNull(eClass.getEPackage()); which fails because eClass is an unloaded proxy...
Comment 3 Cedric Vidal CLA 2011-02-10 04:27:44 EST 
Hi Ed,

Gosh you were fast, thanx. You got me. Indeed, I just tested, it works under EMF 2.7. I should have done that test in the first place. Sorry for the noise ;)

Kind regards,

Cédric

PS: it wasn't the purpose of the test but indeed, the eClass being unloaded is normal ;)
Comment 4 Cedric Vidal CLA 2011-02-10 04:31:05 EST 
Created attachment 188662 [details]
Testcase works under EMF 2.7

For the record, I removed the erroneous line in the testcase.