svn commit: r790506 - /lenya/branches/BRANCH_2_0_X/src/impl/test/org/apache/lenya/cms/usecase/AbstractUsecaseTest.java

[email protected]
Newsgroups gmane.comp.cms.lenya.cvs
Message-ID <[email protected]>
Author: andreas
Date: Thu Jul  2 08:54:37 2009
New Revision: 790506

URL: http://svn.apache.org/viewvc?rev=790506&view=rev
Log:
Improved error handling in AbstractUsecaseTest.

Modified:
    lenya/branches/BRANCH_2_0_X/src/impl/test/org/apache/lenya/cms/usecase/AbstractUsecaseTest.java

Modified: lenya/branches/BRANCH_2_0_X/src/impl/test/org/apache/lenya/cms/usecase/AbstractUsecaseTest.java
URL: http://svn.apache.org/viewvc/lenya/branches/BRANCH_2_0_X/src/impl/test/org/apache/lenya/cms/usecase/AbstractUsecaseTest.java?rev=790506&r1=790505&r2=790506&view=diff
==============================================================================
--- lenya/branches/BRANCH_2_0_X/src/impl/test/org/apache/lenya/cms/usecase/AbstractUsecaseTest.java (original)
+++ lenya/branches/BRANCH_2_0_X/src/impl/test/org/apache/lenya/cms/usecase/AbstractUsecaseTest.java Thu Jul  2 08:54:37 2009
@@ -45,8 +45,13 @@
             invoker = (UsecaseInvoker) getManager().lookup(TestUsecaseInvoker.ROLE);
             invoker.setTestSession(session);
             invoker.invoke(getRequest().getPathInfo(), getUsecaseName(), getParameters());
-            
-            this.targetUrl = invoker.getTargetUrl();
+
+            int result = invoker.getResult();
+            if (result != UsecaseInvoker.PRECONDITIONS_FAILED
+                    && result != UsecaseInvoker.EXECUTION_CONDITIONS_FAILED
+                    && result != UsecaseInvoker.AUTHORIZATION_FAILED) {
+                this.targetUrl = invoker.getTargetUrl();
+            }
 
             List errorMessages = invoker.getErrorMessages();
             for (Iterator i = errorMessages.iterator(); i.hasNext();) {
@@ -72,9 +77,9 @@
         checkPostconditions();
 
     }
-    
+
     private String targetUrl;
-    
+
     protected String getTargetUrl() {
         if (this.targetUrl == null) {
             throw new IllegalStateException("The usecase has not yet been executed.");
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.