Details
Improvement
Resolved
Minor
Description
The name RunnableStory has become outdated and does not reflect the fact that we can run multiple stories or even have access to other Embedder functionality. Embeddable is a more generic name that's better suited.
The two methods that should remain in Embeddable are:
void useEmbedder(Embedder);
void run();
The other methods that were previously in RunnableStory can live in abstract implementations, such as ConfigurableEmbedder which JUnitStory extends and allows the configuration of the embedder for execution of a single story.
Not sure if this is a related issue or not.
The following method change would make more sense in AbstractStory
protected StoryEmbedder configuredEmbedder() { embedder.useConfiguration(getConfiguration()); embedder.useCandidateSteps(getSteps()); return embedder; }
While the getters are no longer part of the interface, they could be protected and part of the AbstractStory.
This allows a user to configure a story via overriding.
Thus the options for story configuration are:
1. Constructor
2. Lifecycle method (@Before)
3. Overriding getter methods
This is important for use with IoC containers as they should not be started up in the constructor of the test class but rather as part of the running of the test.