You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@cxf.apache.org by Dan Diephouse <da...@envoisolutions.com> on 2007/06/13 17:10:17 UTC

Re: svn commit: r546749 - in /incubator/cxf/trunk: rt/transports/http/src/main/java/org/apache/cxf/transport/http/ systests/src/test/java/org/apache/cxf/systest/clustering/ systests/src/test/java/org/apache/cxf/systest/http/

Thanks and good eye! :-)

On 6/13/07, ningjiang@apache.org <ni...@apache.org> wrote:
>
> Author: ningjiang
> Date: Tue Jun 12 23:31:56 2007
> New Revision: 546749
>
> URL: http://svn.apache.org/viewvc?view=rev&rev=546749
> Log:
> Reopened the HTTPS 401 Redirect test and the Default sequential Strategy
> test
>
> Modified:
>
>     incubator/cxf/trunk/rt/transports/http/src/main/java/org/apache/cxf/transport/http/AbstractHTTPDestination.java
>
>     incubator/cxf/trunk/systests/src/test/java/org/apache/cxf/systest/clustering/FailoverTest.java
>
>     incubator/cxf/trunk/systests/src/test/java/org/apache/cxf/systest/http/HTTPConduitTest.java
>
> Modified:
> incubator/cxf/trunk/rt/transports/http/src/main/java/org/apache/cxf/transport/http/AbstractHTTPDestination.java
> URL:
> http://svn.apache.org/viewvc/incubator/cxf/trunk/rt/transports/http/src/main/java/org/apache/cxf/transport/http/AbstractHTTPDestination.java?view=diff&rev=546749&r1=546748&r2=546749
>
> ==============================================================================
> ---
> incubator/cxf/trunk/rt/transports/http/src/main/java/org/apache/cxf/transport/http/AbstractHTTPDestination.java
> (original)
> +++
> incubator/cxf/trunk/rt/transports/http/src/main/java/org/apache/cxf/transport/http/AbstractHTTPDestination.java
> Tue Jun 12 23:31:56 2007
> @@ -390,13 +390,13 @@
>              OutputStream responseStream = flushHeaders(outMessage);
>              if (null != responseStream) {
>                  wrappedStream = responseStream;
> -            }
> +            }
>          }
>
>          /**
>           * Perform any actions required on stream closure (handle
> response etc.)
>           */
> -        protected void doClose() throws IOException {
> +        public void close() throws IOException {
>              if (wrappedStream == null) {
>                  OutputStream responseStream = flushHeaders(outMessage);
>                  if (null != responseStream) {
> @@ -406,8 +406,7 @@
>              commitResponse();
>          }
>
> -        protected void onWrite() throws IOException {
> -        }
> +
>
>          private void commitResponse() {
>              try {
>
> Modified:
> incubator/cxf/trunk/systests/src/test/java/org/apache/cxf/systest/clustering/FailoverTest.java
> URL:
> http://svn.apache.org/viewvc/incubator/cxf/trunk/systests/src/test/java/org/apache/cxf/systest/clustering/FailoverTest.java?view=diff&rev=546749&r1=546748&r2=546749
>
> ==============================================================================
> ---
> incubator/cxf/trunk/systests/src/test/java/org/apache/cxf/systest/clustering/FailoverTest.java
> (original)
> +++
> incubator/cxf/trunk/systests/src/test/java/org/apache/cxf/systest/clustering/FailoverTest.java
> Tue Jun 12 23:31:56 2007
> @@ -45,7 +45,6 @@
> import org.junit.After;
> import org.junit.Before;
> import org.junit.BeforeClass;
> -import org.junit.Ignore;
> import org.junit.Test;
>
>
> @@ -280,8 +279,7 @@
>                      isWSAEnabledForCurrentEndpoint());
>      }
>
> -    @Test
> -    @Ignore
> +    @Test
>      public void testDefaultSequentialStrategy() throws Exception {
>          strategyTest(REPLICA_B, REPLICA_C, REPLICA_A, false);
>      }
>
> Modified:
> incubator/cxf/trunk/systests/src/test/java/org/apache/cxf/systest/http/HTTPConduitTest.java
> URL:
> http://svn.apache.org/viewvc/incubator/cxf/trunk/systests/src/test/java/org/apache/cxf/systest/http/HTTPConduitTest.java?view=diff&rev=546749&r1=546748&r2=546749
>
> ==============================================================================
> ---
> incubator/cxf/trunk/systests/src/test/java/org/apache/cxf/systest/http/HTTPConduitTest.java
> (original)
> +++
> incubator/cxf/trunk/systests/src/test/java/org/apache/cxf/systest/http/HTTPConduitTest.java
> Tue Jun 12 23:31:56 2007
> @@ -61,7 +61,6 @@
> import org.apache.hello_world.services.SOAPService;
>
> import org.junit.Before;
> -import org.junit.Ignore;
> import org.junit.Test;
>
> /**
> @@ -766,8 +765,7 @@
>       * This tests redirects through Gordy to Bethal. Bethal will
>       * supply a series of 401s. See PushBack401.
>       */
> -    @Test
> -    @Ignore
> +    @Test
>      public void testHttpsRedirect401Response() throws Exception {
>          startServer("Gordy");
>          startServer("Bethal");
>
>
>


-- 
Dan Diephouse
Envoi Solutions
http://envoisolutions.com | http://netzooid.com/blog