You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@lucene.apache.org by si...@apache.org on 2020/08/20 06:47:10 UTC

[lucene-solr] branch master updated: Fix test to actually use the resource from the try/finally block

This is an automated email from the ASF dual-hosted git repository.

simonw pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/lucene-solr.git


The following commit(s) were added to refs/heads/master by this push:
     new fa878eb  Fix test to actually use the resource from the try/finally block
fa878eb is described below

commit fa878eb5b8bcc6d52e9a2e0745c176489b931490
Author: Simon Willnauer <si...@apache.org>
AuthorDate: Thu Aug 20 08:45:14 2020 +0200

    Fix test to actually use the resource from the try/finally block
---
 .../src/test/org/apache/lucene/index/TestIndexWriterExceptions.java     | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/lucene/core/src/test/org/apache/lucene/index/TestIndexWriterExceptions.java b/lucene/core/src/test/org/apache/lucene/index/TestIndexWriterExceptions.java
index 2594f2f..aa3a162 100644
--- a/lucene/core/src/test/org/apache/lucene/index/TestIndexWriterExceptions.java
+++ b/lucene/core/src/test/org/apache/lucene/index/TestIndexWriterExceptions.java
@@ -2056,6 +2056,8 @@ public class TestIndexWriterExceptions extends LuceneTestCase {
           return true;
         }
       }) {
+        writer.rollback();
+        fail();
       }
     } catch (RuntimeException e) {
       assertEquals("boom", e.getMessage());