Skip to content

Commit 9c84654

Browse files
committed
Avoid returning null for CoreRunner steps
1 parent aaef16f commit 9c84654

File tree

2 files changed

+2
-2
lines changed

2 files changed

+2
-2
lines changed

java/server/src/org/openqa/selenium/server/htmlrunner/NonReflectiveSteps.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -115,7 +115,7 @@ public NextStepDecorator execute(Selenium selenium, TestState state) {
115115

116116
steps.put("store", (((locator, value) -> ((selenium, state) -> {
117117
state.store(state.expand(locator), state.expand(value));
118-
return null;
118+
return NextStepDecorator.IDENTITY;
119119
}))));
120120

121121
return steps.build();

java/server/src/org/openqa/selenium/server/htmlrunner/ReflectivelyDiscoveredSteps.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -156,7 +156,7 @@ private static ImmutableMap<String, CoreStepFactory> discover() {
156156
String value = state.expand(val);
157157
Object toStore = invokeMethod(method, selenium, buildArgs(method, locator, value));
158158
state.store(locator, toStore);
159-
return null;
159+
return NextStepDecorator.IDENTITY;
160160
}));
161161
}
162162

0 commit comments

Comments
 (0)