search

Tyler-Keith-Thompson / CucumberSwift

A lightweight swift Cucumber implementation
https://tyler-keith-thompson.github.io/CucumberSwift/documentation/cucumberswift/
MIT License
74 stars 19 forks source link

Fix feature execution order #84

Closed DevMobileAS closed 1 year ago

DevMobileAS commented 1 year ago

See: https://cucumber.io/docs/cucumber/api/?lang=java → default: lexical

Thus, since in the actual implementation of CucumberSwift there's no specific configuration for the feature execution order yet, we should ensure that the features are executed regarding their file names.

Tyler-Keith-Thompson commented 1 year ago

Just to make sure I understand. "Lexical" order refers to alphabetical order? So 01 comes before A? This PR adds predictable ordering, but in theory we'd want to allow configuration for similar execution orders to what they have in the docs?

DevMobileAS commented 1 year ago

Hey Tyler,

you're right, lexical == alphabetical here.

Yes, the other documented execution orders like "lexical, reverse, random or random:[seed]" would be cool too. Maybe you could add some optional method reading options in the Cucumber extension someday.

But reading thru some other cucumber implementations, most of them at least follow the alphabetical order which should at least be the default imho. I'm unsure if and why we would ever need another execution order than that.