yegor256#455 fixed the issue

bedward70 · Nov 3, 2017 · 377b5e6 · 377b5e6
1 parent 54747c0
commit 377b5e6
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 1 deletion.
diff --git a/src/main/java/org/cactoos/Proc.java b/src/main/java/org/cactoos/Proc.java
@@ -75,6 +75,7 @@ public void exec(final X input) throws Exception {
                     "NULL instead of a valid input"
                 );
             }
+            this.origin.exec(input);
         }
     }
 }
diff --git a/src/test/java/org/cactoos/ProcTest.java b/src/test/java/org/cactoos/ProcTest.java
@@ -23,6 +23,9 @@
  */
 package org.cactoos;
 
+import java.util.concurrent.atomic.AtomicInteger;
+import org.hamcrest.MatcherAssert;
+import org.hamcrest.Matchers;
 import org.junit.Test;
 
 /**
@@ -46,6 +49,13 @@ public void failForNullInput() throws Exception {
 
     @Test
     public void okForNoNulls() throws Exception {
-        new Proc.NoNulls<>(input -> { }).exec(new Object());
+        final AtomicInteger counter = new AtomicInteger();
+        new Proc.NoNulls<AtomicInteger>(input -> { input.incrementAndGet(); })
+            .exec(counter);
+        MatcherAssert.assertThat(
+            "Can't involve the \"Proc.exec(X input)\" method",
+            counter.get(),
+            Matchers.equalTo(1)
+        );
     }
 }
-Original file line number
+Diff line change
@@ Expand Up / @@ -75,6 +75,7 @@ public void exec(final X input) throws Exception { @@
                         "NULL instead of a valid input"
                     );
                 }
+                this.origin.exec(input);
             }
         }
     }