maven-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From csta...@apache.org
Subject svn commit: r1053390 - /maven/indexer/trunk/indexer-core/src/test/java/org/apache/maven/index/updater/DefaultIndexUpdaterEmbeddingIT.java
Date Tue, 28 Dec 2010 17:07:56 GMT
Author: cstamas
Date: Tue Dec 28 17:07:56 2010
New Revision: 1053390

URL: http://svn.apache.org/viewvc?rev=1053390&view=rev
Log:
Fixing cleanup: this IT was not properly cleaning up resources, the IndexingContext was never
shut down, hence daemon threads were left in there dangling.

Modified:
    maven/indexer/trunk/indexer-core/src/test/java/org/apache/maven/index/updater/DefaultIndexUpdaterEmbeddingIT.java

Modified: maven/indexer/trunk/indexer-core/src/test/java/org/apache/maven/index/updater/DefaultIndexUpdaterEmbeddingIT.java
URL: http://svn.apache.org/viewvc/maven/indexer/trunk/indexer-core/src/test/java/org/apache/maven/index/updater/DefaultIndexUpdaterEmbeddingIT.java?rev=1053390&r1=1053389&r2=1053390&view=diff
==============================================================================
--- maven/indexer/trunk/indexer-core/src/test/java/org/apache/maven/index/updater/DefaultIndexUpdaterEmbeddingIT.java
(original)
+++ maven/indexer/trunk/indexer-core/src/test/java/org/apache/maven/index/updater/DefaultIndexUpdaterEmbeddingIT.java
Tue Dec 28 17:07:56 2010
@@ -67,6 +67,8 @@ public class DefaultIndexUpdaterEmbeddin
                     null ) );
 
             updater.fetchAndUpdateIndex( updateRequest );
+            
+            ctx.close( false );
         }
         finally
         {
@@ -104,6 +106,8 @@ public class DefaultIndexUpdaterEmbeddin
                     }, null ) );
 
             updater.fetchAndUpdateIndex( updateRequest );
+            
+            ctx.close( false );
         }
         finally
         {
@@ -141,6 +145,8 @@ public class DefaultIndexUpdaterEmbeddin
                     }, null ) );
 
             updater.fetchAndUpdateIndex( updateRequest );
+            
+            ctx.close( false );
         }
         finally
         {
@@ -178,6 +184,8 @@ public class DefaultIndexUpdaterEmbeddin
                     }, null ) );
 
             updater.fetchAndUpdateIndex( updateRequest );
+            
+            ctx.close( false );
         }
         finally
         {
@@ -233,6 +241,8 @@ public class DefaultIndexUpdaterEmbeddin
             {
                 System.out.println( "Operation timed out due to short connection timeout,
as expected." );
             }
+            
+            ctx.close( false );
         }
         finally
         {
@@ -288,6 +298,8 @@ public class DefaultIndexUpdaterEmbeddin
             {
                 System.out.println( "Operation timed out due to short transaction timeout,
as expected." );
             }
+            
+            ctx.close( false );
         }
         finally
         {
@@ -330,6 +342,8 @@ public class DefaultIndexUpdaterEmbeddin
             {
                 System.out.println( "Operation timed out due to too many redirects, as expected."
);
             }
+            
+            ctx.close( false );
         }
         finally
         {
@@ -380,6 +394,8 @@ public class DefaultIndexUpdaterEmbeddin
             {
                 System.out.println( "Connection failed as expected." );
             }
+            
+            ctx.close( false );
         }
         finally
         {



Mime
View raw message