Error: Cannot find module 'test/cypress/support/step_definitions/shared.js' about cypress-cucumber-preprocessor

[lgandecki]

As I thought:

putting this:

  "cypress-cucumber-preprocessor": {
    "step_definitions": "test/cypress/support/step_definitions"
  }

in your package.json fixes the problem.

I'm thinking whether I should try to make it work for the mistakenly used "fileServerFolder": "test/cypress", as well, or just make a note in the documentation that this is no longer supported (it was deprecated a few versions back)

[lgandecki]

Any chance of a simple reproduction? Also - could you go back a version or two and see if that helps?

[ChrisBAshton]

Just re-tried it in a few versions:

• 1.2.2 broken, see error in this issue description
• 1.2.0 broken, see error in this issue description
• 1.1.0 works (but 1 of our tests fails for some reason!)
• 1.0.0 works fully

So breaking change was introduced in 1.2.0.

[ChrisBAshton]

I'd take a wild guess and assume it's because path.relative got removed :)

[lgandecki]

I mean, we don't resolve the paths manually anymore, so the whole operation from that line was removed.

For now I'd recommend: don't use fileServerFolder , as it was mistakenly used for our plugin purpose. Go with the cosmicconfig and step_definitions property as explained here: https://github.com/TheBrainFamily/cypress-cucumber-preprocessor#configuration

It would be great to have a reproduction (just try to have the same structure and config and strip everything specific to your project). Unless it will be hard to me to verify the problem.

[ChrisBAshton]

Ok!

Here's a test repo: https://github.com/ChrisBAshton/cypress-cucumber-debug

The master version passes tests, but the broken branch fails them: ChrisBAshton/cypress-cucumber-debug#1

[lgandecki]

Cool! I will take a stab at this tonight.

[lgandecki]

@ChrisBAshton It would be great if you could verify this fix and I'd love to hear your thoughts about removing the ability to use fileServerFolder for good, instead of fixing it :)

[ChrisBAshton]

Hi @lgandecki I can confirm that fixes it 🎉 but it does feel wrong to pollute the package.json with Cypress-specific properties 🤔 can't this be handled in the cypress.json?

Oh, and happy to remove fileServerFolder from the config.

[lgandecki]

cypress.json is for cypress configuration, and you are not configuring cypress here, really. If you don't want to pollute package.json you have all the options that https://github.com/davidtheclark/cosmiconfig gives you