Skip to content
This repository was archived by the owner on Mar 1, 2023. It is now read-only.

Smoke testing after deployment #270

Merged
merged 5 commits into from
Jun 15, 2020
Merged

Smoke testing after deployment #270

merged 5 commits into from
Jun 15, 2020

Conversation

j-lea
Copy link
Contributor

@j-lea j-lea commented Jun 4, 2020

  • A short explanation of the proposed change:

Create a smoke test (create retro, add items, archive retro) for Alex to check that his deployment has succeeded, as mentioned in #35.

This smoke test can be run with the new smoke-test.sh script, which runs the test inside a new Docker container, so that Alex does not need to install any dependencies.

Note: this test is intended to be run during the test-package script, however this is waiting on some pipeline changes so it does not cause the final Travis step to run longer than 50 minutes and time out.

  • An explanation of the use cases your change solves

When Alex has deployed an instance of postfacto but misconfigured it, this will enable him to quickly tell that the deployment has failed, rather than finding out from participants later.

  • I have reviewed the contributing guide

  • I have made this pull request to the master branch

  • I have run all the tests using ./test.sh.

  • I have added the copyright headers to each new file added

  • I have given myself credit in the humans.txt file (assuming I want to)

Jenny Lea added 3 commits June 3, 2020 12:23
Signed-off-by: Marco Garofalo <[email protected]>
Signed-off-by: Othman Alkhamra <[email protected]>
… how to run the smoke tests.

Signed-off-by: Marco Garofalo <[email protected]>
Signed-off-by: Othman Alkhamra <[email protected]>
@j-lea j-lea marked this pull request as ready for review June 4, 2020 11:32
@butzopower butzopower merged commit 4b1ee64 into master Jun 15, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants