diff options
author | Martin Fluegge | 2010-02-21 11:27:25 +0000 |
---|---|---|
committer | Martin Fluegge | 2010-02-21 11:27:25 +0000 |
commit | 0882cad45629dece0f1747286563bec927a031f5 (patch) | |
tree | 3a8da40190700ec8001d13db9314cec4cdd3f3fd | |
parent | a5543c19fba77088424664565046ee4b4c1bf9f5 (diff) | |
download | cdo-0882cad45629dece0f1747286563bec927a031f5.tar.gz cdo-0882cad45629dece0f1747286563bec927a031f5.tar.xz cdo-0882cad45629dece0f1747286563bec927a031f5.zip |
[302233] Undo adding first object to a list throws ArrayIndexOutOfBoundsException
https://bugs.eclipse.org/bugs/show_bug.cgi?id=302233
2 files changed, 73 insertions, 0 deletions
diff --git a/plugins/org.eclipse.emf.cdo.tests/src/org/eclipse/emf/cdo/tests/AllTestsAllConfigs.java b/plugins/org.eclipse.emf.cdo.tests/src/org/eclipse/emf/cdo/tests/AllTestsAllConfigs.java index b0b74e4d22..e24f9191d1 100644 --- a/plugins/org.eclipse.emf.cdo.tests/src/org/eclipse/emf/cdo/tests/AllTestsAllConfigs.java +++ b/plugins/org.eclipse.emf.cdo.tests/src/org/eclipse/emf/cdo/tests/AllTestsAllConfigs.java @@ -64,6 +64,7 @@ import org.eclipse.emf.cdo.tests.bugzilla.Bugzilla_294850_Test; import org.eclipse.emf.cdo.tests.bugzilla.Bugzilla_294859_Test; import org.eclipse.emf.cdo.tests.bugzilla.Bugzilla_298561_Test; import org.eclipse.emf.cdo.tests.bugzilla.Bugzilla_299190_Test; +import org.eclipse.emf.cdo.tests.bugzilla.Bugzilla_302233_Test; import org.eclipse.emf.cdo.tests.config.impl.ConfigTest; import org.eclipse.emf.cdo.tests.config.impl.ConfigTestSuite; @@ -186,6 +187,7 @@ public abstract class AllTestsAllConfigs extends ConfigTestSuite testClasses.add(Bugzilla_292372_Test.class); testClasses.add(Bugzilla_298561_Test.class); testClasses.add(Bugzilla_299190_Test.class); + testClasses.add(Bugzilla_302233_Test.class); // TODO testClasses.add(NonCDOResourceTest.class); // TODO testClasses.add(GeneratedEcoreTest.class); diff --git a/plugins/org.eclipse.emf.cdo.tests/src/org/eclipse/emf/cdo/tests/bugzilla/Bugzilla_302233_Test.java b/plugins/org.eclipse.emf.cdo.tests/src/org/eclipse/emf/cdo/tests/bugzilla/Bugzilla_302233_Test.java new file mode 100644 index 0000000000..0dac617d55 --- /dev/null +++ b/plugins/org.eclipse.emf.cdo.tests/src/org/eclipse/emf/cdo/tests/bugzilla/Bugzilla_302233_Test.java @@ -0,0 +1,71 @@ +/** + * Copyright (c) 2004 - 2010 Eike Stepper (Berlin, Germany) and others. + * All rights reserved. This program and the accompanying materials + * are made available under the terms of the Eclipse Public License v1.0 + * which accompanies this distribution, and is available at + * http://www.eclipse.org/legal/epl-v10.html + * + * Contributors: + * Martin Fluegge - initial API and implementation + */ +package org.eclipse.emf.cdo.tests.bugzilla; + +import org.eclipse.emf.cdo.eresource.CDOResource; +import org.eclipse.emf.cdo.session.CDOSession; +import org.eclipse.emf.cdo.tests.AbstractCDOTest; +import org.eclipse.emf.cdo.tests.model1.Order; +import org.eclipse.emf.cdo.tests.model1.OrderDetail; +import org.eclipse.emf.cdo.transaction.CDOTransaction; + +import org.eclipse.emf.ecore.EStructuralFeature; + +/** + * unset of CDO does not work correctly + * <p> + * See bug 302233 + * + * @author Martin Fluegge + */ +public class Bugzilla_302233_Test extends AbstractCDOTest +{ + public void testBugzilla_302233() throws Exception + { + { + Order order = getModel1Factory().createOrder(); + EStructuralFeature feature = getModel1Package().getOrder_OrderDetails(); + + assertEquals(false, order.eIsSet(feature)); + + order.eUnset(feature); + + for (int i = 0; i < 10; i++) + { + OrderDetail orderDetail = getModel1Factory().createOrderDetail(); + order.getOrderDetails().add(orderDetail); + } + + order.eUnset(feature); + + CDOSession session = openSession(); + session.getPackageRegistry().putEPackage(getModel1Package()); + + CDOTransaction transaction = session.openTransaction(); + CDOResource resource = transaction.createResource("/test1"); + resource.getContents().add(order); + + for (int i = 0; i < 10; i++) + { + OrderDetail orderDetail = getModel1Factory().createOrderDetail(); + order.getOrderDetails().add(orderDetail); + } + + transaction.commit(); + + order.eUnset(feature); + + assertEquals(false, order.eIsSet(getModel1Package().getOrder_OrderDetails())); + + session.close(); + } + } +} |