PR-2091 Update the changelog. Using the same pattern of the compileRu…

…le method as in Ruby.
cucumber · Oct 8, 2022 · c2f80a3 · c2f80a3
1 parent 1816ce7
commit c2f80a3
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 5 deletions.
diff --git a/gherkin/CHANGELOG.md b/gherkin/CHANGELOG.md
@@ -22,6 +22,7 @@ This document is formatted according to the principles of [Keep A CHANGELOG](htt
 
 ### Fixed
 
+* [Javascript] Fix compiling a rule with scenario outline of gherkin document is different from ruby's ([#2091](https://github.com/cucumber/common/pull/2091))
 * [Python] Fix gherkin-python for compiling example values with trailing backslash ([#2048](https://github.com/cucumber/common/pull/2048), [#1954](https://github.com/cucumber/common/issues/1954))
 * [PHP] Disallow installation of Messages `18.x` ([#2034](https://github.com/cucumber/common/pull/2034))
 

diff --git a/gherkin/javascript/src/pickles/compile.ts b/gherkin/javascript/src/pickles/compile.ts
@@ -79,12 +79,18 @@ function compileRule(
   rule.children.forEach((stepsContainer) => {
     if (stepsContainer.background) {
       ruleBackgroundSteps = ruleBackgroundSteps.concat(stepsContainer.background.steps)
+    } else if (stepsContainer.scenario.examples.length === 0) {
+      compileScenario(
+        tags,
+        ruleBackgroundSteps,
+        stepsContainer.scenario,
+        language,
+        pickles,
+        uri,
+        newId
+      )
     } else {
-      const compileFunc = stepsContainer.scenario.examples.length === 0 
-        ? compileScenario
-        : compileScenarioOutline
-
-      compileFunc(
+      compileScenarioOutline(
         tags,
         ruleBackgroundSteps,
         stepsContainer.scenario,