FlowFuse / flowfuse

Build bespoke, flexible, and resilient manufacturing low-code applications with FlowFuse and Node-RED
https://flowfuse.com
Other
246 stars 59 forks source link

Update Test docs after adding the SMTP server for e2e tests #4099

Open cstns opened 3 days ago

cstns commented 3 days ago

Description

After the addition of https://github.com/FlowFuse/flowfuse/issues/3999, the e2e test dependencies were changed and are not reflected in the docs.

The docs should mention the fact that docker is now required to be installed locally in order to run the e2e test suites

Epic/Story

No response

Have you provided an initial effort estimate for this issue?

I have provided an initial effort estimate

knolleary commented 3 days ago

I didn't know docker was now a hard requirement. Is there anyway to bypass those tests if you don't have docker installed?

cstns commented 3 days ago

There is a way to parametrize the the tests in order to avoid having docker installed (using it in CI) but there will be some tests that depend on it which will fail. I haven't updated the scripts to run without the smtp server but it can easily be done