Merge pull request #4 from JSpiner/develop

Create pr/issue template, change groupId, add ignore function

Author: JSpiner

.github/ISSUE_TEMPLATE.md

@@ -0,0 +1,5 @@
+Thanks for creating issue.
+Please don't file issues directly, use one of our available templates:
+
+[find a bug?](https://github.com/JSpiner/SafeExecutor/issues/new?template=bug.md)
+[hava a question?](https://github.com/JSpiner/SafeExecutor/issues/new?template=question.md)

.github/ISSUE_TEMPLATE/bug.md

@@ -0,0 +1,16 @@
+Thanks for creating bug issue.
+
+- library version : (x.x.x)
+- platform : (sdk 26)
+- etc : (run on background...)
+
+## Expected Behavior
+
+
+## Actual Behavior
+
+
+## Steps to Reproduce the Problem
+1.
+1.
+1.

.github/ISSUE_TEMPLATE/question.md

@@ -0,0 +1,7 @@
+Thanks for creating question issue.
+
+- library version : (x.x.x)
+- platform : (sdk 26)
+- etc : (run on background...)
+
+## Question

.github/PULL_REQUEST_TEMPLATE.md

@@ -0,0 +1,5 @@
+Thanks for pull request.
+Please don't create PR directly, use one of our available templates:
+
+[new feature or bug resolve?](?template=feature.md)
+[new release?](?template=release.md)

.github/PULL_REQUEST_TEMPLATE/feature.md

@@ -0,0 +1,11 @@
+## Description
+
+## Proposed Changes
+  -
+  -
+  -
+
+## Related Issues
+  -
+  -
+  -

.github/PULL_REQUEST_TEMPLATE/release.md

@@ -0,0 +1,11 @@
+Release Version : x.x.x
+
+## Proposed Changes
+  -
+  -
+  -
+
+## Related Pull Requests
+  -
+  -
+  -

README.md

@@ -4,6 +4,10 @@
 
 SafeExecutor is event-based error handler.
 
+[find a bug?](https://github.com/JSpiner/SafeExecutor/issues/new?template=bug.md)
+[hava a question?](https://github.com/JSpiner/SafeExecutor/issues/new?template=question.md)
+
+
 # STILL DEVELOPING !!!
 ### now support
 - error listener

pom.xml

@@ -4,9 +4,9 @@
          xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 http://maven.apache.org/xsd/maven-4.0.0.xsd">
     <modelVersion>4.0.0</modelVersion>
 
-    <groupId>net.jspiner</groupId>
+    <groupId>net.jspiner.executor</groupId>
     <artifactId>safeexecutor</artifactId>
-    <version>0.3</version>
+    <version>0.4</version>
 
     <name>SafeExecutor</name> 
     <description>event based error handler </description> 

src/main/java/net.jspiner.executor/SafeExecutor.java

@@ -2,7 +2,7 @@
 
 import java.util.ArrayList;
 
-public class SafeExecutor {
+public final class SafeExecutor {
 
     public SafeExecutor(){
         throw new RuntimeException("create SafeExecutor() is not allowed. use SafeExecutor.build();");
@@ -12,10 +12,11 @@ public static SafeExecutorBuilder build() {
         return new SafeExecutorBuilder();
     }
 
-    public static class SafeExecutorBuilder {
+    public static final class SafeExecutorBuilder {
 
         private ArrayList<Executable> executableList;
         private ErrorListener errorListener;
+        private boolean ignore;
 
         private SafeExecutorBuilder() {
             executableList = new ArrayList<>();
@@ -27,16 +28,24 @@ public SafeExecutorBuilder add(Executable executable) {
         }
 
         public SafeExecutorBuilder onError(ErrorListener errorListener) {
+            this.ignore = false;
             this.errorListener = errorListener;
             return this;
         }
 
+        public SafeExecutorBuilder ignore() {
+            this.ignore = true;
+            return this;
+        }
+
         public void run() {
             for (Executable executable : executableList) {
                 try {
                     executable.execute();
                 } catch (Exception error) {
-                    executeError(error);
+                    if (!ignore) {
+                        executeError(error);
+                    }
                 }
             }
         }

src/test/java/net.jspiner.executor/IgnoreErrorTest.java

@@ -0,0 +1,57 @@
+package net.jspiner.executor;
+
+import org.junit.Test;
+
+public class IgnoreErrorTest {
+
+    @Test(expected = RuntimeException.class)
+    public void notIgnoreTest() {
+        SafeExecutor.build()
+                .add(() -> {
+                            throw new RuntimeException("test error");
+                        }
+                ).run();
+    }
+
+    @Test
+    public void ignoreExceptionTest() {
+        SafeExecutor.build()
+                .add(() -> {
+                            throw new RuntimeException("test error");
+                        }
+                ).ignore()
+                .run();
+    }
+
+    @Test
+    public void ignoreNoExceptionTest() {
+        SafeExecutor.build()
+                .add(() -> doNothing())
+                .ignore()
+                .run();
+    }
+
+    @Test
+    public void ignoreMultipleExceptionTest() {
+        SafeExecutor.build()
+                .add(() -> {
+                    throw new RuntimeException("test error");
+                })
+                .add(() -> {
+                    throw new RuntimeException("test error");
+                })
+                .add(() -> {
+                    throw new RuntimeException("test error");
+                })
+                .add(() -> {
+                    throw new RuntimeException("test error");
+                })
+                .ignore()
+                .run();
+    }
+
+    private void doNothing() {
+        // do nothing for test
+    }
+
+}