Skip to content

CCM-14195: Adding respect of the test exit code #101

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
damientobin1 wants to merge 1 commit into from
Closed

CCM-14195: Adding respect of the test exit code #101

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
13 changes: 11 additions & 2 deletions scripts/run.sh
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -41,13 +41,19 @@ fi
# Set up Python virtual environment and install dependencies
python -m venv .venv \
&& source .venv/bin/activate \
&& poetry install \
&& poetry run pytest --html=tests/evidence/report.html --self-contained-html --capture=tee-sys
&& poetry install

# Run pytest, capturing the exit code so it can be preserved after the S3 upload
set +e
poetry run pytest --html=tests/evidence/report.html --self-contained-html --capture=tee-sys
PYTEST_EXIT_CODE=$?
set -e

# Unset AWS credentials to drop back to default profile
unset AWS_ACCESS_KEY_ID AWS_SECRET_ACCESS_KEY AWS_SESSION_TOKEN

# Upload test evidence to S3 with environment/timestamp prefix
# This always runs regardless of test outcome so the report is always available
TIMESTAMP=$(date +%Y%m%d%H%M%S)
S3_PREFIX="release-tests/${ENVIRONMENT}/${TIMESTAMP}/"

Expand All @@ -57,3 +63,6 @@ if [ -d "tests/evidence" ]; then
else
echo "No evidence directory found to upload."
fi

# Exit with pytest's code so the container exit code reflects the test result
exit $PYTEST_EXIT_CODE