Empty scenario unexpectedly produces test steps

[davidjgoss]

👓 What did you see?

Given an empty scenario and no step definitions, we're seeing a TestStepStarted and TestStepFinished messages when it runs.

✅ What did you expect to see?

Expect to see test case messages but not test step messages.

📦 Which tool/library version are you using?

fake-cucumber 16.5.0

🔬 How could we reproduce it?

Feature: Empty Scenarios
  Scenario: Blank Scenario

And execute fake-cucumber

📚 Any additional context?

See cucumber/compatibility-kit#103.

[davidjgoss]

In makeTestCase.ts:

if (pickle.steps.length === 0) {
    const id = newId()
    const undefinedStep = new EmptyPickleTestStep(
      id,
      undefined,
      true,
      [],
      [],
      clock,
      stopwatch,
      makeErrorMessage
    )
    return new TestCase(newId(), [undefinedStep], pickle.id, clock)
  }

This explains the current behaviour. It can be removed but some other code will likely need making more defensive as a result.