mirror of
https://github.com/rwf2/Rocket.git
synced 2025-01-14 13:39:13 +00:00
4e06ee64aa
Prior to this commit, it was not possible to test Rocket crates in production mode without setting a global secret key or bypassing secret key checking - the testing script did the latter. The consequence is that it became impossible to test secret key related failures because the tests passed regardless. This commit undoes this. As a consequence, all tests are now aware of the difference between debug and release configurations, the latter of which validates 'secret_key' by default. New 'Client::debug()' and 'Client::debug_with()' simplify creating an instance of 'Client' with configuration in debug mode to avoid undesired test failures. The summary of changes in this commit are: * Config 'secret_key' success and failure are now tested. * 'secret_key' validation was moved to pre-launch from 'Config:from()'. * 'Config::from()' only extracts the config. * Added 'Config::try_from()' for non-panicking extraction. * 'Config' now knows the profile it was extracted from. * The 'Config' provider sets a profile of 'Config.profile'. * 'Rocket', 'Client', 'Fairings', implement 'Debug'. * 'fairing::Info' implements 'Copy', 'Clone'. * 'Fairings' keeps track of, logs attach fairings. * 'Rocket::reconfigure()' was added to allow modifying a config. Internally, the testing script was refactored to properly test the codebase with the new changes. In particular, it no longer sets a rustc 'cfg' to avoid secret-key checking. Resolves #1543. Fixes #1564.
51 lines
1.3 KiB
Bash
Executable File
51 lines
1.3 KiB
Bash
Executable File
#! /usr/bin/env bash
|
|
set -e
|
|
|
|
#
|
|
# Publishes the current versions of all Rocket crates to crates.io.
|
|
#
|
|
|
|
# Brings in _ROOT, _DIR, _DIRS globals.
|
|
SCRIPT_DIR="$( cd "$( dirname "${BASH_SOURCE[0]}" )" && pwd )"
|
|
source "${SCRIPT_DIR}/config.sh"
|
|
|
|
function strip_dev_dependencies() {
|
|
perl -i.bak -p0e 's/\[dev-dependencies\].*//smg' "${1}/Cargo.toml"
|
|
}
|
|
|
|
function restore_dev_dependencies() {
|
|
mv "${1}/Cargo.toml.bak" "${1}/Cargo.toml"
|
|
}
|
|
|
|
if ! [ -z "$(git status --porcelain)" ]; then
|
|
echo "There are uncommitted changes! Aborting."
|
|
exit 1
|
|
fi
|
|
|
|
# Ensure everything passes before trying to publish.
|
|
echo ":::: Running complete test suite..."
|
|
cargo clean
|
|
bash "${SCRIPT_DIR}/test.sh" --all
|
|
bash "${SCRIPT_DIR}/test.sh" --all --release
|
|
|
|
# Temporarily remove dev-dependencies so crates.io verifies.
|
|
echo ":::: Stripping [dev-dependencies]..."
|
|
for dir in "${ALL_PROJECT_DIRS[@]}"; do
|
|
strip_dev_dependencies "${dir}"
|
|
done
|
|
|
|
# Publish all the things.
|
|
for dir in "${ALL_PROJECT_DIRS[@]}"; do
|
|
pushd "${dir}"
|
|
echo ":::: Publishing '${dir}'..."
|
|
# We already checked things ourselves. Don't spend time reverifying.
|
|
cargo publish --no-verify --allow-dirty ${@:1}
|
|
popd
|
|
done
|
|
|
|
# Restore dev-dependencies.
|
|
echo ":::: Restoring [dev-dependencies]..."
|
|
for dir in "${ALL_PROJECT_DIRS[@]}"; do
|
|
restore_dev_dependencies "${dir}"
|
|
done
|