Skip to content

Commit b3b5361

Browse files
committed
Missing String.format parameters in DefaultJavaObjectFactory. Closes #336
1 parent 4c7c0de commit b3b5361

File tree

2 files changed

+3
-2
lines changed

2 files changed

+3
-2
lines changed

History.md

+1
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,6 @@
11
## [Git master](https://github.com/cucumber/cucumber-jvm/compare/v1.0.7...master)
22

3+
* [Java] Missing String.format parameters in DefaultJavaObjectFactory ([#336](https://github.com/cucumber/cucumber-jvm/issues/336) paulkrause88, Aslak Hellesøy)
34
* [Core] Exceptions being swallowed if reported in a Hook ([#133](https://github.com/cucumber/cucumber-jvm/issues/133) David Kowis, Aslak Hellesøy)
45
* [Core] Added `DataTable.asMaps()` and made all returned lists immutable. (Aslak Hellesøy).
56
* [Java] The java-helloworld example has a simple example illustrating data tables and doc strings. (Aslak Hellesøy).

java/src/main/java/cucumber/fallback/runtime/java/DefaultJavaObjectFactory.java

+2-2
Original file line numberDiff line numberDiff line change
@@ -40,9 +40,9 @@ private <T> T cacheNewInstance(Class<T> type) {
4040
instances.put(type, instance);
4141
return instance;
4242
} catch (NoSuchMethodException e) {
43-
throw new CucumberException(String.format("%s doesn't have an empty constructor. If you need DI, put cucumber-picocontainer on the classpath"), e);
43+
throw new CucumberException(String.format("%s doesn't have an empty constructor. If you need DI, put cucumber-picocontainer on the classpath", type), e);
4444
} catch (Exception e) {
45-
throw new CucumberException(String.format("Failed to instantiate %s"), e);
45+
throw new CucumberException(String.format("Failed to instantiate %s", type), e);
4646
}
4747
}
4848
}

0 commit comments

Comments
 (0)