You are viewing a plain text version of this content. The canonical link for it is here.
Posted to cactus-dev@jakarta.apache.org by vm...@apache.org on 2002/05/12 21:42:28 UTC

cvs commit: jakarta-cactus/framework/src/java/share/org/apache/cactus/client ClientConfigurationChecker.java

vmassol     02/05/12 12:42:28

  Modified:    framework/src/java/share/org/apache/cactus
                        AbstractTestCase.java AbstractWebTestCase.java
  Removed:     framework/src/java/share/org/apache/cactus/client
                        ClientConfigurationChecker.java
  Log:
  removed client checks as it is not correctly implemented (will need to implement that by using AspectJ and make sure it will work with ServletTestRunner stuff)
  
  Revision  Changes    Path
  1.7       +1 -9      jakarta-cactus/framework/src/java/share/org/apache/cactus/AbstractTestCase.java
  
  Index: AbstractTestCase.java
  ===================================================================
  RCS file: /home/cvs/jakarta-cactus/framework/src/java/share/org/apache/cactus/AbstractTestCase.java,v
  retrieving revision 1.6
  retrieving revision 1.7
  diff -u -r1.6 -r1.7
  --- AbstractTestCase.java	5 May 2002 20:08:50 -0000	1.6
  +++ AbstractTestCase.java	12 May 2002 19:42:28 -0000	1.7
  @@ -75,7 +75,7 @@
    *
    * @author <a href="mailto:vmassol@apache.org">Vincent Massol</a>
    *
  - * @version $Id: AbstractTestCase.java,v 1.6 2002/05/05 20:08:50 vmassol Exp $
  + * @version $Id: AbstractTestCase.java,v 1.7 2002/05/12 19:42:28 vmassol Exp $
    */
   public abstract class AbstractTestCase extends TestCase
   {
  @@ -171,11 +171,6 @@
       }
   
       /**
  -     * Check client side configuration.
  -     */
  -    protected abstract void checkConfiguration();
  -
  -    /**
        * Runs the bare test sequence. This method is overridden from the
        * JUnit <code>TestCase</code> class in order to prevent the latter
        * to call the <code>setUp()</code> and <code>tearDown()</code> methods
  @@ -187,9 +182,6 @@
        */
       public void runBare() throws Throwable
       {
  -        // Run some configuration checks
  -        checkConfiguration();
  -
           // We make sure we reinitialize The logger with the name of the
           // current class (that's why the logged instance is not static).
           this.logger =
  
  
  
  1.3       +1 -12     jakarta-cactus/framework/src/java/share/org/apache/cactus/AbstractWebTestCase.java
  
  Index: AbstractWebTestCase.java
  ===================================================================
  RCS file: /home/cvs/jakarta-cactus/framework/src/java/share/org/apache/cactus/AbstractWebTestCase.java,v
  retrieving revision 1.2
  retrieving revision 1.3
  diff -u -r1.2 -r1.3
  --- AbstractWebTestCase.java	5 May 2002 20:08:50 -0000	1.2
  +++ AbstractWebTestCase.java	12 May 2002 19:42:28 -0000	1.3
  @@ -63,7 +63,6 @@
   import java.net.URLConnection;
   
   import org.apache.cactus.client.AbstractHttpClient;
  -import org.apache.cactus.client.ClientConfigurationChecker;
   import org.apache.cactus.util.ChainedRuntimeException;
   
   /**
  @@ -73,7 +72,7 @@
    *
    * @author <a href="mailto:vmassol@apache.org">Vincent Massol</a>
    *
  - * @version $Id: AbstractWebTestCase.java,v 1.2 2002/05/05 20:08:50 vmassol Exp $
  + * @version $Id: AbstractWebTestCase.java,v 1.3 2002/05/12 19:42:28 vmassol Exp $
    */
   public abstract class AbstractWebTestCase extends AbstractTestCase
   {
  @@ -217,16 +216,6 @@
           }
   
           return webResponse;
  -    }
  -
  -    /**
  -     * @see AbstractTestCase#checkConfiguration()
  -     */
  -    protected void checkConfiguration()
  -    {
  -        ClientConfigurationChecker.getInstance().checkCactusProperties();
  -        ClientConfigurationChecker.getInstance().checkHttpClient();
  -        ClientConfigurationChecker.getInstance().checkLog4j();
       }
   
       /**
  
  
  

--
To unsubscribe, e-mail:   <ma...@jakarta.apache.org>
For additional commands, e-mail: <ma...@jakarta.apache.org>