returns a boolean indicating whether or not stepdefs/hooks for that class should be registered.

aslakhellesoy · aslakhellesoy · commit 65bbb266a0af · 2015-07-09T23:20:48.000+02:00
diff --git a/History.md b/History.md
@@ -1,12 +1,13 @@
 ## [1.2.4-SNAPSHOT](https://github.com/cucumber/cucumber-jvm/compare/v1.2.3...master) (In Git)
 
+* [Java] `ObjectFactory.addClass` returns a boolean indicating whether or not stepdefs/hooks for that class should be registered. (Aslak Hellesøy)
 * [examples] Fix to allow lein test to to run successfully ([#805](https://github.com/cucumber/cucumber-jvm/pull/805) Chris Howe-Jones)
 
 ## [1.2.3](https://github.com/cucumber/cucumber-jvm/compare/v1.2.2...v1.2.3) (2015-07-07)
 
 * [Core] Make the Rerun Formatter consistent with the exit code ([#871](https://github.com/cucumber/cucumber-jvm/pull/871) Björn Rasmusson)
 * [OSGi] Cucumber is ready to run in OSGi containers ([#873](https://github.com/cucumber/cucumber-jvm/pull/873), [#799](https://github.com/cucumber/cucumber-jvm/pull/799) @HendrikSP)
-* [Core] `cucumber.runtime.java.ObjectFactory` moved to `cucumber.api.java.ObjectFactory`. Custom implementation can
+* [Java] `cucumber.runtime.java.ObjectFactory` moved to `cucumber.api.java.ObjectFactory`. Custom implementation can
   be specified in `cucumber.properties` with `cucumber.api.java.ObjectFactory=my.special.KindOfObjectFactory`. (Closes [#290](https://github.com/cucumber/cucumber-jvm/issues/290) Aslak Hellesøy)
 * [Core] Properly decode jar URLs with spaces (%20) - ([#866](https://github.com/cucumber/cucumber-jvm/issues/866) Aslak Hellesøy)
 * [Java] Arity mismatch Java8 Step Definition error ([#852](https://github.com/cucumber/cucumber-jvm/issues/852), [#847](https://github.com/cucumber/cucumber-jvm/pull/847) David Coelho)
@@ -18,10 +19,10 @@
 * [Core] Call all formatters, also in case of unimplemented methods ([#842](https://github.com/cucumber/cucumber-jvm/pull/842), [#803](https://github.com/cucumber/cucumber-jvm/issues/803) Björn Rasmusson)
 * [TestNG] Run each feature as separate TestNG test ([#817](https://github.com/cucumber/cucumber-jvm/pull/817), [#653](https://github.com/cucumber/cucumber-jvm/pull/653) Dmitry Sidorenko, Björn Rasmusson)
 * [Core] Implement TestNG-compatible XML formatter ([#818](https://github.com/cucumber/cucumber-jvm/pull/818), [#621](https://github.com/cucumber/cucumber-jvm/pull/621) Dmitry Berezhony, Björn Rasmusson)
-* `DataTable.diff(List)` gives proper error message when the `List` argument is empty (Aslak Hellesøy)
-* Execute no scenarios when the rerun file is empty ([#840](https://github.com/cucumber/cucumber-jvm/issues/840) Björn Rasmusson)
-* Snippets for quoted arguments changed from `(.*?)` to `([^\"]*)` (which is how it was before 1.1.6). See [cucumber/cucumber#663](https://github.com/cucumber/cucumber/pull/663) (Aslak Hellesøy)
-* Fix non running gradle example ([#839](https://github.com/cucumber/cucumber-jvm/pull/839) Ole Christian Langfjæran)
+* [Core] `DataTable.diff(List)` gives proper error message when the `List` argument is empty (Aslak Hellesøy)
+* [Core] Execute no scenarios when the rerun file is empty ([#840](https://github.com/cucumber/cucumber-jvm/issues/840) Björn Rasmusson)
+* [Core] Snippets for quoted arguments changed from `(.*?)` to `([^\"]*)` (which is how it was before 1.1.6). See [cucumber/cucumber#663](https://github.com/cucumber/cucumber/pull/663) (Aslak Hellesøy)
+* [Core] Fix non running gradle example ([#839](https://github.com/cucumber/cucumber-jvm/pull/839) Ole Christian Langfjæran)
 * [Clojure] Improved documentation for the clojure module ([#864](https://github.com/cucumber/cucumber-jvm/pull/864) Paul Doran)
 
 ## [1.2.2](https://github.com/cucumber/cucumber-jvm/compare/v1.2.0...v1.2.2) (2015-01-13)
diff --git a/android/src/main/java/cucumber/runtime/android/AndroidObjectFactory.java b/android/src/main/java/cucumber/runtime/android/AndroidObjectFactory.java
@@ -48,8 +48,8 @@ public void stop() {
     }
 
     @Override
-    public void addClass(final Class<?> clazz) {
-        delegate.addClass(clazz);
+    public boolean addClass(final Class<?> clazz) {
+        return delegate.addClass(clazz);
     }
 
     @Override
diff --git a/guice/src/main/java/cucumber/runtime/java/guice/impl/GuiceFactory.java b/guice/src/main/java/cucumber/runtime/java/guice/impl/GuiceFactory.java
@@ -31,7 +31,9 @@ public GuiceFactory() throws IOException {
         this.injector = injector;
     }
 
-    public void addClass(Class<?> clazz) {}
+    public boolean addClass(Class<?> clazz) {
+        return true;
+    }
 
     public void start() {
         injector.getInstance(ScenarioScope.class).enterScope();
diff --git a/java/src/main/java/cucumber/api/java/ObjectFactory.java b/java/src/main/java/cucumber/api/java/ObjectFactory.java
@@ -19,8 +19,9 @@ public interface ObjectFactory {
      * Collects glue classes in the classpath. Called once on init.
      *
      * @param glueClass Glue class containing cucumber.api annotations (Before, Given, When, ...)
+     * @return true if stepdefs and hooks in this class should be used, false if they should be ignored.
      */
-    void addClass(Class<?> glueClass);
+    boolean addClass(Class<?> glueClass);
 
     /**
      * Provides the glue instances used to execute the current scenario. The instance can be prepared in
diff --git a/java/src/main/java/cucumber/runtime/java/DefaultJavaObjectFactory.java b/java/src/main/java/cucumber/runtime/java/DefaultJavaObjectFactory.java
@@ -22,7 +22,8 @@ public void stop() {
         instances.clear();
     }
 
-    public void addClass(Class<?> clazz) {
+    public boolean addClass(Class<?> clazz) {
+        return true;
     }
 
     public <T> T getInstance(Class<T> type) {
diff --git a/java/src/main/java/cucumber/runtime/java/JavaBackend.java b/java/src/main/java/cucumber/runtime/java/JavaBackend.java
@@ -93,8 +93,9 @@ public void loadGlue(Glue glue, List<String> gluePaths) {
                     continue;
                 }
 
-                objectFactory.addClass(glueClass);
-                glueBaseClasses.add(glueClass);
+                if (objectFactory.addClass(glueClass)) {
+                    glueBaseClasses.add(glueClass);
+                }
             }
         }
     }
@@ -146,8 +147,9 @@ public String getSnippet(Step step, FunctionNameGenerator functionNameGenerator)
 
     void addStepDefinition(Annotation annotation, Method method) {
         try {
-            objectFactory.addClass(method.getDeclaringClass());
-            glue.addStepDefinition(new JavaStepDefinition(method, pattern(annotation), timeoutMillis(annotation), objectFactory));
+            if (objectFactory.addClass(method.getDeclaringClass())) {
+                glue.addStepDefinition(new JavaStepDefinition(method, pattern(annotation), timeoutMillis(annotation), objectFactory));
+            }
         } catch (DuplicateStepDefinitionException e) {
             throw e;
         } catch (Throwable e) {
@@ -164,16 +166,16 @@ public void addStepDefinition(String regexp, long timeoutMillis, StepdefBody bod
     }
 
     void addHook(Annotation annotation, Method method) {
-        objectFactory.addClass(method.getDeclaringClass());
-
-        if (annotation.annotationType().equals(Before.class)) {
-            String[] tagExpressions = ((Before) annotation).value();
-            long timeout = ((Before) annotation).timeout();
-            glue.addBeforeHook(new JavaHookDefinition(method, tagExpressions, ((Before) annotation).order(), timeout, objectFactory));
-        } else {
-            String[] tagExpressions = ((After) annotation).value();
-            long timeout = ((After) annotation).timeout();
-            glue.addAfterHook(new JavaHookDefinition(method, tagExpressions, ((After) annotation).order(), timeout, objectFactory));
+        if (objectFactory.addClass(method.getDeclaringClass())) {
+            if (annotation.annotationType().equals(Before.class)) {
+                String[] tagExpressions = ((Before) annotation).value();
+                long timeout = ((Before) annotation).timeout();
+                glue.addBeforeHook(new JavaHookDefinition(method, tagExpressions, ((Before) annotation).order(), timeout, objectFactory));
+            } else {
+                String[] tagExpressions = ((After) annotation).value();
+                long timeout = ((After) annotation).timeout();
+                glue.addAfterHook(new JavaHookDefinition(method, tagExpressions, ((After) annotation).order(), timeout, objectFactory));
+            }
         }
     }
 
diff --git a/java/src/test/java/cucumber/runtime/java/SingletonFactory.java b/java/src/test/java/cucumber/runtime/java/SingletonFactory.java
@@ -22,7 +22,8 @@ public void stop() {
     }
 
     @Override
-    public void addClass(Class<?> clazz) {
+    public boolean addClass(Class<?> clazz) {
+        return true;
     }
 
     @Override
diff --git a/needle/src/main/java/cucumber/runtime/java/needle/NeedleFactory.java b/needle/src/main/java/cucumber/runtime/java/needle/NeedleFactory.java
@@ -53,13 +53,14 @@ public void stop() {
     }
 
     @Override
-    public void addClass(final Class<?> type) {
+    public boolean addClass(final Class<?> type) {
         logger.trace("addClass(): " + type.getCanonicalName());
 
         // build up cache keys ...
         if (!cachedStepsInstances.containsKey(type)) {
             cachedStepsInstances.put(type, null);
         }
+        return true;
     }
 
     private void assertTypeHasBeenAdded(final Class<?> type) {
diff --git a/openejb/src/main/java/cucumber/runtime/java/openejb/OpenEJBObjectFactory.java b/openejb/src/main/java/cucumber/runtime/java/openejb/OpenEJBObjectFactory.java
@@ -39,8 +39,9 @@ public void stop() {
     }
 
     @Override
-    public void addClass(Class<?> clazz) {
+    public boolean addClass(Class<?> clazz) {
         classes.add(clazz.getName());
+        return true;
     }
 
     @Override
diff --git a/osgi/src/main/java/cucumber/java/runtime/osgi/OsgiObjectFactoryBase.java b/osgi/src/main/java/cucumber/java/runtime/osgi/OsgiObjectFactoryBase.java
@@ -21,7 +21,8 @@ public void stop() {
     }
 
     @Override
-    public void addClass(Class<?> glueClass) {
+    public boolean addClass(Class<?> glueClass) {
+        return true;
     }
 
     @Override
diff --git a/picocontainer/src/main/java/cucumber/runtime/java/picocontainer/PicoFactory.java b/picocontainer/src/main/java/cucumber/runtime/java/picocontainer/PicoFactory.java
@@ -26,10 +26,11 @@ public void stop() {
         pico.dispose();
     }
 
-    public void addClass(Class<?> clazz) {
+    public boolean addClass(Class<?> clazz) {
         if (Utils.isInstantiable(clazz) && classes.add(clazz)) {
             addConstructorDependencies(clazz);
         }
+        return true;
     }
 
     public <T> T getInstance(Class<T> type) {
diff --git a/spring/src/main/java/cucumber/runtime/java/spring/SpringFactory.java b/spring/src/main/java/cucumber/runtime/java/spring/SpringFactory.java
@@ -56,7 +56,7 @@ public SpringFactory() {
     }
 
     @Override
-    public void addClass(final Class<?> stepClass) {
+    public boolean addClass(final Class<?> stepClass) {
         if (!stepClasses.contains(stepClass)) {
             if (dependsOnSpringContext(stepClass)) {
                 if (stepClassWithSpringContext == null) {
@@ -66,8 +66,8 @@ public void addClass(final Class<?> stepClass) {
                 }
             }
             stepClasses.add(stepClass);
-
         }
+        return true;
     }
 
     private void checkAnnotationsEqual(Class<?> stepClassWithSpringContext, Class<?> stepClass) {
diff --git a/weld/src/main/java/cucumber/runtime/java/weld/WeldFactory.java b/weld/src/main/java/cucumber/runtime/java/weld/WeldFactory.java
@@ -43,7 +43,8 @@ public void stop() {
     }
 
     @Override
-    public void addClass(Class<?> clazz) {
+    public boolean addClass(Class<?> clazz) {
+        return true;
     }
 
     @Override

Original file line number	Diff line number	Diff line change
`@@ -48,8 +48,8 @@ public void stop() {`
`48`	`48`	`}`
`49`	`49`
`50`	`50`	`@Override`
`51`		`- public void addClass(final Class<?> clazz) {`
`52`		`- delegate.addClass(clazz);`
	`51`	`+ public boolean addClass(final Class<?> clazz) {`
	`52`	`+ return delegate.addClass(clazz);`
`53`	`53`	`}`
`54`	`54`
`55`	`55`	`@Override`
Original file line number	Diff line number	Diff line change
`@@ -31,7 +31,9 @@ public GuiceFactory() throws IOException {`
`31`	`31`	`this.injector = injector;`
`32`	`32`	`}`
`33`	`33`
`34`		`- public void addClass(Class<?> clazz) {}`
	`34`	`+ public boolean addClass(Class<?> clazz) {`
	`35`	`+ return true;`
	`36`	`+ }`
`35`	`37`
`36`	`38`	`public void start() {`
`37`	`39`	`injector.getInstance(ScenarioScope.class).enterScope();`
Original file line number	Diff line number	Diff line change
`@@ -22,7 +22,8 @@ public void stop() {`
`22`	`22`	`instances.clear();`
`23`	`23`	`}`
`24`	`24`
`25`		`- public void addClass(Class<?> clazz) {`
	`25`	`+ public boolean addClass(Class<?> clazz) {`
	`26`	`+ return true;`
`26`	`27`	`}`
`27`	`28`
`28`	`29`	`public <T> T getInstance(Class<T> type) {`
Original file line number	Diff line number	Diff line change
`@@ -53,13 +53,14 @@ public void stop() {`
`53`	`53`	`}`
`54`	`54`
`55`	`55`	`@Override`
`56`		`- public void addClass(final Class<?> type) {`
	`56`	`+ public boolean addClass(final Class<?> type) {`
`57`	`57`	`logger.trace("addClass(): " + type.getCanonicalName());`
`58`	`58`
`59`	`59`	`// build up cache keys ...`
`60`	`60`	`if (!cachedStepsInstances.containsKey(type)) {`
`61`	`61`	`cachedStepsInstances.put(type, null);`
`62`	`62`	`}`
	`63`	`+ return true;`
`63`	`64`	`}`
`64`	`65`
`65`	`66`	`private void assertTypeHasBeenAdded(final Class<?> type) {`
Original file line number	Diff line number	Diff line change
`@@ -39,8 +39,9 @@ public void stop() {`
`39`	`39`	`}`
`40`	`40`
`41`	`41`	`@Override`
`42`		`- public void addClass(Class<?> clazz) {`
	`42`	`+ public boolean addClass(Class<?> clazz) {`
`43`	`43`	`classes.add(clazz.getName());`
	`44`	`+ return true;`
`44`	`45`	`}`
`45`	`46`
`46`	`47`	`@Override`
Original file line number	Diff line number	Diff line change
`@@ -21,7 +21,8 @@ public void stop() {`
`21`	`21`	`}`
`22`	`22`
`23`	`23`	`@Override`
`24`		`- public void addClass(Class<?> glueClass) {`
	`24`	`+ public boolean addClass(Class<?> glueClass) {`
	`25`	`+ return true;`
`25`	`26`	`}`
`26`	`27`
`27`	`28`	`@Override`
Original file line number	Diff line number	Diff line change
`@@ -26,10 +26,11 @@ public void stop() {`
`26`	`26`	`pico.dispose();`
`27`	`27`	`}`
`28`	`28`
`29`		`- public void addClass(Class<?> clazz) {`
	`29`	`+ public boolean addClass(Class<?> clazz) {`
`30`	`30`	`if (Utils.isInstantiable(clazz) && classes.add(clazz)) {`
`31`	`31`	`addConstructorDependencies(clazz);`
`32`	`32`	`}`
	`33`	`+ return true;`
`33`	`34`	`}`
`34`	`35`
`35`	`36`	`public <T> T getInstance(Class<T> type) {`
Original file line number	Diff line number	Diff line change
`@@ -56,7 +56,7 @@ public SpringFactory() {`
`56`	`56`	`}`
`57`	`57`
`58`	`58`	`@Override`
`59`		`- public void addClass(final Class<?> stepClass) {`
	`59`	`+ public boolean addClass(final Class<?> stepClass) {`
`60`	`60`	`if (!stepClasses.contains(stepClass)) {`
`61`	`61`	`if (dependsOnSpringContext(stepClass)) {`
`62`	`62`	`if (stepClassWithSpringContext == null) {`
`@@ -66,8 +66,8 @@ public void addClass(final Class<?> stepClass) {`
`66`	`66`	`}`
`67`	`67`	`}`
`68`	`68`	`stepClasses.add(stepClass);`
`69`		`-`
`70`	`69`	`}`
	`70`	`+ return true;`
`71`	`71`	`}`
`72`	`72`
`73`	`73`	`private void checkAnnotationsEqual(Class<?> stepClassWithSpringContext, Class<?> stepClass) {`
Original file line number	Diff line number	Diff line change
`@@ -43,7 +43,8 @@ public void stop() {`
`43`	`43`	`}`
`44`	`44`
`45`	`45`	`@Override`
`46`		`- public void addClass(Class<?> clazz) {`
	`46`	`+ public boolean addClass(Class<?> clazz) {`
	`47`	`+ return true;`
`47`	`48`	`}`
`48`	`49`
`49`	`50`	`@Override`