Merge pull request #480 from metafacture/inputUsed

Reset `inputUsed` after processing to fix reusing opener
metafacture · Mar 10, 2023 · 59ea7e4 · 59ea7e4
2 parents 3d01453 + dc636b8
commit 59ea7e4
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 1 deletion.
diff --git a/metafacture-io/src/main/java/org/metafacture/io/HttpOpener.java b/metafacture-io/src/main/java/org/metafacture/io/HttpOpener.java
@@ -265,6 +265,9 @@ public void process(final String input) {
         catch (final IOException e) {
             throw new MetafactureException(e);
         }
+        finally {
+            inputUsed = false;
+        }
     }
 
     private String getInput(final String input, final String value) {

diff --git a/metafacture-io/src/test/java/org/metafacture/io/HttpOpenerTest.java b/metafacture-io/src/test/java/org/metafacture/io/HttpOpenerTest.java
@@ -39,6 +39,8 @@
 import org.mockito.junit.MockitoJUnit;
 import org.mockito.junit.MockitoRule;
 
+import static org.mockito.Mockito.times;
+
 import java.io.IOException;
 import java.io.Reader;
 import java.util.Arrays;
@@ -318,11 +320,15 @@ private void shouldPerformRequest(final String input, final HttpOpener.Method me
         WireMock.stubFor(stub);
 
         opener.process(String.format(input, baseUrl));
+
+        // use the opener a second time in a workflow:
+        opener.process(String.format(input, baseUrl));
+
         opener.closeStream();
 
         WireMock.verify(request);
 
-        Mockito.verify(receiver).process(processedObject.capture());
+        Mockito.verify(receiver, times(2)).process(processedObject.capture());
         Assert.assertEquals(responseBody, ResourceUtil.readAll(processedObject.getValue()));
     }