incorporate review feedback vol. 1

docs/extensions.adoc

@@ -156,7 +156,7 @@ The main difference to `Ignore` is that the test are executed, but test failures
 If the test passes without an error, then it will be reported as failure since the `PendingFeature` annotation should be removed.
 This way the tests will become part of the normal tests instead of being ignored forever.
 
-Groovy has the `groovy.transform.NotYetImplemented` annotation which is similar but behaves a differently.
+Groovy has the `groovy.transform.NotYetImplemented` annotation which is similar but behaves differently.
 
 * it will mark failing tests as passed
 * if at least one iteration of a data-driven test passes it will be reported as error
@@ -172,13 +172,21 @@ Groovy has the `groovy.transform.NotYetImplemented` annotation which is similar
 include::{sourcedir}/extension/PendingFeatureDocSpec.groovy[tag=example-a]
 ----
 
+You can also list one or multiple exceptions that are expected and cause the test to be skipped.
+If other exceptions occur, the test fails as if no annotation would be present.
+
+[source,groovy,indent=0]
+----
+include::{sourcedir}/extension/PendingFeatureDocSpec.groovy[tag=example-b]
+----
+
 It is also supported to have multiple `@PendingFeature` annotations or a mixture of `@PendingFeature` and
 `@PendingFeatureIf`, for example to ignore certain exceptions only under certain conditions or show different
 reasons for different exceptions.
 
 [source,groovy,indent=0]
 ----
-include::{sourcedir}/extension/PendingFeatureDocSpec.groovy[tag=example-b]
+include::{sourcedir}/extension/PendingFeatureDocSpec.groovy[tag=example-c]
 ----
 
 === PendingFeatureIf

spock-specs/src/test/groovy/org/spockframework/docs/extension/PendingFeatureDocSpec.groovy

@@ -1,6 +1,5 @@
 package org.spockframework.docs.extension
 
-import org.spockframework.runtime.extension.builtin.PreconditionContext
 import spock.lang.PendingFeature
 import spock.lang.PendingFeatureIf
 import spock.lang.Specification
@@ -15,6 +14,17 @@ class PendingFeatureDocSpec extends Specification {
   }
 
 // tag::example-b[]
+  @PendingFeature(exceptions = [
+    UnsupportedOperationException,
+    IllegalArgumentException
+  ])
+  def "I throw one of two exceptions"() {
+// end::example-b[]
+    expect:
+    throw new IllegalArgumentException()
+  }
+
+// tag::example-c[]
   @PendingFeature(
     exceptions = UnsupportedOperationException,
     reason = 'operation not yet supported')
@@ -25,7 +35,7 @@ class PendingFeatureDocSpec extends Specification {
     value = { os.windows },
     reason = 'Does not yet work on Windows')
   def "I have various problems in certain situations"() {
-// end::example-b[]
+// end::example-c[]
     expect:
     throw new IllegalArgumentException()
   }