Skip to content

Commit

Permalink
removing duplicated step insertion
Browse files Browse the repository at this point in the history
  • Loading branch information
vantreeseba committed Feb 24, 2014
1 parent e7e2198 commit 4f76479
Showing 1 changed file with 5 additions and 14 deletions.
19 changes: 5 additions & 14 deletions lib/cucumber/ast/assembler.js
Original file line number Diff line number Diff line change
Expand Up @@ -73,7 +73,7 @@ var Assembler = function (features, filter) {
self.suggestFeature(feature);
},

insertScenario: function insertScenario(scenario) {
insertScenario: function insertScenario(scenario) {
self.applyCurrentFeatureTagsToElement(scenario);
self.applyStashedTagsToElement(scenario);
self.setCurrentScenarioOrBackground(scenario);
Expand Down Expand Up @@ -103,30 +103,21 @@ var Assembler = function (features, filter) {
self.stashTag(tag);
},

insertBuiltScenarioAndSteps: function insertBuiltScenarioAndSteps(scenario){
self.insertScenario(scenario);
var steps = scenario.getSteps();

steps.syncForEach(function(step){
self.insertStep(step);
});
},

convertScenarioOutlineToScenarios: function convertScenarioOutlineToScenarios(scenario){
var subScenarios = scenario.buildScenarios();
subScenarios.syncForEach(self.insertBuiltScenarioAndSteps);
var subScenarios = scenario.buildScenarios();
subScenarios.syncForEach(self.insertScenario);
},

convertScenarioOutlinesToScenarios: function convertScenarioOutlinesToScenarios(){
var currentFeature = self.getCurrentFeature();
var scenarios = currentFeature.getFeatureElements();

scenarios.syncForEach(function(scenario){
self.convertScenarioOutlineToScenarios(scenario);
});
},

finish: function finish() {
finish: function finish() {
self.convertScenarioOutlinesToScenarios();
self.tryEnrollingSuggestedFeature();
},
Expand Down

0 comments on commit 4f76479

Please sign in to comment.