Skip to content
Browse files

New test for modifying the ExceptionStack

  • Loading branch information...
1 parent d21bfe4 commit 953a5557832709e17b52d6fdb184cc77782c8cd4 @LightGuard LightGuard committed Jan 4, 2011
View
7 api/src/main/java/org/jboss/seam/exception/control/ExceptionStack.java
@@ -137,12 +137,7 @@ public Throwable getCurrent()
return current;
}
- public Collection<Throwable> getElements()
- {
- return Collections.unmodifiableCollection(elements);
- }
-
- public void setElements(Collection<Throwable> elements)
+ public void setCauseElements(Collection<Throwable> elements)
{
this.elements = Collections.unmodifiableCollection(elements);
this.init();
View
4 impl/src/test/java/org/jboss/seam/exception/control/test/stack/ExceptionStackTest.java
@@ -46,7 +46,7 @@ public void testSQLExceptionUnwrap()
ExceptionStack es = new ExceptionStack(e);
- assertThat(es.getElements().size(), is(4));
- assertThat(es.getElements(), hasItems(e, transactionRollbackException, recoverableException, syntaxErrorException));
+ assertThat(es.getCauseElements().size(), is(4));
+ assertThat(es.getCauseElements(), hasItems(e, transactionRollbackException, recoverableException, syntaxErrorException));
}
}
View
88 impl/src/test/java/org/jboss/seam/exception/control/test/stack/ModifyExceptionStackTest.java
@@ -0,0 +1,88 @@
+/*
+ * JBoss, Home of Professional Open Source
+ * Copyright 2011, Red Hat, Inc., and individual contributors
+ * by the @authors tag. See the copyright.txt in the distribution for a
+ * full listing of individual contributors.
+ *
+ * Licensed under the Apache License, Version 2.0 (the "License");
+ * you may not use this file except in compliance with the License.
+ * You may obtain a copy of the License at
+ * http://www.apache.org/licenses/LICENSE-2.0
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+
+/**
+ *
+ * @author <a href="http://community.jboss.org/people/LightGuard">Jason Porter</a>
+ */
+package org.jboss.seam.exception.control.test.stack;
+
+import java.util.ArrayList;
+
+import javax.enterprise.event.Event;
+import javax.enterprise.event.Observes;
+
+import org.jboss.arquillian.api.Deployment;
+import org.jboss.arquillian.junit.Arquillian;
+import org.jboss.seam.exception.control.CaughtException;
+import org.jboss.seam.exception.control.ExceptionStack;
+import org.jboss.seam.exception.control.ExceptionToCatch;
+import org.jboss.seam.exception.control.Handles;
+import org.jboss.seam.exception.control.HandlesExceptions;
+import org.jboss.seam.exception.control.extension.CatchExtension;
+import org.jboss.shrinkwrap.api.Archive;
+import org.jboss.shrinkwrap.api.ArchivePaths;
+import org.jboss.shrinkwrap.api.ShrinkWrap;
+import org.jboss.shrinkwrap.api.asset.ByteArrayAsset;
+import org.jboss.shrinkwrap.api.spec.JavaArchive;
+import org.junit.Test;
+import org.junit.runner.RunWith;
+
+import static org.hamcrest.CoreMatchers.not;
+import static org.junit.Assert.assertThat;
+import static org.junit.matchers.JUnitMatchers.hasItem;
+
+@RunWith(Arquillian.class)
+@HandlesExceptions
+public class ModifyExceptionStackTest
+{
+ private Exception startException;
+
+ @Deployment
+ public static Archive<?> createTestArchive()
+ {
+ return ShrinkWrap.create(JavaArchive.class)
+ .addPackage(CaughtException.class.getPackage())
+ .addClasses(ModifyExceptionStackTest.class, CatchExtension.class)
+ .addManifestResource("META-INF/services/javax.enterprise.inject.spi.Extension")
+ .addManifestResource(new ByteArrayAsset(new byte[0]), ArchivePaths.create("beans.xml"));
+ }
+
+ public static class ApplicationException extends Exception
+ {
+
+ }
+
+ @Test
+ public void assertModifyingStack(Event<ExceptionToCatch> event)
+ {
+ this.startException = new Exception(new NullPointerException());
+ event.fire(new ExceptionToCatch(this.startException));
+ }
+
+ public void changeStackObserver(@Observes ExceptionStack stack)
+ {
+ ArrayList<Throwable> causes = new ArrayList<Throwable>(stack.getCauseElements());
+ causes.set(0, new ApplicationException());
+ stack.setCauseElements(causes);
+ }
+
+ public void assertionHandler(@Handles CaughtException<Throwable> e)
+ {
+ assertThat(e.getExceptionStack().getCauseElements(), not(hasItem((Throwable) this.startException)));
+ }
+}

0 comments on commit 953a555

Please sign in to comment.
Something went wrong with that request. Please try again.