Merge pull request #198 from Papooch/fix/scenario-outline-with-placeh…

…olders

Fix scenario outline title placeholder to work with autoBindSteps

examples/typescript/specs/step-definitions/auto-step-binding-outline.steps.ts

@@ -0,0 +1,27 @@
+import { StepDefinitions, autoBindSteps, loadFeature } from '../../../../src';
+import { OnlineSales } from '../../src/online-sales';
+
+export const salesSteps: StepDefinitions = ({ given, when, then }) => {
+  let onlineSales: OnlineSales;
+  let salesPrice: number | null;
+
+  beforeEach(() => {
+    onlineSales = new OnlineSales();
+  });
+
+  given(/^I have a\(n\) (.+)$/, (item: string) => {
+    onlineSales.listItem(item);
+  });
+
+  when(/^I sell the (.+)$/, (item: string) => {
+    salesPrice = onlineSales.sellItem(item);
+  });
+
+  then(/^I should get \$(\d+)$/, (expectedSalesPrice: string) => {
+    expect(salesPrice).toBe(parseInt(expectedSalesPrice, 10));
+  });
+};
+
+const feature = loadFeature('./examples/typescript/specs/features/scenario-outlines.feature');
+
+autoBindSteps([feature], [salesSteps]);

src/automatic-step-binding.ts

@@ -31,7 +31,12 @@ export const createAutoBindSteps = (jestLike: IJestLike) => {
 
     features.forEach(feature => {
       defineFeature(feature, test => {
-        const scenarioOutlineScenarios = feature.scenarioOutlines.map(scenarioOutline => scenarioOutline.scenarios[0]);
+        const scenarioOutlineScenarios = feature.scenarioOutlines.map(scenarioOutline => ({
+          ...scenarioOutline.scenarios[0],
+          // we need to use the original title with un-expanded placeholders,
+          // otherwise it will try to match the expanded version in the feature file and fail.
+          title: scenarioOutline.title,
+        }));
 
         const scenarios = [...feature.scenarios, ...scenarioOutlineScenarios];