Handle TestNG SkipException correctly, fixes #355

jgiven-core/src/main/java/com/tngtech/jgiven/impl/intercept/StepInterceptorImpl.java

@@ -216,7 +216,8 @@ private void handleMethod( Object stageInstance, Method paramMethod, Object[] ar
     }
 
     private void handleThrowable( Throwable t ) throws Throwable {
-        if( t.getClass().getName().equals( "org.junit.AssumptionViolatedException" ) ) {
+        if( t.getClass().getName().equals( "org.junit.AssumptionViolatedException" ) ||
+            t.getClass().getName().equals( "org.testng.SkipException")) {
             throw t;
         }
 

jgiven-testng/src/test/java/com/tngtech/jgiven/testng/SkipExceptionTest.java

@@ -0,0 +1,29 @@
+package com.tngtech.jgiven.testng;
+
+import static org.assertj.core.api.Assertions.assertThat;
+
+import org.assertj.core.api.Assertions;
+
+import com.tngtech.jgiven.annotation.Description;
+import com.tngtech.jgiven.report.model.ScenarioCaseModel;
+import com.tngtech.jgiven.report.model.StepStatus;
+import org.testng.SkipException;
+import org.testng.annotations.Test;
+
+@Description( "SkipException are handled correctly" )
+public class SkipExceptionTest extends SimpleScenarioTest<TestNgTest.TestSteps> {
+
+    @Test
+    public void TestNG_skipped_exceptions_should_be_treated_correctly() throws Throwable {
+        try {
+            given().skipped_exception_is_thrown();
+            Assertions.fail( "SkipException should have been thrown" );
+        } catch( SkipException e ) {}
+
+        getScenario().finished();
+
+        ScenarioCaseModel aCase = getScenario().getModel().getLastScenarioModel().getCase( 0 );
+        assertThat( aCase.getStep( 0 ).getStatus() ).isEqualTo( StepStatus.PASSED );
+    }
+
+}

jgiven-testng/src/test/java/com/tngtech/jgiven/testng/TestNgTest.java

@@ -115,5 +115,9 @@ public void mixed_with( String something ) {}
 
         public void something() {}
 
+        public void skipped_exception_is_thrown() {
+            throw new org.testng.SkipException( "should be skipped" );
+        }
+
     }
 }