Cucumber doesnt have a standard on using locator - Locator selection is standard across and locator should be unique and reliable(Also above selector isnt reliable as it just says button, if you are building data-test-id attribute, give it with value which is unique and reliable than just saying button). Another suggestion would be - never pass locators from feature file, you can keep it in JSON if thats gonna work for you. Cucumber/BDD is all about user behaviour and Workflows.