fix: gracefully handle errors in the e2e tests

[nicholasgriffintn]

Resolves NA

Description:

Made changes to the Cucumber config to fix how it responds to an error from SQS so that it is gracefully handled

Type of change:

• [x] Bug fix (non-breaking change which fixes an issue)
• [ ] New feature (non-breaking change which adds functionality)
• [ ] Breaking change (fix or feature that would cause existing functionality to change)

Why is this change required?:

When an error occurs in the E2E tests, the Consumer was not correctly stopped and so it was still listening for messages, this causes subsequent tests to fail as the previous run is handling messages instead of the new run.

Alongside this, Cucumber was not exiting after an error, which caused it to run indefinitely.

Code changes:

• Changed config to force Cucumber to exit
• Added After to the tests to stop the Consumer after a test has finished running

---

Checklist:

• [x] My code follows the code style of this project.
• [ ] My change requires a change to the documentation.
• [ ] I have updated the documentation accordingly.
• [x] I have read the CONTRIBUTING document.
• [ ] I have added tests to cover my changes.
• [x] All new and existing tests passed.

[codeclimate[bot]]

Code Climate has analyzed commit 0e1b1391 and detected 0 issues on this pull request.

The test coverage on the diff in this pull request is 100.0% (80% is the threshold).

This pull request will bring the total coverage in the repository to 94.3% (0.0% change).

View more on Code Climate.