Skip to main content
aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorMartin Fluegge2010-02-21 11:27:25 +0000
committerMartin Fluegge2010-02-21 11:27:25 +0000
commit0882cad45629dece0f1747286563bec927a031f5 (patch)
tree3a8da40190700ec8001d13db9314cec4cdd3f3fd
parenta5543c19fba77088424664565046ee4b4c1bf9f5 (diff)
downloadcdo-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
-rw-r--r--plugins/org.eclipse.emf.cdo.tests/src/org/eclipse/emf/cdo/tests/AllTestsAllConfigs.java2
-rw-r--r--plugins/org.eclipse.emf.cdo.tests/src/org/eclipse/emf/cdo/tests/bugzilla/Bugzilla_302233_Test.java71
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();
+ }
+ }
+}

Back to the top