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
         {


Reply via email to