-
Notifications
You must be signed in to change notification settings - Fork 10
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.
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
Integration Tests - Replace "echo" in the tests with "log INFO" #31
Comments
@karolh2000 Do we want to do this globally? I found From log INFO "Integration tests are running using this version of Pyrsia: ${latest_commit}"
log INFO "For more details, refer to https://github.com/pyrsia/pyrsia/commit/$(echo "${latest_commit}" | awk '{print $1}') (This link works properly only when you test pyrsia/pyrsia repository.)"
echo "Building the Pyrsia CLI sources (Pyrsia CLI source dir: $PYRSIA_TEMP_DIR), it might take a while..." >&3
cargo build -q --profile=release --package=pyrsia_cli --manifest-path=$PYRSIA_TEMP_DIR/Cargo.toml |
@efrisch |
Suggestion details
Since we introduced a logger framework in the integration tests we should go ahead and replace all log messages which are printed with "echo" with "log INFO".
Additional details
No response
The text was updated successfully, but these errors were encountered: